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

Re: [PATCH v2 5/9] x86/vmx: reduce scope of GAS_VMX_OP definition

To: Jan Beulich <jbeulich@xxxxxxxx>
From: Xenia Ragiadakou <burzalodowa@xxxxxxxxx>
Date: Thu, 23 Feb 2023 12:45:49 +0200
Cc: Jun Nakajima <jun.nakajima@xxxxxxxxx>, Kevin Tian <kevin.tian@xxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 23 Feb 2023 10:46:04 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>


On 2/23/23 12:31, Jan Beulich wrote:

On 22.02.2023 13:00, Xenia Ragiadakou wrote:

Since GAS_VMX_OP is used only by vm{read,write}_safe, move its definition
just above those functions and undefine it after use.

Signed-off-by: Xenia Ragiadakou <burzalodowa@xxxxxxxxx>


This can easily be done as part of the next patch, with less code churn
overall.

Sure. I ve spotted a bug anyway in the next patch (I should not havedeleted the #include <asm/hvm/vmx/vmcs.h>) so I will squeeze them in v3.

Jan


--
Xenia

References:
- [PATCH v2 0/9] x86/hvm: {svm,vmx}.{c,h} cleanup
  - From: Xenia Ragiadakou
- [PATCH v2 5/9] x86/vmx: reduce scope of GAS_VMX_OP definition
  - From: Xenia Ragiadakou
- Re: [PATCH v2 5/9] x86/vmx: reduce scope of GAS_VMX_OP definition
  - From: Jan Beulich

Prev by Date: Re: [PATCH v2 4/9] x86/vmx: remove unused included headers from vmx.h
Next by Date: Re: [PATCH v2 6/9] x86/vmx: move declations used only by vmx code from vmx.h to private header
Previous by thread: Re: [PATCH v2 5/9] x86/vmx: reduce scope of GAS_VMX_OP definition
Next by thread: [PATCH v2 6/9] x86/vmx: move declations used only by vmx code from vmx.h to private header
Index(es):
- Date
- Thread

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