Re: [PATCH] drm: virtio: reinstate drm_virtio_set_busid()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/18/16 08:38, Greg Kroah-Hartman wrote:
> On Mon, Oct 17, 2016 at 11:10:28PM +0200, Laszlo Ersek wrote:
>> Greg,
>>
>> On 10/03/16 19:43, Laszlo Ersek wrote:
>>> Before commit a325725633c2 ("drm: Lobotomize set_busid nonsense for !pci
>>> drivers"), several DRM drivers for platform devices used to expose an
>>> explicit "drm_driver.set_busid" callback, invariably backed by
>>> drm_platform_set_busid().
>>>
>>> Commit a325725633c2 removed drm_platform_set_busid(), along with the
>>> referring .set_busid field initializations. This was justified because
>>> interchangeable functionality had been implemented in drm_dev_alloc() /
>>> drm_dev_init(), which DRM_IOCTL_SET_VERSION would rely on going forward.
>>>
>>> However, commit a325725633c2 also removed drm_virtio_set_busid(), for
>>> which the same consolidation was not appropriate: this .set_busid callback
>>> had been implemented with drm_pci_set_busid(), and not
>>> drm_platform_set_busid(). The error regressed Xorg/xserver on QEMU's
>>> "virtio-vga" card; the drmGetBusid() function from libdrm would no longer
>>> return stable PCI identifiers like "pci:0000:00:02.0", but rather unstable
>>> platform ones like "virtio0".
>>>
>>> Reinstate drm_virtio_set_busid() with judicious use of
>>>
>>>   git checkout -p a325725633c2^ -- drivers/gpu/drm/virtio
>>>
>>> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
>>> Cc: Daniel Vetter <daniel.vetter@xxxxxxxxx>
>>> Cc: David Airlie <airlied@xxxxxxxxxx>
>>> Cc: Emil Velikov <emil.l.velikov@xxxxxxxxx>
>>> Cc: Gerd Hoffmann <kraxel@xxxxxxxxxx>
>>> Cc: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
>>> Cc: Hans de Goede <hdegoede@xxxxxxxxxx>
>>> Cc: Joachim Frieben <jfrieben@xxxxxxxxxxx>
>>> Cc: stable@xxxxxxxxxxxxxxx
>>> Reported-by: Joachim Frieben <jfrieben@xxxxxxxxxxx>
>>> Fixes: a325725633c26aa66ab940f762a6b0778edf76c0
>>> Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1366842
>>> Signed-off-by: Laszlo Ersek <lersek@xxxxxxxxxx>
>>> Reviewed-by: Emil Velikov <emil.l.velikov@xxxxxxxxx>
>>> ---
>>>  drivers/gpu/drm/virtio/virtgpu_drm_bus.c | 10 ++++++++++
>>>  drivers/gpu/drm/virtio/virtgpu_drv.c     |  1 +
>>>  drivers/gpu/drm/virtio/virtgpu_drv.h     |  1 +
>>>  3 files changed, 12 insertions(+)
>>
>> Is there any particular reason this patch hasn't been cherry-picked to
>> linux-4.8.y? (I checked v4.8.2.)
> 
> Yes, it just showed up in 4.9-rc1 3 days ago, give me a chance to catch
> up!  I have 210 patches right now that are in this same status,

My apologies; I didn't realize the volume.

(
BTW I tried to fetch stable-queue this morning, and just now as well --
for some reason that git command seems consistently stuck for me. It
produces some low network traffic, but makes no progress:

> Looking up git.kernel.org ... done.
> Connecting to git.kernel.org (port 9418) ... 199.204.44.194 done.
> <stuck here>

Should I report this elsewhere and/or capture packets with tcpdump?
)

> you are
> in good company.

Always :)

Thanks!
Laszlo

> Don't worry, it will eventually make it in, patience please...
> 
> greg k-h
> 

--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]