From mboxrd@z Thu Jan 1 00:00:00 1970 From: Imre Deak Subject: Re: [Intel-gfx] [PATCH 1/3] drm: Add an "expose 3d modes" property Date: Thu, 04 Oct 2012 16:56:24 +0300 Message-ID: <1349358984.2423.4.camel@localhost> References: <1348771268-3436-1-git-send-email-damien.lespiau@gmail.com> <1348771268-3436-2-git-send-email-damien.lespiau@gmail.com> <20121004133551.GG5661@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20121004133551.GG5661@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org 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 > > > > 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 > > 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