Re: [PATCH] virtio: 9p: correctly pass physical address to userspace for high pages

2012-10-18 Thread Rusty Russell
Will Deacon writes: > On Thu, Oct 18, 2012 at 03:19:06AM +0100, Rusty Russell wrote: >> We do have an alternate solution: masking out __GFP_HIGHMEM from the >> kmalloc of desc. If it fails, we will fall back to laying out the >> virtio request directly inside the ring; if it doesn't fit, we'll wa

Re: [PATCH] virtio: 9p: correctly pass physical address to userspace for high pages

2012-10-18 Thread Will Deacon
Hi Rusty, On Thu, Oct 18, 2012 at 03:19:06AM +0100, Rusty Russell wrote: > Will Deacon writes: > > When using a virtio transport, the 9p net device allocates pages to back > > the descriptors inserted into the virtqueue. These allocations may be > > performed from atomic context (under the channe

Re: [PATCH] virtio: 9p: correctly pass physical address to userspace for high pages

2012-10-17 Thread Rusty Russell
Will Deacon writes: > When using a virtio transport, the 9p net device allocates pages to back > the descriptors inserted into the virtqueue. These allocations may be > performed from atomic context (under the channel lock) and can therefore > return high mappings which aren't suitable for virt_to

Re: [PATCH] virtio: 9p: correctly pass physical address to userspace for high pages

2012-10-17 Thread Will Deacon
On Wed, Oct 17, 2012 at 05:05:00PM +0100, Sasha Levin wrote: > On Wed, Oct 17, 2012 at 6:14 AM, Will Deacon wrote: > > When using a virtio transport, the 9p net device allocates pages to back > > the descriptors inserted into the virtqueue. These allocations may be > > performed from atomic contex

Re: [PATCH] virtio: 9p: correctly pass physical address to userspace for high pages

2012-10-17 Thread Sasha Levin
On Wed, Oct 17, 2012 at 6:14 AM, Will Deacon wrote: > When using a virtio transport, the 9p net device allocates pages to back > the descriptors inserted into the virtqueue. These allocations may be > performed from atomic context (under the channel lock) and can therefore > return high mappings w

[PATCH] virtio: 9p: correctly pass physical address to userspace for high pages

2012-10-17 Thread Will Deacon
When using a virtio transport, the 9p net device allocates pages to back the descriptors inserted into the virtqueue. These allocations may be performed from atomic context (under the channel lock) and can therefore return high mappings which aren't suitable for virt_to_phys. This patch ensures th