[Intel-gfx] [PATCH v2] Revert "sna: Refresh last detection timestamp on hotplug notifies"

Dave Airlie airlied at gmail.com
Mon Jun 27 20:07:56 UTC 2016


On 18 June 2016 at 05:57, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> On Fri, Jun 17, 2016 at 03:44:34PM -0400, Lyude wrote:
>> From: Lyude Paul <cpaul at redhat.com>
>>
>> DRM does not always update the status of each connector during a
>> hotplug event, and it's generally expected that userspace is supposed to
>> handle that by reprobing. This happens in a couple situations:
>> suspend/resume, MST hotplugs, and probably a few others. As a result,
>> making this assumption actually breaks MST hotplugging.
>
> Generally expected? So logind reprobes? We force a probe on wakeup?

Yes, expected by the API from day one.

If you want an API to do something else, write a new API. Don't go shoehorning
behaviour into a generic API that is driver specific.

So please fix the DDX or I'll push the patch and you can revert it if
you don't like but at least
then I know you are being painful and I can justify carrying the patch
upstream until the
Intel DDX is no longer something we care about.

Dave.


More information about the Intel-gfx mailing list