[Xen-devel] [PATCH v2 0/2] xen/gntdev: sanitize user interface handling

Juergen Gross posted 2 patches 35 weeks ago
Failed in applying to current master (apply log)
drivers/xen/gntdev-common.h |  2 +-
drivers/xen/gntdev-dmabuf.c | 11 +++------
drivers/xen/gntdev.c        | 55 +++++++++++++++++++--------------------------
3 files changed, 27 insertions(+), 41 deletions(-)

[Xen-devel] [PATCH v2 0/2] xen/gntdev: sanitize user interface handling

Posted by Juergen Gross 35 weeks ago
The Xen gntdev driver's checking of the number of allowed mapped pages
is in need of some sanitizing work.

Changes in V2:
- enhanced commit message of patch 1 (Andrew Cooper)

Juergen Gross (2):
  xen/gntdev: replace global limit of mapped pages by limit per call
  xen/gntdev: switch from kcalloc() to kvcalloc()

 drivers/xen/gntdev-common.h |  2 +-
 drivers/xen/gntdev-dmabuf.c | 11 +++------
 drivers/xen/gntdev.c        | 55 +++++++++++++++++++--------------------------
 3 files changed, 27 insertions(+), 41 deletions(-)

-- 
2.16.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH v2 0/2] xen/gntdev: sanitize user interface handling

Posted by Jürgen Groß 32 weeks ago
On 07.11.19 12:15, Juergen Gross wrote:
> The Xen gntdev driver's checking of the number of allowed mapped pages
> is in need of some sanitizing work.
> 
> Changes in V2:
> - enhanced commit message of patch 1 (Andrew Cooper)
> 
> Juergen Gross (2):
>    xen/gntdev: replace global limit of mapped pages by limit per call
>    xen/gntdev: switch from kcalloc() to kvcalloc()
> 
>   drivers/xen/gntdev-common.h |  2 +-
>   drivers/xen/gntdev-dmabuf.c | 11 +++------
>   drivers/xen/gntdev.c        | 55 +++++++++++++++++++--------------------------
>   3 files changed, 27 insertions(+), 41 deletions(-)
> 

Boris, could you please comment on the patches?


Juergen

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH v2 0/2] xen/gntdev: sanitize user interface handling

Posted by Boris Ostrovsky 32 weeks ago
On 11/28/19 3:48 AM, Jürgen Groß wrote:
> On 07.11.19 12:15, Juergen Gross wrote:
>> The Xen gntdev driver's checking of the number of allowed mapped pages
>> is in need of some sanitizing work.
>>
>> Changes in V2:
>> - enhanced commit message of patch 1 (Andrew Cooper)
>>
>> Juergen Gross (2):
>>    xen/gntdev: replace global limit of mapped pages by limit per call
>>    xen/gntdev: switch from kcalloc() to kvcalloc()
>>
>>   drivers/xen/gntdev-common.h |  2 +-
>>   drivers/xen/gntdev-dmabuf.c | 11 +++------
>>   drivers/xen/gntdev.c        | 55
>> +++++++++++++++++++--------------------------
>>   3 files changed, 27 insertions(+), 41 deletions(-)
>>
>
> Boris, could you please comment on the patches?

Sure, both look good (and so R-b). I haven't commented since Oleksandr
already gave you his.


-boris

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel