Re: [PATCH 2/2] drm/vc4: Add get/set tiling ioctls.

From: Eric Anholt
Date: Fri Jun 16 2017 - 14:00:57 EST

Daniel Stone <daniel@xxxxxxxxxxxxx> writes:

> On 13 June 2017 at 16:49, Eric Anholt <eric@xxxxxxxxxx> wrote:
>> Daniel Stone <daniel@xxxxxxxxxxxxx> writes:
>>> I posted a DRI3 v1.1 patch series which can advertise and also transit
>>> modifiers directly under X11, and have also typed up the support for
>>> Wayland which is working just fine with Weston from git. If you
>>> implement DRIimage v15 to advertise and import modifiers, then you can
>>> transit them for free without a magic-back-channel ioctl. Would that
>>> be enough to convince you to drop this series?
>> Not really -- this patch is pretty small, and doesn't require updating
>> the entire world.
> The modifier interface is already landed in mainline for KMS, GBM, and
> Gallium. It's supported in i965 and freedreno, and Lucas has patches
> to support it for etnaviv/imx-drm as well.
> While I get that the {get,set}_tiling interface is necessary to route
> around the X11 support not existing until very recently, I'm unhappy
> that it's now landed in mainline imposing a performance penalty on
> everyone else (Wayland compositors, Kodi, etc etc), with no way to
> route around it.

Is your wayland compositor planes-only? Because if it's doing any GL
compositing, this will be a huge win for it. I'd recommend actually
trying out the code to see. (I considered this tradeoff when deciding
to make the change).

Kodi, yes, this will be a small performance hit for. Given that we're
not even using hardware decode in the current Kodi pipeline, worrying
about this seems misplaced. The fix would be pretty trivial, though:
that sends that hint down to gallium, which already has that distinction
in its flags.

Attachment: signature.asc
Description: PGP signature