[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v6 02/11] xen/arm: add iounmap after initrd has been loaded in domain_build
- To: Julien Grall <julien@xxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Henry Wang <Henry.Wang@xxxxxxx>
- Date: Mon, 7 Nov 2022 09:11:39 +0000
- Accept-language: zh-CN, en-US
- Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
- Arc-message-signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=FKXbMp+b50VBowQGYpFZCnf4rjoV+cMblfYMIeEGgrI=; b=mT5x9DZOM+KSlW64kjYKwlOkjRuVe+Fi7csnvvvdywKFRO+VdytUeFpCQ5qo1SLjWjUbYng3cpaUsSyfkgHiCrTF9QnJQ2Ff5FuHPyOqEaI+lULBmIWTeW0QhhGVSPbDbyk1rFCY++z895G1yPl5xRjIhwiEO6tYifCnQCX4XHYTX18nttYUABKXn+SBGapXawBfORVwqTqOOAB3IqbhnN6uD/pklSq3zBu13k5uQqO4YmLK/i1sFNkvEy58Tf8vJoIVFpItECtdQpdC3UK+ucAiXP9Cw+a6gnsfGwnY/33vgk3oSqD+OivKCE9yU212n5RDltEy+C7WgSB/ysF25Q==
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=FKXbMp+b50VBowQGYpFZCnf4rjoV+cMblfYMIeEGgrI=; b=MVxswJ7v475WeP9b2X2l3eVz4ozGh57StbvG+LpDvLw8I7q3hmrbfxkU9Cbk/gLQxbZx0m45yUX/oX/8KddNgtC5nnQHVNnly0fswq/gDkwArr6UvX6uOQdOkpqk1Ln1D2kpk/0nrew2n0wIoY0/J/711jxH42KCKwFfsXrhLAkP3QlJpMxJUuzkqfgXCLfj6/ulRvil8j5XhUGd55O9MG/z/LCHRGr62KVFKosZLvJNDfb5jB2xnL6Z5VRzgzVavrZjydRyKvQgVj0QRKbkcCT6tw9/6Wf7eom9/IebSDj6FxUst2Pq0YQc8Rm6lEf6qIWK2qMlInh3PBswVp/f+Q==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=crOfXNKqv66+CD8gDo97XbgyoSJtRoAeid+Pa0NISR2+vsTIn5d7ZCpeqziqJzbRN+FlhitDpNPFf/Oj7ZzpdysdWvlId7m4xC37UqMHjC8O1GMOnsfOUj7KJ6+y9IlFLpNKHBHOJCckjk27kAXQSpX7oQNfpSlLbRG0ov5A5xW7DYL21eI9oKRdmfoW70K0CCM4Q6efLQqqt038h+vnoDA6P4OOgY4h803W4d5uhQDNq5mvlk0inYS6bLaZcjzTLN5RVPGewsDbYXLq+DWWN+7Q5ctmBz5mA5Dzgqj6/MHsJIQzVx6nWH1FwEVyRRuczbqSVEPSWDLszhEE6vxLOQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YlxnbOpHHo++Rx/LB/vtAqBqwrCN/7c8Yctt7WOfJ0EbKfR9/06e6wcpjFp3BH8iDYAjhRDR+Y5VfrEVbPIakga7s+DadK8ofB6smbYNjEjdBT3rVJXkOGreG74eKUwmSsU0yADkJJ5v5QUlLag8Uk0FnH4x2yo8leD0ToaKjZVm4vEi2DDZJ/fI/ILb2mnDxtcqR8J+54JUdRKE/l3WfYrjFY6o3iFhu75VGPQN9vnKpf5mdvkCnbeTqMVguGsHO4BV83L12fPu77taQeDIdQtLt59G/BgPQBAR0wC1myEzQXkKeNjk6QMWAZqh1qid3ILL5Ie4UOh+gwgWdBdqag==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: nd <nd@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
- Delivery-date: Mon, 07 Nov 2022 09:11:57 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Nodisclaimer: true
- Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Thread-index: AQHY8DVtbyg+f3V3FU++xEj+lazfQ64yQiqAgABu86CAAH/OgIAAAEDA
- Thread-topic: [PATCH v6 02/11] xen/arm: add iounmap after initrd has been loaded in domain_build
Hi Julien,
> -----Original Message-----
> Subject: Re: [PATCH v6 02/11] xen/arm: add iounmap after initrd has been
> loaded in domain_build
> >> (+ Henry)
> >> I think this wants to be in 4.17. This will avoid Xen to have two
> >> mappings with different caching attribute (initrd is part of the RAM and
> >> therefore directmap).
> >
> > Sounds good to me, I am wondering if we need to include also patch #1
> though.
>
> If we were earlier in the release, I would have say yes. But now, it is
> a no as this just a cleanup (having extra definitions is harmless).
Thanks for your confirmation :) No problem.
Kind regards,
Henry
|