From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH 3/4] xen/granttable: Grant tables V2 implementation Date: Tue, 22 Nov 2011 09:31:23 -0500 Message-ID: <20111122143123.GA3430@phenom.dumpdata.com> References: <4ECA416D.90602@oracle.com> <1321927161-3987-1-git-send-email-annie.li@oracle.com> <1321954186.3664.403.camel@zakaz.uk.xensource.com> <4ECB7324.9080008@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <4ECB7324.9080008@oracle.com> Sender: linux-kernel-owner@vger.kernel.org To: ANNIE LI Cc: Ian Campbell , "xen-devel@lists.xensource.com" , "linux-kernel@vger.kernel.org" , "jeremy@goop.org" , "kurt.hackel@oracle.com" , Paul Durrant List-Id: xen-devel@lists.xenproject.org On Tue, Nov 22, 2011 at 06:02:12PM +0800, ANNIE LI wrote: > > > On 2011-11-22 17:29, Ian Campbell wrote: > >On Tue, 2011-11-22 at 01:59 +0000, annie.li@oracle.com wrote: > >[...] > >>+#ifdef CONFIG_X86 > >>+ barrier(); > >>+#else > >>+ mb(); > >>+#endif > >>+ } > >Niggle: Indentation of this final } is off. > Sorry, I did not notice that. > >Otherwise all 4 patches: > >Acked-by: Ian Campbell > > > >[...] > Thanks, I will resend those patches after above indentation issue is fixed. That is OK. I can modify them. > > Thanks. > > Thanks > Annie