[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/3] xen/arm, tools: Add a new HVM_PARAM_MAGIC_BASE_PFN key in HVMOP
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Henry Wang <xin.wang2@xxxxxxx>
- Date: Fri, 26 Apr 2024 14:30:23 +0800
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=suse.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
- 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=CJO5mcHwdPaiF8pZ44dYl57wpjCjF4DE9swaQwsptfo=; b=fwxiPQcxMt8Lyv8j0t0Vsau0lLxhCjb0bnqrPoQKKf9od6PVZ2ovUeGizoU2Lbtj/0eETTh3jjtURTvxIK3EkUE28nbx309FlQg2YpmoLh9DYsBLNDtvRVcAHvSWpfzKsw4jq+quYLx4+cdcmZL0eKXqEXNWvJrj0rzsppyzXToxlfoDhzkJ+5x31sgvCgouLxPw+tyCOfVUy/KSpVEla93PjFNzsfcJCXJUToNkpOeTI8hFcpTb74NMcZDZfsaCZXTF9dOrqdjQhejaoVenoSHowLj0wugzaj1rKRw+0ac2Tr7p6lycGakowH2ZZKJ3GtSSxKOsFCDza8XK4FBs6w==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=O6Rua/Fv/9YrkTAoHd/lWzjuQS1Z3dkk7Z7pq0+oQRxc3Pwa0ANmy8t/tglBJDu9bJmRescbvX2T/naBq1deyhRwPz/zqv8nNGs+YnBkRYsHFkXdiQlj5SGVHfYDtZTKs3CgyXxVZNenJs6KzjQbQDN7GmTsOXMXrZASWkgj636J2t7vGupbXSUNVRwZTRm2v+QzAt4SuXfLUzbbnWPRLoubfF0b5FDpPdOQG2Qv9uQTUNXsOYmYtmboEJFVNwKETJqVIRX9WXhjZrmJSCgal08p15GdP0qnrFHz2/Ky1VwIrVbPX2vbkpBND5p5Ns7kYUbHFJgisBUmaStxnAF9UA==
- Cc: Anthony PERARD <anthony.perard@xxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Alec Kwapis <alec.kwapis@xxxxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Fri, 26 Apr 2024 06:30:36 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
Hi Jan,
On 4/26/2024 2:21 PM, Jan Beulich wrote:
On 26.04.2024 05:14, Henry Wang wrote:
--- a/xen/include/public/hvm/params.h
+++ b/xen/include/public/hvm/params.h
@@ -76,6 +76,7 @@
*/
#define HVM_PARAM_STORE_PFN 1
#define HVM_PARAM_STORE_EVTCHN 2
+#define HVM_PARAM_MAGIC_BASE_PFN 3
#define HVM_PARAM_IOREQ_PFN 5
Considering all adjacent values are used, it is overwhelmingly likely that
3 was once used, too. Such re-use needs to be done carefully. Since you
need this for Arm only, that's likely okay, but doesn't go without (a)
saying and (b) considering the possible future case of dom0less becoming
arch-agnostic, or hyperlaunch wanting to extend the scope. Plus (c) imo
this also needs at least a comment, maybe even an #ifdef, seeing how x86-
focused most of the rest of this header is.
Thanks for the feedback. These make sense. I think probably
dom0less/hyperlaunch will have similar use cases so the number 3 can be
reused at that time. Therefore, in v2, I will add more description in
commit message, a comment on top of this macro and protect it with
#ifdef. Hope this will address your concern. Thanks.
Kind regards,
Henry
Jan
|
