[PATCH 2/2] randr: add provider object (v3.1)

[Aaron Plattner]

On 06/28/2012 03:21 AM, Dave Airlie wrote:
> On Thu, Jun 28, 2012 at 10:36 AM, Dave Airlie <airlied at gmail.com> wrote:
>>>>    /* Event selection bits */
>>>>    #define RRScreenChangeNotifyMask  (1L << 0)
>>>>    /* V1.2 additions */
>>>>    #define RRCrtcChangeNotifyMask     (1L << 1)
>>>>    #define RROutputChangeNotifyMask    (1L << 2)
>>>>    #define RROutputPropertyNotifyMask  (1L << 3)
>>>> +#define RRProviderPropertyNotifyMask        (1L << 4)
>>>
>>>
>>> Do we need a RRProviderChangeNotifyMask?  Can providers dynamically appear
>>> and disappear?  I assume they can due to things like USB hotplug.
>>>
>
> I was wondering about this a bit more, I've added stuff in v5, to
> notify off role
> changes which seems useful, however I'm no sure how we notify of a provider
> disappearing, it seems like it would be akin to an output/crtc disappearing
> which we also don't support yet. I'm not sure if we should be sending
> screen change
> in that event, or we need a new RRScreenResourcesChangeNotifyMask, to provoke
> a complete rescan.

Hmm, that's a good point.  The server itself does support dynamically 
creating and destroying outputs and CRTCs, but it looks like there's no 
other event generated.  I guess just punting on Provider events for now 
is reasonable.

Keith, thoughts?

-- Aaron

-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------