]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
xen: fix gntdev
authorRoger Pau Monné <royger@FreeBSD.org>
Wed, 2 May 2018 10:19:17 +0000 (10:19 +0000)
committerRoger Pau Monné <royger@FreeBSD.org>
Wed, 2 May 2018 10:19:17 +0000 (10:19 +0000)
commit2602ef7cfabf0fe981066bf9b60917f8500e898a
treebfd44bad7c9ce1f9b9743dd1a9eda18e330f1abc
parentfc9bdb4ee5086fb706da0ee61f46487e735186b5
xen: fix gntdev

Current interface to the gntdev in FreeBSD is wrong, and mostly worked
out of luck before the PTI FreeBSD fixes, when kernel and user-space
where sharing the same page tables.

On FreeBSD ioctls have the size of the passed struct encoded in the
ioctl number, because the generic ioctl handler in the OS takes care
of copying the data from user-space to kernel space, and then calls
the device specific ioctl handler. Thus using ioctl structs with
variable sizes is not possible.

The fix is to turn the array of structs at the end of
ioctl_gntdev_alloc_gref and ioctl_gntdev_map_grant_ref into pointers,
that can be properly accessed from the kernel gntdev driver using the
copyin/copyout functions. Note that this is exactly how it's done for
the privcmd driver.

Sponsored by:   Citrix Systems R&D
sys/dev/xen/gntdev/gntdev.c
sys/xen/gntdev.h