[Mesa-stable] [PATCH 2/2] auxiliary: use vl_drm_screen_create method for surfaceless

[Emil Velikov]

On 11 October 2017 at 07:13, Guttula, Suresh <Suresh.Guttula at amd.com> wrote:
> HI,
>
>>- why do we need "surfaceless" support
>    ChromeOS supports surfacelsess and we need this va enablement for surfaceless in chromium.
Ack, that should have been part of the commit message.

>> - does upstream VAAPI has surfaceless platform
>     Yes. It uses headless support of VA-API for decoding.
There's no VA_DISPLAY_SURFACELESS in libva [1]. Thus adding one here
is _very_ confusing and misleading.

https://github.com/01org/libva/blob/master/va/va_backend.h#L39

>>  - why is the surfaceless implementation identical to the DRM one
>     If I understand your question correctly, In case of surfaceless platform ,it uses headless support of VAAPI, which will use drm implementation. If I miss something here please provide some more details on the question.
>
To put it otherwise:

You're "adding" support for surfaceless for the sake of adding a name.
There's no functional difference nor upstream (see the libva question
above) demand for it.

What is stopping you from using --with-platforms=drm ?

Thanks
Emil