On March 08, 2016 8:13pm, <dario.faggi...@citrix.com> 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...@intel.com>
> > CC: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com>
> > CC: Aravind Gopalakrishnan <aravind.gopalakrish...@amd.com>
> >
> 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.faggi...@citrix.com>
> > CC: Jan Beulich <jbeul...@suse.com>
> >
> (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.faggi...@citrix.com>
> 
Dario, thanks.

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

Reply via email to