Dave Airlie <[email protected]> writes:

>       dix/xf86: initial offload slave tracking (v1.1)
>       randr: add hooks for offload sink provider protocol

So familiar by now.

Reviewed-by: Keith Packard <[email protected]>

>       xf86: store scanout pixmap in the xf86 struct as well.
>       xf86: add callback for offloak sink setting support.

Reviewed-by: Keith Packard <[email protected]>

>       dix/randr: add a hook into screen to replace scanout pixmap

This separates out the CrtcSet from SetScanoutPixmap -- will
that actually work if the size is changing at the same time? Or is this
ABI guaranteed to not set the size? If so, why are there a bunch of size
checks, and why is it bothering to call rrCrtcSet at all?

>       dri2: add initial prime support. (v1.2)

prime_id? Seriously? What happens if you change the offload list while
some client is active?

For cases where a compositing manager is not running, you can simply set
the window to automatic redirect, as is done for windows with weird
visuals today. If someone sets it to manual later, then your automatic
will be overridden, and when that manual redirect goes away, automatic
starts up again. Simple.

Otherwise, this stuff all seems OK to me.

-- 
[email protected]

Attachment: pgpnITrsyW9ke.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to