[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/5] xen/arm: only map the init text section RW in free_init_memory
- To: Julien Grall <julien@xxxxxxx>
- From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
- Date: Mon, 18 Nov 2024 09:27:42 +0000
- Accept-language: en-GB, 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=arm.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=arcselector10001; 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=l0oLy9dmSiRMZjrhCGjCbaX76T82uwBQA0SOrJp3Uxw=; b=iWhpvorJ5z4wR3RGRrWD/UA9FPv/zD3h5waRiKjhHEkedquVjpr4sg1BYoRYcAT0Gra1TRqTCZdpYRK+y7ap2sztnDzM/x4YppVW/6T67lLOgw25C9ZyzaepWJomjIBxcRv7rV4YQjY5/ROjp+m0SUgwGJNj8VR5CNRITiTduU4W0+Pn7KjwtPCEYTkz/ox08PfRdIV9Bhfl40IT43Cb8Toi3r6GZ54pAYbv9InW0pKMdkkSW+ZueV2yq/6L1WLmXdVOroqgPfJKy0KNiH413txsQr/6wJjD2kAQ7UjdAtGNiquGNP3qCgv8XHGdota0RhU7W5+LRMR9Gjwq6T+nzw==
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=l0oLy9dmSiRMZjrhCGjCbaX76T82uwBQA0SOrJp3Uxw=; b=RUnGxbT+Hpnd5b0kgB1SZ9gOZTYGpcV1zyS+bzByzFeZ1n5Z9IBrg9eo+IpwqIQe0WCRFcDcH795AEYhQQnwSs88+ut6hDb3Zc999SMpFvQuHkqwZDQk8gdrv0T6xTCbq/kQD/kVCoonFWyzYhjmG0EI3l7x44+k9ziyKwmanPK85tV3Lg/WzZQbInMymskOIUMEwFhQgOnuv6UQ7yRwluu56wBY8BiQyqIgKej6ZiOYGFQcNMRcdA+ETLO+V92ktaqhoVh/8tZkNqVaZvbF+wWxdamtBiG1M33w57RfoN3rv+jr9S1Xaft50jt8i494oGZQOOC7U2c5ryVMjHM21Q==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=pass; b=w9wyMP99Juit6KvdP+DNJ/Nm9qaTLYzmADjRm+JOeaTnN2TLE1nEGtABonziBcOWEWv2kFljdl7mJUc1JQ6Y6xAGKuyj2ofm4e0V9mhlkY/bIqi0MfBdXeJ+2fOF27acHf0O49VcU6AJGmCXQXk58u+JsPckDody8JF8WkZaQBwB/erH6tS+Zkd2REvlDI52NihOBQAjQ6CEICMDAEOwNou7GO29WydEpdZg+YpdM9kqAgte2uSEsaboRgUyeOYL7ZKFOyvOGXmZKec3bLsusIT8hVP3v5esX1KhCMNGGyCPpTjnATAXy5D+TMDR3MaxJPTaZ2CqgAq1zChlcYF+zQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=yxCv2TuyERX5nO3Mv4sHhNrfEM0PEvWNfeXDFSm+eHCAwBHi3xU6Q+VIeE5PXkvpAN6L+Yowv8G9d9LXuHs36sAiSzhuRKlJE2mQaBjnKro/w59KlrRkxuJ24AJGE/Vzb9X+y8JxuPKetQlluXc1CUNLxMF36fFkhYZ4xHT8+uECnVRysfp3QJP2Vf8UbJcHBOg/V3wPtR/exCpfAWDqx1phmNM8X6ryRATJpR19v5ddgiLWxpWvhSeRl2aMGY3LWWuCeD9AqjuLHxAHfrPIclaa+pyU1Q+ZJ6IFTPf/skxRjRWbACWT2GuNkgowLsZg8mMhdro/hgrPjMtynxLyTg==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Penny Zheng <Penny.Zheng@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
- Delivery-date: Mon, 18 Nov 2024 09:28:10 +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: AQHbN0xG58VLgb/dzkerHg/RLPURhLK7xh0AgAEDYgA=
- Thread-topic: [PATCH 3/5] xen/arm: only map the init text section RW in free_init_memory
Hi Julien,
> On 17 Nov 2024, at 17:58, Julien Grall <julien@xxxxxxx> wrote:
>
> Hi Luca,
>
> On 15/11/2024 10:50, Luca Fancellu wrote:
>> From: Penny Zheng <Penny.Zheng@xxxxxxx>
>> In free_init_memory, we do not need to map the whole init section RW,
>> as only init text section is mapped RO in boot time.
>
> So originally, this was done because the function was generic. But now this
> is MMU specific, we don't really gain that much during boot but will impair
> any work that would restrict some init further (for instance .init.rodata
> could be RO). So is it actually worth it?
Probably not, I’ll drop this one.
>
>> Signed-off-by: Luca Fancellu <luca.fancellu@xxxxxxx>
>
> Given the link below, why are Penny and Wei's signed-off-by are missing?
Internal policy, authorship is preserved though.
Cheers,
Luca
|
