Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1667951

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

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Daniel Stone <daniel@fooishbar.org> writes:

> On 13 June 2017 at 16:49, Eric Anholt <eric@anholt.net> wrote:
>> Daniel Stone <daniel@fooishbar.org> 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:
make a new GBM_BO_USE_RENDER_TARGET as a subset of GBM_BO_USE_RENDERING
that sends that hint down to gallium, which already has that distinction
in its flags.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] drm/vc4: Add T-format scanout support. Eric Anholt <eric@anholt.net> - 2017-06-08 02:20 +0200
  [PATCH 2/2] drm/vc4: Add get/set tiling ioctls. Eric Anholt <eric@anholt.net> - 2017-06-08 02:20 +0200
    Re: [PATCH 2/2] drm/vc4: Add get/set tiling ioctls. Boris Brezillon <boris.brezillon@free-electrons.com> - 2017-06-13 09:50 +0200
      Re: [PATCH 2/2] drm/vc4: Add get/set tiling ioctls. Eric Anholt <eric@anholt.net> - 2017-06-16 00:20 +0200
    Re: [PATCH 2/2] drm/vc4: Add get/set tiling ioctls. Daniel Stone <daniel@fooishbar.org> - 2017-06-13 10:40 +0200
      Re: [PATCH 2/2] drm/vc4: Add get/set tiling ioctls. Eric Anholt <eric@anholt.net> - 2017-06-13 17:50 +0200
        Re: [PATCH 2/2] drm/vc4: Add get/set tiling ioctls. Daniel Stone <daniel@fooishbar.org> - 2017-06-16 11:10 +0200
          Re: [PATCH 2/2] drm/vc4: Add get/set tiling ioctls. Eric Anholt <eric@anholt.net> - 2017-06-16 20:10 +0200
  Re: [PATCH 1/2] drm/vc4: Add T-format scanout support. Boris Brezillon <boris.brezillon@free-electrons.com> - 2017-06-13 09:50 +0200

csiph-web