From: Imre Deak <imre.deak@intel.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH 1/3] drm: Add an "expose 3d modes" property
Date: Thu, 04 Oct 2012 16:56:24 +0300 [thread overview]
Message-ID: <1349358984.2423.4.camel@localhost> (raw)
In-Reply-To: <20121004133551.GG5661@phenom.ffwll.local>
On Thu, 2012-10-04 at 15:35 +0200, Daniel Vetter wrote:
> On Thu, Sep 27, 2012 at 07:41:06PM +0100, Damien Lespiau wrote:
> > From: Damien Lespiau <damien.lespiau@intel.com>
> >
> > The "expose 3D modes" property can be attached to connectors to allow
> > user space to indicate it can deal with 3D modes and that the drm driver
> > should expose those 3D modes.
> >
> > Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
>
> I've thought a bit more about this and I don't like the connector prop so
> much any more:
> - connector properties stick around, so if you mix up your userspace
> things could go wrong (e.g. i-g-t tests and ddx).
> - connector properties are exposed to users by default.
> - it makes little sense for userspace to not enable this on all
> connectors.
>
> So I think a per-file_priv option to not filter out 3d modes would be
> better suited.
>
> We don't yet have any driver-agnostic ioctl yet for such things, so would
> require a bit more work ...
This would be useful for the monotonic time stamp support too. At the
moment the client would have to set a flag whether it wants real or raw
timestamp for each wait-for-vblank and page-flip ioctl, though what it
really needs is a one time setting.
--Imre
next prev parent reply other threads:[~2012-10-04 13:56 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-27 18:41 Stereo 3D modes support v2 Damien Lespiau
2012-09-27 18:41 ` [PATCH 1/3] drm: Add an "expose 3d modes" property Damien Lespiau
2012-09-29 0:45 ` [Intel-gfx] " Rodrigo Vivi
2012-09-29 9:01 ` Chris Wilson
2012-10-04 13:35 ` Daniel Vetter
2012-10-04 13:56 ` Imre Deak [this message]
2012-09-27 18:41 ` [PATCH 2/3] drm: Parse the HDMI cea vendor block for 3D present Damien Lespiau
2012-09-29 0:46 ` Rodrigo Vivi
2012-09-27 18:41 ` [PATCH 3/3] drm/i915: Add HDMI vendor info frame support Damien Lespiau
2012-09-29 0:50 ` [Intel-gfx] " Rodrigo Vivi
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=1349358984.2423.4.camel@localhost \
--to=imre.deak@intel.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.