[Mesa-dev] [PATCH] i965: import prime buffers in the current context, not screen

[Eric Anholt]

Martin Peres <martin.peres at linux.intel.com> writes:

> This mirrors the codepath taken by DRI2 in IntelSetTexBuffer2() and
> fixes many applications when using DRI3:
>  - Totem with libva on hw-accelerated decoding
>  - obs-studio, using Window Capture (Xcomposite) as a Source
>  - gstreamer with VAAPI
>
> Cc: mesa-stable at lists.freedesktop.org
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71759
> Tested-by: Fabrice Bellet <fabrice at bellet.info>
> Signed-off-by: Martin Peres <martin.peres at linux.intel.com>
> ---
>
> I have been testing the patch on my main desktop for a day and did not
> encounter any issue with it. This is however quite a corner case and
> most people never had this issue anyway.
>
> If this patch is not accepted, then we will need to change the loader's
> GetBuffers() method so as it either does not import the buffers or takes
> the context as a parameter.

Not sure if you've seen, but the gallium drivers are doing an fstat on
the DRM fd, hashing the st_dev/st_ino/st_rdev, and reusing an existing
screen when it matches.  This might be really useful for Intel, too.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160804/313d4c95/attachment.sig>