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 ... >> 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
