From: Alexandru Stefan ISAILA <aisaila@bitdefender.com>
To: "JBeulich@suse.com" <JBeulich@suse.com>
Cc: "Ian.Jackson@citrix.com" <Ian.Jackson@citrix.com>,
"andrew.cooper3@citrix.com" <andrew.cooper3@citrix.com>,
"wei.liu2@citrix.com" <wei.liu2@citrix.com>,
"paul.durrant@citrix.com" <paul.durrant@citrix.com>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [PATCH v7 08/15] x86/cpu: Remove loop form vmce_save_vcpu_ctxt() func
Date: Thu, 21 Jun 2018 07:47:33 +0000 [thread overview]
Message-ID: <1529567253.4608.19.camel@bitdefender.com> (raw)
In-Reply-To: <5B2B561902000078001CC95C@prv1-mh.provo.novell.com>
On Jo, 2018-06-21 at 01:39 -0600, Jan Beulich wrote:
> >
> > >
> > > >
> > > > On 20.06.18 at 10:56, <aisaila@bitdefender.com> wrote:
> > So the final plan for the series is like this:
> > - Base everything on Roger's series
> > - Keep patches 1-7
> > - Have patch 8 add an extra arg to HVM_REGISTER_SAVE_RESTORE
> > and hvm_sr_handlers
> > - Have patch 9 like the patch 15 form now and have it call the
> > save_one handlers
> > - Have the next patches remove the redundant save handlers and
> > rename the save one
> > - The final patch should remove the extra arg. This one can be
> > kept or not.
> Sounds reasonable, albeit I have to admit I'm not sure what the last
> sentence is supposed to mean. Either it is possible to eliminate the
> extra arg, then you should do so, or it is not possible (in which
> case
> you can't possibly add a respective patch to the series).
I wanted to say that the last patch will be independent and it can go
upstream or not because you said that you are not sure if the extra
args should be removed or not.
Alex
________________________
This email was scanned by Bitdefender
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
next prev parent reply other threads:[~2018-06-21 7:47 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-07 14:59 [PATCH v7 00/15] x86/domctl: Save info for one vcpu instance Alexandru Isaila
2018-06-07 14:59 ` [PATCH v7 01/15] x86/cpu: Introduce vmce_save_vcpu_ctxt_one() func Alexandru Isaila
2018-06-08 14:45 ` Jan Beulich
2018-06-07 14:59 ` [PATCH v7 02/15] x86/hvm: Introduce hvm_save_tsc_adjust_one() func Alexandru Isaila
2018-06-07 14:59 ` [PATCH v7 03/15] x86/hvm: Introduce hvm_save_cpu_ctxt_one func Alexandru Isaila
2018-06-08 14:47 ` Jan Beulich
2018-06-07 14:59 ` [PATCH v7 04/15] x86/hvm: Introduce hvm_save_cpu_xsave_states_one Alexandru Isaila
2018-06-08 14:49 ` Jan Beulich
2018-06-07 14:59 ` [PATCH v7 05/15] x86/hvm: Introduce hvm_save_cpu_msrs_one func Alexandru Isaila
2018-06-08 14:50 ` Jan Beulich
2018-06-07 14:59 ` [PATCH v7 06/15] x86/hvm: Introduce hvm_save_mtrr_msr_one func Alexandru Isaila
2018-06-08 14:57 ` Jan Beulich
2018-06-07 14:59 ` [PATCH v7 07/15] x86/hvm: Introduce viridian_save_vcpu_ctxt_one() func Alexandru Isaila
2018-06-08 8:27 ` Paul Durrant
2018-06-07 14:59 ` [PATCH v7 08/15] x86/cpu: Remove loop form vmce_save_vcpu_ctxt() func Alexandru Isaila
2018-06-08 8:33 ` Paul Durrant
2018-06-08 8:51 ` Alexandru Stefan ISAILA
2018-06-08 12:46 ` Paul Durrant
2018-06-08 14:42 ` Jan Beulich
2018-06-20 8:56 ` Alexandru Stefan ISAILA
2018-06-21 7:39 ` Jan Beulich
2018-06-21 7:47 ` Alexandru Stefan ISAILA [this message]
2018-06-21 8:15 ` Jan Beulich
2018-06-07 14:59 ` [PATCH v7 09/15] x86/hvm: Remove loop from hvm_save_tsc_adjust() func Alexandru Isaila
2018-06-07 14:59 ` [PATCH v7 10/15] x86/hvm: Remove loop from hvm_save_cpu_ctxt func Alexandru Isaila
2018-06-07 14:59 ` [PATCH v7 11/15] x86/hvm: Remove loop from hvm_save_cpu_xsave_states Alexandru Isaila
2018-06-07 14:59 ` [PATCH v7 12/15] x86/hvm: Remove loop from hvm_save_cpu_msrs func Alexandru Isaila
2018-06-07 14:59 ` [PATCH v7 13/15] x86/hvm: Remove loop from hvm_save_mtrr_msr func Alexandru Isaila
2018-06-07 14:59 ` [PATCH v7 14/15] x86/hvm: Remove loop from viridian_save_vcpu_ctxt() func Alexandru Isaila
2018-06-07 14:59 ` [PATCH v7 15/15] x86/domctl: Don't pause the whole domain if only getting vcpu state Alexandru Isaila
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=1529567253.4608.19.camel@bitdefender.com \
--to=aisaila@bitdefender.com \
--cc=Ian.Jackson@citrix.com \
--cc=JBeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=paul.durrant@citrix.com \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xen.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.