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

Re: [Xen-devel] [PATCH v10 0/7] Add VT-d Posted-Interrupts support




> -----Original Message-----
> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
> Sent: Thursday, December 3, 2015 7:19 PM
> To: Wu, Feng <feng.wu@xxxxxxxxx>
> Cc: xen-devel@xxxxxxxxxxxxx
> Subject: Re: [Xen-devel] [PATCH v10 0/7] Add VT-d Posted-Interrupts support
> 
> >>> On 03.12.15 at 09:35, <feng.wu@xxxxxxxxx> wrote:
> > VT-d Posted-Interrupts is an enhancement to CPU side Posted-Interrupt.
> > With VT-d Posted-Interrupts enabled, external interrupts from
> > direct-assigned devices can be delivered to guests without VMM
> > intervention when guest is running in non-root mode.
> >
> > You can find the VT-d Posted-Interrtups Spec. in the following URL:
> > http://www.intel.com/content/www/us/en/intelligent-systems/intel-
> technology/vt-
> > directed-io-spec.html
> >
> > Feng Wu (17):
> >  r   VT-d Posted-intterrupt (PI) design
> >  ra  vmx: Suppress posting interrupts when 'SN' is set
> >  r   vt-d: Add API to update IRTE when VT-d PI is used
> >   a  Update IRTE according to guest interrupt config changes
> >   a  vmx: Properly handle notification event when vCPU is running
> >      vmx: VT-d posted-interrupt core logic handling
> >  ra  Add a command line parameter for VT-d posted-interrupts
> >
> >  r = has been 'Reviewed-by'
> >  a = has been 'Acked-by'
> 
> Actually this kind of summary is misleading (and I understand
> you're just following what others did before, i.e. this is not a
> problem just here): For it to be meaningful, one would need to
> know whether reviews/acks are in place by all necessary
> maintainers. But of course the summary helps in drawing
> attention to things completely un-reviewed/un-acked.

Yes, I agree with you. For this case, only patch
"vmx: VT-d posted-interrupt core logic handling" doesn't have
reviewed-by or acked-by, so it may need more attention by
the maintainers.

> 
> Perhaps one could use upper case letters to indicate full
> maintainer review/ack state?

Sounds great!

Thanks,
Feng

> 
> Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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