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

["Wu, Feng" <feng.wu@xxxxxxxxx>]

> -----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?

Thanks,
Feng

> 
> Also, personally, I don't think that all those Suggested-by add much
> useful info, and I'd be fine to see mine one removed (in any case, but
> even more so, if the Rev-by: above is applied).
> 
> But, again, no big deal.
> 
> Thanks and 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)

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