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

[Christian König]

Hi Suresh,

> If I miss something here please provide some more details on the question.
Emils question is if the "surfaceless" platform is identical to the 
"drm" platform why not use the drm platform in the first place?

And yeah, that is a rather good question.

Regards,
Christian.

Am 11.10.2017 um 08:13 schrieb Guttula, Suresh:
> HI,
>
>>> - why do we need "surfaceless" support
>     ChromeOS supports surfacelsess and we need this va enablement for surfaceless in chromium.
>>> - does upstream VAAPI has surfaceless platform
>      Yes. It uses headless support of VA-API for decoding.
>   - 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.
>
> Thanks,
> Suresh G
>
> -----Original Message-----
> From: Emil Velikov [mailto:emil.l.velikov at gmail.com]
> Sent: Tuesday, October 10, 2017 9:09 PM
> To: Guttula, Suresh <Suresh.Guttula at amd.com>
> Cc: ML mesa-dev <mesa-dev at lists.freedesktop.org>; Sharma, Deepak <Deepak.Sharma at amd.com>; Koenig, Christian <Christian.Koenig at amd.com>; ML mesa-stable <mesa-stable at lists.freedesktop.org>; Emil Velikov <emil.velikov at collabora.com>
> Subject: Re: [Mesa-stable] [PATCH 2/2] auxiliary: use vl_drm_screen_create method for surfaceless
>
> Hi Suresh,
>
> On 10 October 2017 at 07:20, sguttula <suresh.guttula at amd.com> wrote:
>> when display type is VA_DISPLAY_DRM_RENDERNODES and if drm screen
>> create method is not enabled then vscreen is NULL. To fix this
>> issue,this patch will enable
>> vl_drm_screen_create() for surfaceless platforms
>>
> Repeating some earlier questions:
>
>   - why do we need "surfaceless" support
>   - does upstream VAAPI has surfaceless platform
>   - why is the surfaceless implementation identical to the DRM one
>
> Without any additional information, I think you should be using the drm platform.
>
> Thanks
> Emil