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

Re: [Xen-devel] [PATCH v14 1/2] vmx: VT-d posted-interrupt core logic handling



>>> On 01.03.16 at 06:39, <feng.wu@xxxxxxxxx> wrote:

> 
>> -----Original Message-----
>> From: Dario Faggioli [mailto:dario.faggioli@xxxxxxxxxx]
>> Sent: Monday, February 29, 2016 9:52 PM
>> To: Jan Beulich <JBeulich@xxxxxxxx>; George Dunlap
>> <george.dunlap@xxxxxxxxxxxxx>; Wu, Feng <feng.wu@xxxxxxxxx>
>> Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>; Tian, Kevin
>> <kevin.tian@xxxxxxxxx>; xen-devel@xxxxxxxxxxxxx; Keir Fraser <keir@xxxxxxx>
>> Subject: Re: [Xen-devel] [PATCH v14 1/2] vmx: VT-d posted-interrupt core 
> logic
>> handling
>> 
>> On Mon, 2016-02-29 at 06:33 -0700, Jan Beulich wrote:
>> > > > > On 29.02.16 at 04:00, <feng.wu@xxxxxxxxx> wrote:
>> > > This is the core logic handling for VT-d posted-interrupts.
>> > > Basically it
>> > > deals with how and when to update posted-interrupts during the
>> > > following
>> > > scenarios:
>> > > - vCPU is preempted
>> > > - vCPU is slept
>> > > - vCPU is blocked
>> > >
>> > [...]
>> > > CC: Keir Fraser <keir@xxxxxxx>
>> > > CC: Jan Beulich <jbeulich@xxxxxxxx>
>> > > CC: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>> > > CC: Kevin Tian <kevin.tian@xxxxxxxxx>
>> > > CC: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
>> > > CC: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
>> > > Suggested-by: Yang Zhang <yang.z.zhang@xxxxxxxxx>
>> > > Suggested-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
>> > > Suggested-by: George Dunlap <george.dunlap@xxxxxxxxxx>
>> > > Suggested-by: Jan Beulich <jbeulich@xxxxxxxx>
>> > > Signed-off-by: Feng Wu <feng.wu@xxxxxxxxx>
>> > > Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>
>> > With the comments George gave on v13 subsequent to this tag
>> > I'm not sure it was correct to retain it. George?
>> >
>> > Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
>> > albeit in case another version is needed ...
>> >
>> And, if I'm still in time (if not, no big deal :-))
>> 
>> Reviewed-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> 
> Really appreciate your huge efforts on reviewing this series, Jan,
> George, and Dario! 
> 
> BTW, Jan, considering that Kevin, as the VMX maintainer also gave
> his acked-by for this patch, do you think another version is needed?

Not unless further review comments make one necessary; namely
pending George's confirmation of the R-b that you've kept in place.

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®.