Re: [Xen-devel] [PATCH V3 6/6]: PVH:privcmd changes.

2012-10-18 Thread Mukesh Rathor
On Thu, 18 Oct 2012 11:35:53 +0100 Ian Campbell wrote: > > > @@ -439,6 +490,19 @@ static long privcmd_ioctl(struct file *file, > > return ret; > > } > > > > +static void privcmd_close(struct vm_area_struct *vma) > > +{ > > + struct page **pages = vma ? vma->vm_private_data : NULL; > >

Re: [PATCH V3 6/6]: PVH:privcmd changes.

2012-10-18 Thread Stefano Stabellini
On Thu, 18 Oct 2012, Mukesh Rathor wrote: > PVH: privcmd changes. PVH only supports the batch interface. To map a foreign > page to a process, pfn must be allocated. PVH path uses ballooning for that > purpose. The returned pfn is then mapped to the foreign page. > xen_unmap_domain_mfn_range()

Re: [Xen-devel] [PATCH V3 6/6]: PVH:privcmd changes.

2012-10-18 Thread Ian Campbell
> @@ -439,6 +490,19 @@ static long privcmd_ioctl(struct file *file, > return ret; > } > > +static void privcmd_close(struct vm_area_struct *vma) > +{ > + struct page **pages = vma ? vma->vm_private_data : NULL; Can VMA really be NULL?... > + int numpgs = (vma->vm_end -

Re: [Xen-devel] [PATCH V3 6/6]: PVH:privcmd changes.

2012-10-18 Thread Ian Campbell
@@ -439,6 +490,19 @@ static long privcmd_ioctl(struct file *file, return ret; } +static void privcmd_close(struct vm_area_struct *vma) +{ + struct page **pages = vma ? vma-vm_private_data : NULL; Can VMA really be NULL?... + int numpgs = (vma-vm_end - vma-vm_start)

Re: [PATCH V3 6/6]: PVH:privcmd changes.

2012-10-18 Thread Stefano Stabellini
On Thu, 18 Oct 2012, Mukesh Rathor wrote: PVH: privcmd changes. PVH only supports the batch interface. To map a foreign page to a process, pfn must be allocated. PVH path uses ballooning for that purpose. The returned pfn is then mapped to the foreign page. xen_unmap_domain_mfn_range() is

Re: [Xen-devel] [PATCH V3 6/6]: PVH:privcmd changes.

2012-10-18 Thread Mukesh Rathor
On Thu, 18 Oct 2012 11:35:53 +0100 Ian Campbell ian.campb...@citrix.com wrote: @@ -439,6 +490,19 @@ static long privcmd_ioctl(struct file *file, return ret; } +static void privcmd_close(struct vm_area_struct *vma) +{ + struct page **pages = vma ? vma-vm_private_data :

[PATCH V3 6/6]: PVH:privcmd changes.

2012-10-17 Thread Mukesh Rathor
PVH: privcmd changes. PVH only supports the batch interface. To map a foreign page to a process, pfn must be allocated. PVH path uses ballooning for that purpose. The returned pfn is then mapped to the foreign page. xen_unmap_domain_mfn_range() is introduced to unmap these pages via the privcmd

[PATCH V3 6/6]: PVH:privcmd changes.

2012-10-17 Thread Mukesh Rathor
PVH: privcmd changes. PVH only supports the batch interface. To map a foreign page to a process, pfn must be allocated. PVH path uses ballooning for that purpose. The returned pfn is then mapped to the foreign page. xen_unmap_domain_mfn_range() is introduced to unmap these pages via the privcmd