[Mesa-dev] radeonsi: always set the scanout flag?
axel.davy at ens.fr
Wed Nov 27 07:37:34 PST 2013
This fixes the issue for me,
However looking at your patch, I believe you made a small mistake.
I think with your patch, the RADEON_SURF_SCANOUT flag
would be always be set for cards inferior to SI,
since your calculation of scanout:
*scanout = !(bo->rws->gen >= DRV_SI && args.tiling_flags &
Would make scanout set to 1 when the card is inferior to SI.
I think the correct calculus is:
*scanout = (bo->rws->gen >= DRV_SI) && !(args.tiling_flags &
Thank you for having fixed this issue,
On 26/11/2013, Marek Olšák wrote :
> Does the attached patch fix the issue for you?
> On Tue, Nov 26, 2013 at 9:20 AM, Axel Davy <axel.davy at ens.fr> wrote:
>> When importing an handle (src/gallium/drivers/radeon/r600_texture.c),
>> the RADEON_SURF_SCANOUT flag is always set on SI.
>> The code is associated with a comment: /* always set the scanout flags on SI
>> I was getting bad tiling mode on Wayland with my radeonsi card, and recent
>> and I discovered that setting this flag there was causing my tiling issue.
>> I think one possible reason for this code is X dri2, since the DDX allocates
>> the buffers with this flag.
>> Could this flag be passed via buffer_get_tiling?
>> If not, then there would probably need to force this flag when creating a
>> resource on SI.
>> But ideally, if it makes a performance difference, we would like to be able
>> to control that
>> with Wayland when allocating the buffers.
>> Axel Davy
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
More information about the mesa-dev