From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wei Liu Subject: Re: [v7][PATCH 00/16] Fix RMRR Date: Thu, 16 Jul 2015 09:46:57 +0100 Message-ID: <20150716084657.GJ12455@zion.uk.xensource.com> References: <1436420047-25356-1-git-send-email-tiejun.chen@intel.com> <55A77F950200007800091AEC@mail.emea.novell.com> <55A76560.4090302@intel.com> <55A7828E0200007800091B2A@mail.emea.novell.com> <1437035454.32371.131.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1437035454.32371.131.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Wei Liu , George Dunlap , Ian Jackson , "xen-devel@lists.xen.org" , Jan Beulich , Tiejun Chen List-Id: xen-devel@lists.xenproject.org On Thu, Jul 16, 2015 at 09:30:54AM +0100, Ian Campbell wrote: > On Thu, 2015-07-16 at 09:08 +0100, Jan Beulich wrote: > > >>> On 16.07.15 at 10:03, wrote: > > > On 2015/7/16 15:55, Jan Beulich wrote: > > >>>>> On 10.07.15 at 16:50, wrote: > > >>> On Thu, Jul 9, 2015 at 6:33 AM, Tiejun Chen wrote: > > >>>> v7: > > >>> > > >>> It looks like most of the libxl/libxc patches have been acked. It > > >>> seems to me that most of the hypervisor patches (1-3, 14-15) are > > >>> either ready to go in or pretty close. > > >> > > >> Now that I looked over v8 I have to admit that if I was a tools > > >> maintainer I wouldn't want to see some of the tools patches in > > >> with just an ack, but without any review. > > > > > > I'm somewhat confused at this point. > > > > > > Acked-by: is often used by the maintainer of the affected code when that > > > maintainer neither contributed to nor forwarded the patch. It is a > > > record that the acker has at least reviewed the patch and has indicated > > > acceptance. > > > > > > Does this imply this is already reviewed? > > > > No, that would be expressed by Reviewed-by. Acked-by merely > > means no objection by the maintainer for the change to go in. > > For my part I, perhaps wrongly, use Acked-by for both. If I haven't > actually carefully reviewed the change I will usually say so, e.g. "I > see XXX has reviewed this already, so that's fine by me" or something > similar (which I admit gets lost once it becomes just the tags). > > I can't speak for Ian or Wei (now CCd) but Ian at least I think operates > similarly. > I do the same. I will explicitly say that if I give my ack without looking at the code. I use Reviewed-by when the component is not maintained by me. Wei. > Ian.