Re: [Xen-devel] [PATCH 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization.

["Xu, Quan" <quan.xu@xxxxxxxxx>]

On March 08, 2016 8:13pm, <dario.faggioli@xxxxxxxxxx> wrote:
> On Tue, 2016-03-08 at 19:09 +0800, Quan Xu wrote:
> > Doing what we do serves as a fix for a bug found in AMD IOMMU
> > initialization.
> >
> This first line is rather useless, if not worse. :-)
> 
I will remove it in next v2. :)

> I don't know (provided a new version is not necessary, and provided 
> maintainers
> agree with me :-)) whether you need to repost or it can be removed when code
> is committed.
> 

I think I'd better send out v2. :)

> > Signed-off-by: Quan Xu <quan.xu@xxxxxxxxx>
> > CC: Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>
> > CC: Aravind Gopalakrishnan <Aravind.Gopalakrishnan@xxxxxxx>
> >
> get_maintainer.pl gives me only Suravee, as Aravind stepped down a few days
> ago, so he shouldn't be bothered (and, in fact, I'm moving him from Cc to 
> Bcc).
> 

Got it, thanks for your advice.

> > CC: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> > CC: Jan Beulich <jbeulich@xxxxxxxx>
> >
> (BTW, it's of course fine to include me and Jan, despite what 
> get_maintainer.pl's
> output, as we've been involved in previous rounds of review.)
> 
> All that being said:
> 
> Reviewed-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> 
Dario, thanks.

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