Hi Jan,
On 17.11.25 11:47, Jan Beulich wrote:
On 15.11.2025 00:21, Jason Andryuk wrote:
On 2025-11-14 09:44, Grygorii Strashko wrote:
From: Grygorii Strashko <[email protected]>
Now Xen supports only two paging modes: HAP and SHADOW, so
!paging_mode_hap() is actually means paging_mode_shadow().
For HVM, that is. For PV both could produce "false". Luckily the code
changes are all in line with that, but somewhere this detail wants
mentioning imo (as it's also not part of the patch subject). Happy to
make a small adjustment while committing; then ...
Sure. Thank you very much.
For an abstract, future case of there being a 3rd paging mode it is also
better to explicitly mention checked paging mode (SHADOW) instead of using
negative check of another paging mode (HAP).
Hence, s/!paging_mode_hap()/paging_mode_shadow() which also allows DCE drop
unused code when SHADOW_PAGING=n.
The !paging_mode_hap() in hap.c not changed as HAP is checking for itself
to be enabled.
Inspired by [1].
[1]
https://patchwork.kernel.org/project/xen-devel/patch/[email protected]/
Signed-off-by: Grygorii Strashko <[email protected]>
Reviewed-by: Jason Andryuk <[email protected]>
Acked-by: Jan Beulich <[email protected]>
Jan
--
Best regards,
-grygorii
