[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [XEN][PATCH] x86/paging: replace !paging_mode_hap() with paging_mode_shadow()


  • To: Grygorii Strashko <grygorii_strashko@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Jason Andryuk <jason.andryuk@xxxxxxx>
  • Date: Fri, 14 Nov 2025 18:21:16 -0500
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=epam.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=arcselector10001; 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=4QrBhMJvFc9AU5l8tuxzY/KBXfv9uFBDoGcf0mObcf4=; b=UFim9dZfctSMk5h1+5ZvibKoGkW4nIrAn7It+6JgJHxIF7Bpt0I/JSVx1ZHD2QO/0dhxQd37q6IIel7tlug07nC69vO7qfz6W5Wn8KgHf3NTVcn5tmpV/TSofmRUqjwLw6bQTTkQRgbjQ+TLC1L1q36pAk4nD11Cdz0fBj+0ZCNXauMlMMF9xB+Y/TCK6LHtr0sQmRncLeyHqfaE/pLJRuUP3U3Q2J0hn2USgvu56IcqEO9hfyMhiORgJJnjebtKPNPJquWL/pokc2ajE+ISGSFnayng6eXTSBp483BOG2R18mD6D5Lrg3khuUsRIfk7ZS1aHzlYjMIQbHhAeL0sig==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=GTY35LWkMYKq0EgktfczgZ6ogN3Zn5vXVCvgJVaaoqs2xIYA/qny/DDSwh9FrpKqaEhtGam9kTN4WT+DlAyCUn6re+hHoJba8oQeIPi48Y8Db1jklbBWs0gAVIR7TZ2kiqEBRzm1YfoTrXfD5CbfYeIZ5AeLy+h3ckIbY82bmNIFMCx5IecDfSPac5yIgNW15ZJkzt+PIdK8JimLD1PgvnemXiL7QlKTXqlmOEyYSwcYICMr9l60sw23b/yqZ9ItnfsfUASdV//VSPbBvA97F4257x6pJYHzAztxQGEgXM53tbYje3O+njoCFtXuN66NJ0+IUbhlr6JzayJXHMdtpw==
  • Cc: Jan Beulich <jbeulich@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Delivery-date: Sat, 15 Nov 2025 00:39:29 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 2025-11-14 09:44, Grygorii Strashko wrote:
From: Grygorii Strashko <grygorii_strashko@xxxxxxxx>

Now Xen supports only two paging modes: HAP and SHADOW, so
!paging_mode_hap() is actually means paging_mode_shadow().

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/20251111200958.3576341-3-grygorii_strashko@xxxxxxxx/
Signed-off-by: Grygorii Strashko <grygorii_strashko@xxxxxxxx>

Reviewed-by: Jason Andryuk <jason.andryuk@xxxxxxx>



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.