Re: [PATCH] xen-gnttab: do not add m2p override entries for blkback mappings

2013-11-13 Thread Stefano Stabellini
On Wed, 13 Nov 2013, David Vrabel wrote: > On 13/11/13 01:48, Anthony Liguori wrote: > > From: Anthony Liguori > > > > Commit 5dc03639 switched blkback to also add m2p override entries > > when mapping grant pages but history seems to have forgotten why > > this is useful if it ever was. > > > >

Re: [Xen-devel] [PATCH] xen-gnttab: do not add m2p override entries for blkback mappings

2013-11-13 Thread Ian Campbell
On Tue, 2013-11-12 at 17:48 -0800, Anthony Liguori wrote: > See http://article.gmane.org/gmane.linux.kernel/1590932 for a full > analysis of current users. Silly nit: Please can you include the message ID in case this goes away, or use a URL which includes the ID already so it can be translated if

Re: [PATCH] xen-gnttab: do not add m2p override entries for blkback mappings

2013-11-13 Thread David Vrabel
On 13/11/13 01:48, Anthony Liguori wrote: > From: Anthony Liguori > > Commit 5dc03639 switched blkback to also add m2p override entries > when mapping grant pages but history seems to have forgotten why > this is useful if it ever was. > > The blkback driver does not need m2p override entries to

Re: [PATCH] xen-gnttab: do not add m2p override entries for blkback mappings

2013-11-12 Thread Matt Wilson
On Tue, Nov 12, 2013 at 05:48:56PM -0800, Anthony Liguori wrote: > From: Anthony Liguori > > Commit 5dc03639 switched blkback to also add m2p override entries > when mapping grant pages but history seems to have forgotten why > this is useful if it ever was. > > The blkback driver does not need

[PATCH] xen-gnttab: do not add m2p override entries for blkback mappings

2013-11-12 Thread Anthony Liguori
From: Anthony Liguori Commit 5dc03639 switched blkback to also add m2p override entries when mapping grant pages but history seems to have forgotten why this is useful if it ever was. The blkback driver does not need m2p override entries to exist and there is significant overhead due to the lock