Re: [PATCH 1/2] xen/grant-table: Use put_page instead of free_page

2018-01-16 Thread Boris Ostrovsky
On 01/11/2018 04:36 AM, Ross Lagerwall wrote: > The page given to gnttab_end_foreign_access() to free could be a > compound page so use put_page() instead of free_page() since it can > handle both compound and single pages correctly. > > This bug was discovered when migrating a Xen VM with several

Re: [PATCH 1/2] xen/grant-table: Use put_page instead of free_page

2018-01-11 Thread Ross Lagerwall
+CC netdev On 01/11/2018 09:36 AM, Ross Lagerwall wrote: The page given to gnttab_end_foreign_access() to free could be a compound page so use put_page() instead of free_page() since it can handle both compound and single pages correctly. This bug was discovered when migrating a Xen VM with sev

[PATCH 1/2] xen/grant-table: Use put_page instead of free_page

2018-01-11 Thread Ross Lagerwall
The page given to gnttab_end_foreign_access() to free could be a compound page so use put_page() instead of free_page() since it can handle both compound and single pages correctly. This bug was discovered when migrating a Xen VM with several VIFs and CONFIG_DEBUG_VM enabled. It hits a BUG usually