[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 1/7] docs/arm: Document Xen booting protocol on Armv8-R


  • To: Julien Grall <julien@xxxxxxx>
  • From: Luca Fancellu <Luca.Fancellu@xxxxxxx>
  • Date: Tue, 6 May 2025 12:40:41 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 4.158.2.129) smtp.rcpttodomain=xen.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=TqxA7DL/8hKb3eyzF6bkdo41KMSU0BUos4XM8bLohQw=; b=Qsow8XggffT2ZiCtNbCYNZ/QBv9pF5r4Mdj+aqChzPWvnqng6SkgWfkzbldNSfoDJisgsaz3+Ku1n71Sz+4ymcRw2Il4uoU1Bw9eVzXzhJUBfwa7cdrSL919pKkhpjvA+VvFjMNgVXtknemH+LtugqYuuddcBQiGfhT6iRWJWq+eb8GCm1VYb+qG/zJqMCllEJ0AUDHZLjkXMTGSeXepkk4p03A9IJWfC/YXazArn8cLCNf9Svj69h6yB0a0icI6VmEPbyXmatLjp9llqAjmgiwTLzvyoLd4QAzdfYzyRs8QM+TY6RM2DwPhVlWkfIm3EDXB9puAbwdd4RkczowAXg==
  • 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=TqxA7DL/8hKb3eyzF6bkdo41KMSU0BUos4XM8bLohQw=; b=f7wH1iNSY6Teb7XKnt8Z/ZAzNGT2g9XCFjcpJWKQVNJNVOu0pMdj14t07BOSi9Sm/AzWLaaggCEHIEutzP3MJS+7pl/PfVXG55L7i4jpZGSspJDnK5YOjB8ibCCIziMTc5JbZhCTEBitqft3Bxu2yYH7cNkNfbNYva5LXCxp0SVWTu4Uef8zaih+QvCfok7hH+YsvxCsYwhwA3zufg3l3370+1wgT1WsOmJiJ74Lr2iWJTQEXD3X24bt+MfoNTRkvpUOvJHPBHZm2jVz7y50e/8KDxWASmw+zlnUe5KLoYzmFi6jqn6O3dvlzJbCrMIdp7SiXuhv+wrEA2v/AoZ0/g==
  • Arc-seal: i=2; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=pass; b=jFzI+PTumRQIb/e1xLLidEUqFziLDR+E1B3LjE5ccGfuPp38EGTN98pLBze5BiAwBNhHojSNm30cRosR7MhKBSvs77D2Gr7QAU+uQw/6EVyAvI5DGsAz8TPcnt5g69TjcAWBaz8iicjqJauDVsbIbDFi/Ua6YC96lDoust3xYAfme4vSfyx02plRZIazUkIvfpYcDf8R6CSQVMmZFRsgztj/3JzShTM3utBPWN2qA866UwIt6P251/p6m8cjE9N5ORnd3Qvpeori004F+cC2sApGPi3F86uSUIPRkQldHpRbdB+kx4OozW8l3AhvuevxqGGYipJely9jewlsSrjMOg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=JmtQlWW5R3TkbD0pd3TmPPj8eK/QGuKYeTAx+olVFX1IIeO8YBvhTL5M291ZBOLb037FsQA6vIu2A8X8yCDU+CNlWBWBPPyNIZ+tV7ZStIXIAsmfXS9r2N/6V/X9/8WfKVhXxujwhdAMVVchKAdwo47CsiSTzQfvV97C0S3O6BkbsECUsJTlN5Mbuz2Y4+EL1Ni9GALOvLLhkSN4/e5rHoy5cGoX2Mgdy0CWKNtDrS1OsbRNRr8QktcVGr6XD1jYsTliGUejg4bPnzCNMINbvRZ9ojq7rTwQVjqrCJduePY+nAXhmBi2IhJU81J0OYGKamNlxoLZ29xM7/L7rEz0Bg==
  • 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>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Ayan Kumar Halder <ayankuma@xxxxxxx>
  • Delivery-date: Tue, 06 May 2025 12:41:39 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Thread-index: AQHbuRpmN9lJogKQoUiVWa6WakCsQrPFhjEAgAALLQCAAAF7AIAAAvWA
  • Thread-topic: [PATCH v4 1/7] docs/arm: Document Xen booting protocol on Armv8-R

Hi Julien,

>> Just to be sure to be on the same page, are you suggesting these changes on 
>> the original file?
> 
> Yes with one tweak.
> 
> > > diff --git a/docs/misc/arm/booting.txt b/docs/misc/arm/booting.txt
>> index 21ae74837dcc..c00c651805d7 100644
>> --- a/docs/misc/arm/booting.txt
>> +++ b/docs/misc/arm/booting.txt
>> @@ -58,10 +58,14 @@ Firmware/bootloader requirements
>>    Xen relies on some settings the firmware has to configure in EL3 before 
>> starting Xen.
> 
> I think you want to update this sentence to remove the reference to EL3. Even 
> on A-profile EL3 is not mandatory (I vaguely remember one of the platform I 
> worked on had no EL3).
> 
>>  -* Xen must be entered in NS EL2 mode
>> +* Xen must be entered in:
>> +  * Non-Secure EL2 mode for Armv8-A Arm64 and Arm32, Armv8-R Arm32.
>> +  * Secure EL2 mode for Armv8-R Arm64.
>>    * The bit SCR_EL3.HCE (resp. SCR.HCE for 32-bit ARM) must be set to 1.
> 
> And then here: "When EL3 is supported, ...". This would also cover the 
> R-profile change.
> 

Thanks for the clarification, @Michal, @Ayan, are you ok if I retain your R-by 
with these changes or
should I drop it?


> Cheers,
> 
>>  +* Xen must be entered with MMU/MPU off and data cache disabled 
>> (SCTLR_EL2.M bit
>> +  and SCTLR_EL2.C set to 0).
>>    [1] linux/Documentation/arm/booting.rst
>>  Latest version: 
>> http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/arch/arm/booting.rst

Cheers,
Luca





 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.