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

[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