Re: [XenPPC] Re: [Xen-devel] [PATCH] [GNTTAB] expandable grant table - Hollis Blanchard - com.xensource.lists.xen-ia64-devel

5 messages in com.xensource.lists.xen-ia64-develRe: [XenPPC] Re: [Xen-devel] [PATCH] ...

From	Sent On	Attachments
Isaku Yamahata	19 Feb 2007 00:02	.patch, .patch, .patch
Hollis Blanchard	26 Feb 2007 15:30
Keir Fraser	26 Feb 2007 15:38
Hollis Blanchard	26 Feb 2007 16:04
Keir Fraser	26 Feb 2007 23:50

Subject:	Re: [XenPPC] Re: [Xen-devel] [PATCH] [GNTTAB] expandable grant table
From:	Hollis Blanchard (holl...@us.ibm.com)
Date:	02/26/2007 03:30:45 PM
List:	com.xensource.lists.xen-ia64-devel

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.