On 20.10.20 13:41, Paul Durrant wrote:
Hi Paul
Sorry for the late response.
-----Original Message-----
From: Oleksandr Tyshchenko <olekst...@gmail.com>
Sent: 15 October 2020 17:44
To: xen-devel@lists.xenproject.org
Cc: Oleksandr Tyshchenko <oleksandr_tyshche...@epam.com>; Paul Durrant
<p...@xen.org>; Jan Beulich
<jbeul...@suse.com>; Andrew Cooper <andrew.coop...@citrix.com>; Roger Pau Monné
<roger....@citrix.com>; Wei Liu <w...@xen.org>; Julien Grall <jul...@xen.org>;
Stefano Stabellini
<sstabell...@kernel.org>; Julien Grall <julien.gr...@arm.com>
Subject: [PATCH V2 08/23] xen/ioreq: Introduce ioreq_params to abstract
accesses to arch.hvm.params
From: Oleksandr Tyshchenko <oleksandr_tyshche...@epam.com>
We don't want to move HVM params field out of *arch.hvm* in this particular
case as although it stores a few IOREQ params, it is not a (completely)
IOREQ stuff and is specific to the architecture. Instead, abstract
accesses by the proposed macro.
This is a follow up action to reduce layering violation in the common code.
Signed-off-by: Oleksandr Tyshchenko <oleksandr_tyshche...@epam.com>
CC: Julien Grall <julien.gr...@arm.com>
Keeping the 'legacy' magic page code under an x86 ioreq.c would avoid the need
for this patch.
In that case, yes, agree.
--
Regards,
Oleksandr Tyshchenko
