Re: [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-14 Thread Konrad Rzeszutek Wilk
On Mon, Jan 14, 2013 at 10:29:41AM +0100, Matt Wilson wrote: > On Fri, Jan 11, 2013 at 04:35:50PM -0500, Konrad Rzeszutek Wilk wrote: > > > > With this patch I get: > > > > [2.555087] GHES: HEST is not enabled! > > [2.560522] ioatdma: Intel(R) QuickData Technology Driver 4.00 > > [2.5

Re: [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-14 Thread Matt Wilson
On Fri, Jan 11, 2013 at 04:35:50PM -0500, Konrad Rzeszutek Wilk wrote: > > With this patch I get: > > [2.555087] GHES: HEST is not enabled! > [2.560522] ioatdma: Intel(R) QuickData Technology Driver 4.00 > [2.569651] [ cut here ] > [2.570529] kernel BUG at

Re: [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-11 Thread Konrad Rzeszutek Wilk
On Sun, Jan 06, 2013 at 11:14:42AM +, Matt Wilson wrote: > Commit 85ff6acb075a484780b3d763fdf41596d8fc0970 (xen/granttable: Grant > tables V2 implementation) changed the GREFS_PER_GRANT_FRAME macro from > a constant to a conditional expression. The expression depends on > grant_table_version be

Re: [Xen-devel] [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-10 Thread ANNIE LI
On 2013-1-10 17:16, Matt Wilson wrote: On Wed, Jan 09, 2013 at 12:02:09PM +, Ian Campbell wrote: On Wed, 2013-01-09 at 02:40 +, ANNIE LI wrote: @@ -1080,18 +1081,18 @@ static void gnttab_request_version(void) panic("we need grant tables version 2, but only version 1 is

Re: [Xen-devel] [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-10 Thread Ian Campbell
On Thu, 2013-01-10 at 09:16 +, Matt Wilson wrote: > On Wed, Jan 09, 2013 at 12:02:09PM +, Ian Campbell wrote: > > On Wed, 2013-01-09 at 02:40 +, ANNIE LI wrote: > > > > @@ -1080,18 +1081,18 @@ static void gnttab_request_version(void) > > > > panic("we need grant tables v

Re: [Xen-devel] [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-10 Thread Matt Wilson
On Wed, Jan 09, 2013 at 12:02:09PM +, Ian Campbell wrote: > On Wed, 2013-01-09 at 02:40 +, ANNIE LI wrote: > > > @@ -1080,18 +1081,18 @@ static void gnttab_request_version(void) > > > panic("we need grant tables version 2, but only version > > > 1 is available"); > > >

Re: [Xen-devel] [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-09 Thread annie li
On 2013-1-9 20:02, Ian Campbell wrote: On Wed, 2013-01-09 at 02:40 +, ANNIE LI wrote: @@ -1080,18 +1081,18 @@ static void gnttab_request_version(void) panic("we need grant tables version 2, but only version 1 is available"); } else { grant_table_

Re: [Xen-devel] [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-09 Thread Ian Campbell
On Wed, 2013-01-09 at 02:40 +, ANNIE LI wrote: > > @@ -1080,18 +1081,18 @@ static void gnttab_request_version(void) > > panic("we need grant tables version 2, but only version 1 is > > available"); > > } else { > > grant_table_version = 1; > > + grefs_per_

Re: [Xen-devel] [PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-08 Thread ANNIE LI
Thanks so much for posting this. On 2013-1-6 19:14, Matt Wilson wrote: Commit 85ff6acb075a484780b3d763fdf41596d8fc0970 (xen/granttable: Grant tables V2 implementation) changed the GREFS_PER_GRANT_FRAME macro from a constant to a conditional expression. The expression depends on grant_table_versi

[PATCH v2] xen/grant-table: correctly initialize grant table version 1

2013-01-06 Thread Matt Wilson
Commit 85ff6acb075a484780b3d763fdf41596d8fc0970 (xen/granttable: Grant tables V2 implementation) changed the GREFS_PER_GRANT_FRAME macro from a constant to a conditional expression. The expression depends on grant_table_version being appropriately set. Unfortunately, at init time grant_table_versio