From: Dario Faggioli <dario.faggioli@citrix.com>
To: "Wu, Feng" <feng.wu@intel.com>, Jan Beulich <JBeulich@suse.com>
Cc: GeorgeDunlap <george.dunlap@eu.citrix.com>,
Andrew Cooper <andrew.cooper3@citrix.com>,
"Tian, Kevin" <kevin.tian@intel.com>, Keir Fraser <keir@xen.org>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [PATCH v8 15/17] vmx: VT-d posted-interrupt core logic handling
Date: Thu, 29 Oct 2015 10:26:31 +0100 [thread overview]
Message-ID: <1446110791.28782.24.camel@citrix.com> (raw)
In-Reply-To: <E959C4978C3B6342920538CF579893F00AE13999@SHSMSX104.ccr.corp.intel.com>
[-- Attachment #1.1: Type: text/plain, Size: 1107 bytes --]
On Thu, 2015-10-29 at 05:39 +0000, Wu, Feng wrote:
>
> > -----Original Message-----
> > From: Dario Faggioli [mailto:dario.faggioli@citrix.com]
> > Of course, that does not mean that we shouldn't add them for the
> > sake
> > of this patch, if we can't avoid doing that. But if we can avoid
> > them,
> > that is perhaps one more reason for doing things that way.
>
> Jan & Dario, thanks a lot for you guys' input.
>
Thanks to you for listening and replying. :-)
> If you agree, I will remove
> the arch_vcpu_block_cancel() and maybe arch_vcpu_wake_prepare()
> as well, and implement the logic before VM-entry in V9, the patch
> will
> be coming soon.
>
Ok. Just to be sure I understand, you're plan is removing
arch_vcpu_block_cancel(), arch_vcpu_wake_prepare() and *leaving*
arch_vcpu_block()?
Regards,
Dario
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
[-- Attachment #2: Type: text/plain, Size: 126 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2015-10-29 9:26 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-12 8:54 [PATCH v8 00/17] Add VT-d Posted-Interrupts support Feng Wu
2015-10-12 8:54 ` [PATCH v8 01/17] VT-d Posted-intterrupt (PI) design Feng Wu
2015-10-12 10:41 ` Andrew Cooper
2015-10-13 0:53 ` Wu, Feng
2015-10-29 15:56 ` Dario Faggioli
2015-10-12 8:54 ` [PATCH v8 02/17] Add cmpxchg16b support for x86-64 Feng Wu
2015-10-13 15:29 ` Jan Beulich
2015-10-14 5:57 ` Wu, Feng
2015-10-14 9:05 ` Jan Beulich
2015-10-14 9:29 ` Wu, Feng
2015-10-14 9:45 ` Jan Beulich
2015-10-14 10:03 ` Wu, Feng
2015-10-14 10:21 ` Jan Beulich
2015-10-14 10:25 ` Wu, Feng
2015-10-12 8:54 ` [PATCH v8 03/17] iommu: Add iommu_intpost to control VT-d Posted-Interrupts feature Feng Wu
2015-10-12 8:54 ` [PATCH v8 04/17] vt-d: VT-d Posted-Interrupts feature detection Feng Wu
2015-10-12 8:54 ` [PATCH v8 05/17] vmx: Extend struct pi_desc to support VT-d Posted-Interrupts Feng Wu
2015-10-12 8:54 ` [PATCH v8 06/17] vmx: Add some helper functions for Posted-Interrupts Feng Wu
2015-10-12 8:54 ` [PATCH v8 07/17] vmx: Initialize VT-d Posted-Interrupts Descriptor Feng Wu
2015-10-12 8:54 ` [PATCH v8 08/17] vmx: Suppress posting interrupts when 'SN' is set Feng Wu
2015-10-12 8:54 ` [PATCH v8 09/17] VT-d: Remove pointless casts Feng Wu
2015-10-12 8:54 ` [PATCH v8 10/17] vt-d: Extend struct iremap_entry to support VT-d Posted-Interrupts Feng Wu
2015-10-12 8:54 ` [PATCH v8 11/17] vt-d: Add API to update IRTE when VT-d PI is used Feng Wu
2015-10-12 8:54 ` [PATCH v8 12/17] x86: move some APIC related macros to apicdef.h Feng Wu
2015-10-12 8:54 ` [PATCH v8 13/17] Update IRTE according to guest interrupt config changes Feng Wu
2015-10-12 8:55 ` [PATCH v8 14/17] vmx: Properly handle notification event when vCPU is running Feng Wu
2015-10-12 8:55 ` [PATCH v8 15/17] vmx: VT-d posted-interrupt core logic handling Feng Wu
2015-10-26 14:39 ` Dario Faggioli
2015-10-27 5:19 ` Wu, Feng
2015-10-27 9:51 ` Jan Beulich
2015-10-28 1:50 ` Dario Faggioli
2015-10-28 2:58 ` Wu, Feng
2015-10-28 9:03 ` Jan Beulich
2015-10-28 16:36 ` Dario Faggioli
2015-10-29 5:39 ` Wu, Feng
2015-10-29 9:26 ` Dario Faggioli [this message]
2015-10-29 14:07 ` Wu, Feng
2015-10-27 12:16 ` Dario Faggioli
2015-10-28 2:40 ` Wu, Feng
2015-10-27 12:22 ` Dario Faggioli
2015-10-12 8:55 ` [PATCH v8 16/17] VT-d: Dump the posted format IRTE Feng Wu
2015-10-12 8:55 ` [PATCH v8 17/17] Add a command line parameter for VT-d posted-interrupts Feng Wu
2015-10-23 2:12 ` [PATCH v8 00/17] Add VT-d Posted-Interrupts support Wu, Feng
2015-10-23 8:13 ` Jan Beulich
2015-10-23 8:35 ` Wu, Feng
2015-10-23 8:46 ` Dario Faggioli
2015-10-23 8:52 ` Wu, Feng
2015-10-29 10:51 ` Dario Faggioli
2015-10-29 14:05 ` Wu, Feng
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=1446110791.28782.24.camel@citrix.com \
--to=dario.faggioli@citrix.com \
--cc=JBeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=feng.wu@intel.com \
--cc=george.dunlap@eu.citrix.com \
--cc=keir@xen.org \
--cc=kevin.tian@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).