From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hollis Blanchard Subject: Re: Re: [Xen-devel] [PATCH] [GNTTAB] expandable grant table Date: Mon, 26 Feb 2007 17:31:08 -0600 Message-ID: <1172532668.910.34.camel@basalt> References: <20070106064820.GA25217%yamahata@valinux.co.jp> <20070219080312.GB10839%yamahata@valinux.co.jp> Reply-To: Hollis Blanchard Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20070219080312.GB10839%yamahata@valinux.co.jp> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-ppc-devel-bounces@lists.xensource.com Errors-To: xen-ppc-devel-bounces@lists.xensource.com To: Isaku Yamahata Cc: xen-ia64-devel@lists.xensource.com, xen-devel , Keir Fraser , xen-ppc-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org On Mon, 2007-02-19 at 17:03 +0900, Isaku Yamahata wrote: > Attached the updated patches. Please find them > > - 13985_4fa9b86d41b8_grow_granttable_acm.patch > acm part. I did only compile test. > > - 13986_cca098fd122c_grow_granttable_ia64.patch > ia64 part. > > - 13987_f5f232aef7bd_grow_granttable_powerpc.patch > powerpc part. This patch prohibits growing grant table. > I didn't even compile test. > It needs review by xen/powerpc developers and they may > have their own opinions. Hi Yamahata-san, thanks very much for your patch! I'm confused about one thing though: why do we need to take a lock to read d->grant_table->nr_grant_frames? It's a simple integer, so no lock is required or useful. -- Hollis Blanchard IBM Linux Technology Center