From: Eric Anholt <eric@anholt.net>
To: Daniel Stone <daniel@fooishbar.org>
Cc: dri-devel <dri-devel@lists.freedesktop.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Boris Brezillon <boris.brezillon@free-electrons.com>
Subject: Re: [PATCH 2/2] drm/vc4: Add get/set tiling ioctls.
Date: Tue, 13 Jun 2017 08:49:17 -0700 [thread overview]
Message-ID: <87y3svyjk2.fsf@eliezer.anholt.net> (raw)
In-Reply-To: <CAPj87rP1OxFxNok8-yyTgdS75BSZz8kBTStfWuq0i0e6FTp41Q@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1067 bytes --]
Daniel Stone <daniel@fooishbar.org> writes:
> Hi Eric,
>
> On 8 June 2017 at 01:13, Eric Anholt <eric@anholt.net> wrote:
>> This allows mesa to set the tiling format for a BO and have that
>> tiling format be respected by mesa on the other side of an
>> import/export (and by vc4 scanout in the kernel), without defining a
>> protocol to pass the tiling through userspace.
>
> 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.
I've been delaying writing this patch since, what, XDC last year,
waiting for the modifiers pipeline to materialize, and I'm not convinced
that this One Last Patchset is going to land soon enough.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]
next prev parent reply other threads:[~2017-06-13 15:49 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-08 0:13 [PATCH 1/2] drm/vc4: Add T-format scanout support Eric Anholt
2017-06-08 0:13 ` [PATCH 2/2] drm/vc4: Add get/set tiling ioctls Eric Anholt
2017-06-13 7:46 ` Boris Brezillon
2017-06-15 22:11 ` Eric Anholt
2017-06-13 8:38 ` Daniel Stone
2017-06-13 15:49 ` Eric Anholt [this message]
2017-06-16 9:08 ` Daniel Stone
2017-06-16 18:00 ` Eric Anholt
2017-06-13 7:47 ` [PATCH 1/2] drm/vc4: Add T-format scanout support Boris Brezillon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87y3svyjk2.fsf@eliezer.anholt.net \
--to=eric@anholt.net \
--cc=boris.brezillon@free-electrons.com \
--cc=daniel@fooishbar.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®