]> git.baikalelectronics.ru Git - kernel.git/commit
drm/virtio: document drm_dev_set_unique workaround
authorEmil Velikov <emil.velikov@collabora.com>
Wed, 24 Oct 2018 14:42:52 +0000 (15:42 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Thu, 25 Oct 2018 12:25:58 +0000 (13:25 +0100)
commit7960f19bb7898af3359690b115fd57d3ff22d548
treeaa21006993e69411aa7dd0dad8565d18174117c4
parentfd87bc8a126da87e46056c7e00ed4cfcbe37d27f
drm/virtio: document drm_dev_set_unique workaround

A while back we removed it, yet that lead to regressions. At some later
point, I've attempted to remove it again without fully grasping the
unique (pun intended) situation that virtio is in.

Add a bulky comment to document why the call should stay as-is, for the
next person who's around.

As a Tl;Dr: virtio sits on top of struct virtio_device, which confuses
dev_is_pci(), wrong info gets sent to userspace and X doesn't start.
Driver needs to explicitly call drm_dev_set_unique() to keep it working.

v2: Fix handful of typos (Laszlo)

Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Hans de Goede <hdegoede@redhat.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181024144252.16518-1-emil.l.velikov@gmail.com
drivers/gpu/drm/virtio/virtgpu_drm_bus.c