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

RE: [PATCH v4 07/14] xen/arm32: head: Jump to the runtime mapping in enable_mmu()


  • To: Julien Grall <julien@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Henry Wang <Henry.Wang@xxxxxxx>
  • Date: Sat, 14 Jan 2023 01:33:39 +0000
  • Accept-language: zh-CN, en-US
  • 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=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=OTjK6UGNM9axJgPFI4jB7B6Gis3T+zUgCHh8Qdg4OWY=; b=LOO/zDiKXOzIeoJoS3DEzl3sR1mO8q3bkpUFsOTG6MeywZIkHagpFonrGZelWj/yE1ibSPZQRi7k0e9xgCsug6GFo7qsNVSzLLko6cGdiBnM8ww6ZFE21RHGBZ7kTFBsUUyi6WvICSnGU7FNYHZOnoPgaWAVjOprgJ6Y5htDYDT5BFDHmswkXISxK3z9NMgniV64QaUFx012ilUgGxyfulpZJj3cm69LgO76xRGfjra1ijYoUjaOVf+TmAgR0FxL2rSHXFEDRSvw5fuoQTARGBoCytGW9aqumNYCpK16+YpTSX/NCD/AGGsKRYz7h4dhnk+2Eq1+mkE6dTf91tr1wQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bOY+Glw9p4udld8+47Si4bUzm0ckqOswZZUp7YRHUG5Gh47foMJ7xWRb7xk1JG5zy/afUdlXpXF7+TBwLQ2YWtGg/q7o1n8/b99/a4mu/7cJfsL9L2Rpo//G3PNNDOipSetuT2rjC8VwCjaC5NdhfQ97WPVgJHVd/s+of8hAPmG8O9AHcWs7oMeCMOhI0fQ+UuP1gCbdGXQqosqdVJZ36CADSuD2+g2hBWP675NhvHJ413qnN5xMiOfyvRc6FaSYYP3qODSDwaqR1yhsRnJ+4m5zSebLefCF4yfrNv1VEnHt67O/0iCyxFunG0S+bNiAEMskvx+VINi3dLBjZX+TbA==
  • Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
  • Cc: Luca Fancellu <Luca.Fancellu@xxxxxxx>, Julien Grall <jgrall@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
  • Delivery-date: Sat, 14 Jan 2023 01:34:06 +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: AQHZJzd9G8hkGu/MR0eHbN40F8df366dDCsw
  • Thread-topic: [PATCH v4 07/14] xen/arm32: head: Jump to the runtime mapping in enable_mmu()

Hi Julien,

> -----Original Message-----
> Subject: [PATCH v4 07/14] xen/arm32: head: Jump to the runtime mapping in
> enable_mmu()
> 
> From: Julien Grall <jgrall@xxxxxxxxxx>
> 
> At the moment, enable_mmu() will return to an address in the 1:1 mapping
> and each path is responsible to switch to the runtime mapping.
> 
> In a follow-up patch, the behavior to switch to the runtime mapping
> will become more complex. So to avoid more code/comment duplication,
> move the switch in enable_mmu().
> 
> Lastly, take the opportunity to replace load from literal pool with
> mov_w.
> 
> Signed-off-by: Julien Grall <jgrall@xxxxxxxxxx>
> Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

I've tested this patch on FVP in arm32 execution mode, and
this patch is good, so:

Tested-by: Henry Wang <Henry.Wang@xxxxxxx>

Kind regards,
Henry



 


Rackspace

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