Re: [PATCH v2 2/3] xen/privcmd: Mark pages as dirty

2020-07-07 Thread Jürgen Groß
On 07.07.20 21:30, John Hubbard wrote: On 2020-07-07 04:43, Jürgen Groß wrote: On 07.07.20 13:30, Souptick Joarder wrote: On Tue, Jul 7, 2020 at 3:08 PM Jürgen Groß wrote: ... diff --git a/drivers/xen/privcmd.c b/drivers/xen/privcmd.c index 33677ea..f6c1543 100644 --- a/drivers/xen/privcmd.c

Re: [PATCH v2 2/3] xen/privcmd: Mark pages as dirty

2020-07-07 Thread John Hubbard
On 2020-07-07 04:43, Jürgen Groß wrote: On 07.07.20 13:30, Souptick Joarder wrote: On Tue, Jul 7, 2020 at 3:08 PM Jürgen Groß wrote: ... diff --git a/drivers/xen/privcmd.c b/drivers/xen/privcmd.c index 33677ea..f6c1543 100644 --- a/drivers/xen/privcmd.c +++ b/drivers/xen/privcmd.c @@ -612,8 +

Re: [PATCH v2 2/3] xen/privcmd: Mark pages as dirty

2020-07-07 Thread Jürgen Groß
On 07.07.20 13:30, Souptick Joarder wrote: On Tue, Jul 7, 2020 at 3:08 PM Jürgen Groß wrote: On 06.07.20 20:16, Souptick Joarder wrote: pages need to be marked as dirty before unpinned it in unlock_pages() which was oversight. This is fixed now. Signed-off-by: Souptick Joarder Suggested-by:

Re: [PATCH v2 2/3] xen/privcmd: Mark pages as dirty

2020-07-07 Thread Souptick Joarder
On Tue, Jul 7, 2020 at 3:08 PM Jürgen Groß wrote: > > On 06.07.20 20:16, Souptick Joarder wrote: > > pages need to be marked as dirty before unpinned it in > > unlock_pages() which was oversight. This is fixed now. > > > > Signed-off-by: Souptick Joarder > > Suggested-by: John Hubbard > > Cc: Jo

Re: [PATCH v2 2/3] xen/privcmd: Mark pages as dirty

2020-07-07 Thread Jürgen Groß
On 06.07.20 20:16, Souptick Joarder wrote: pages need to be marked as dirty before unpinned it in unlock_pages() which was oversight. This is fixed now. Signed-off-by: Souptick Joarder Suggested-by: John Hubbard Cc: John Hubbard Cc: Boris Ostrovsky Cc: Paul Durrant --- drivers/xen/privcmd

[PATCH v2 2/3] xen/privcmd: Mark pages as dirty

2020-07-06 Thread Souptick Joarder
pages need to be marked as dirty before unpinned it in unlock_pages() which was oversight. This is fixed now. Signed-off-by: Souptick Joarder Suggested-by: John Hubbard Cc: John Hubbard Cc: Boris Ostrovsky Cc: Paul Durrant --- drivers/xen/privcmd.c | 5 - 1 file changed, 4 insertions(+),