On 20.10.20 13:41, Paul Durrant wrote:
Hi Paul
Sorry for the late response.
-----Original Message-----
From: Oleksandr Tyshchenko <[email protected]>
Sent: 15 October 2020 17:44
To: [email protected]
Cc: Oleksandr Tyshchenko <[email protected]>; Paul Durrant
<[email protected]>; Jan Beulich
<[email protected]>; Andrew Cooper <[email protected]>; Roger Pau Monné
<[email protected]>; Wei Liu <[email protected]>; Julien Grall <[email protected]>;
Stefano Stabellini
<[email protected]>; Julien Grall <[email protected]>
Subject: [PATCH V2 08/23] xen/ioreq: Introduce ioreq_params to abstract
accesses to arch.hvm.params
From: Oleksandr Tyshchenko <[email protected]>
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 <[email protected]>
CC: Julien Grall <[email protected]>
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
