All of lore.kernel.org
 help / color / mirror / Atom feed
From: Xenia Ragiadakou <burzalodowa@gmail.com>
To: Jan Beulich <jbeulich@suse.com>
Cc: "Jun Nakajima" <jun.nakajima@intel.com>,
	"Kevin Tian" <kevin.tian@intel.com>,
	"Andrew Cooper" <andrew.cooper3@citrix.com>,
	"Roger Pau Monné" <roger.pau@citrix.com>, "Wei Liu" <wl@xen.org>,
	xen-devel@lists.xenproject.org
Subject: Re: [PATCH v2 5/9] x86/vmx: reduce scope of GAS_VMX_OP definition
Date: Thu, 23 Feb 2023 12:45:49 +0200	[thread overview]
Message-ID: <19079d03-748a-fe5b-9941-84ede2bf8554@gmail.com> (raw)
In-Reply-To: <f9841c7d-c9aa-7754-1a90-4a069c94f3c2@suse.com>


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@gmail.com>
> 
> 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 have 
deleted the #include <asm/hvm/vmx/vmcs.h>) so I will squeeze them in v3.

> 
> Jan

-- 
Xenia


  reply	other threads:[~2023-02-23 10:46 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-22 12:00 [PATCH v2 0/9] x86/hvm: {svm,vmx}.{c,h} cleanup Xenia Ragiadakou
2023-02-22 12:00 ` [PATCH v2 1/9] x86/svm: remove unused forward declaration of struct vcpu from svm.h Xenia Ragiadakou
2023-02-22 12:59   ` Jan Beulich
2023-02-23 10:50     ` Andrew Cooper
2023-02-23 11:00       ` Xenia Ragiadakou
2023-02-22 12:00 ` [PATCH v2 2/9] x86/svm: opencode SVM_PAUSE{FILTER,THRESH}_INIT Xenia Ragiadakou
2023-02-23 10:19   ` Jan Beulich
2023-02-22 12:00 ` [PATCH v2 3/9] x86/svm: move declarations used only by svm code from svm.h to private header Xenia Ragiadakou
2023-02-23 10:24   ` Jan Beulich
2023-02-23 11:16   ` Andrew Cooper
2023-02-23 14:40     ` Xenia Ragiadakou
2023-02-22 12:00 ` [PATCH v2 4/9] x86/vmx: remove unused included headers from vmx.h Xenia Ragiadakou
2023-02-23 10:29   ` Jan Beulich
2023-02-23 10:42     ` Xenia Ragiadakou
2023-02-22 12:00 ` [PATCH v2 5/9] x86/vmx: reduce scope of GAS_VMX_OP definition Xenia Ragiadakou
2023-02-23 10:31   ` Jan Beulich
2023-02-23 10:45     ` Xenia Ragiadakou [this message]
2023-02-22 12:00 ` [PATCH v2 6/9] x86/vmx: move declations used only by vmx code from vmx.h to private header Xenia Ragiadakou
2023-02-23 10:47   ` Jan Beulich
2023-02-23 11:56     ` Andrew Cooper
2023-02-24  7:49     ` Xenia Ragiadakou
2023-02-24  8:26       ` Jan Beulich
2023-02-22 12:00 ` [PATCH v2 7/9] x86/vmx: remove unused included headers from vmx.c Xenia Ragiadakou
2023-02-23 11:16   ` Jan Beulich
2023-02-22 12:00 ` [PATCH v2 8/9] x86/vmx: declare nvmx_enqueue_n2_exceptions() static Xenia Ragiadakou
2023-02-23 11:19   ` Jan Beulich
2023-02-22 12:00 ` [PATCH v2 9/9] x86/vmx: move vmx_update_debug_state() in vmcs.c and declare it static Xenia Ragiadakou
2023-02-23 11:21   ` Jan Beulich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=19079d03-748a-fe5b-9941-84ede2bf8554@gmail.com \
    --to=burzalodowa@gmail.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=jbeulich@suse.com \
    --cc=jun.nakajima@intel.com \
    --cc=kevin.tian@intel.com \
    --cc=roger.pau@citrix.com \
    --cc=wl@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.