Depth 30 enablement for ati-ddx + exa. Rev 3

[Michel Dänzer]

On 2018-01-26 10:42 PM, Mario Kleiner wrote:
> On 01/25/2018 05:06 PM, Michel Dänzer wrote:
>> On 2018-01-24 06:35 PM, Mario Kleiner wrote:
>>>
>>> It only happens if a client wants a fbconfig with alpha channel, for
>>> destination alpha blending etc., as my application happens to. If you
>>> only run glxgears or a desktop compositor like kde-5's, it won't happen
>>> as they only use bgrx configs without alpha.
>>
>> I sent out a patch which should fix this, please test.
> 
> Yes, that patch "[PATCH xf86-video-ati] Always use screen depth/bpp for
> KMS framebuffers" makes sense to me and fixes it nicely, as tested under
> dri3 and dri2 with glamor and exa:
> 
> Reviewed-and-Tested-by: Mario Kleiner <mario.kleiner.de at gmail.com>

Thanks, pushed.


> I assume amdgpu-ddx will need the same treatment, as "Use reference
> counting for tracking KMS framebuffer lifetimes" which caused the
> problem has been ported to amdgpu-ddx.

Right. All applicable patches get ported between xf86-video-ati/amdgpu
in due time.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer