Re: [Intel-gfx] [PULL] drm-misc-next

* Re: [Intel-gfx] [PULL] drm-misc-next
  2019-10-21 15:48       ` Sean Paul
@ 2019-10-22  2:17         ` Dave Airlie
  2019-10-22  7:01           ` Daniel Vetter
  0 siblings, 1 reply; 5+ messages in thread
From: Dave Airlie @ 2019-10-22  2:17 UTC (permalink / raw)
  To: Sean Paul
  Cc: Wen He, Qiang Yu,
	DRM maintainer tools announcements, discussion, and development,
	Sam Ravnborg, Jonas Karlman, Daniel Vetter,
	Intel Graphics Development, Rodrigo Siqueira, dri-devel,
	Steven Price, Oleg Vasilev, Tomi Valkeinen, Laurent Pinchart,
	Thomas Zimmermann, Robert Chiras, Jean-Jacques Hiblot, Lowry Li,
	Ezequiel Garcia, Gerd Hoffmann

On Tue, 22 Oct 2019 at 01:49, Sean Paul <sean@poorly.run> wrote:
>
> On Mon, Oct 21, 2019 at 4:11 AM Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
> >
> > Hi,
> >
> > On 18/10/2019 23:11, Sean Paul wrote:
> > > On Fri, Oct 18, 2019 at 9:46 AM Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
> > >>
> > >> Hi Sean,
> > >>
> > >> On 17/10/2019 22:26, Sean Paul wrote:
> > >>
> > >>> concern for those. The omap OMAP_BO_MEM_* changes though I don't think have
> > >>> really reached non-TI eyes. There's no link in the commit message to a UAPI
> > >>> implementation and the only reference I could find is libkmsxx which can set
> > >>> them through the python bindings. Since this is TI-specific gunk in TI-specific
> > >>> headers I'm inclined to let it pass, but I would have liked to have this
> > >>> conversation upfront. I figured I'd call this out so you have final say.
> > >>
> > >> There was some discussion about that a few years back when I initially
> > >> sent the patches, but now that I look, the discussion died before really
> > >> even starting.
> > >>
> > >> This is what I said about userspace implementation:
> > >>
> > >>> Yes, unfortunately that is not going to happen. I don't see how this
> > >>> could be used in a generic system like Weston or X. It's meant for very
> > >>> specific use cases, where you know exactly the requirements of your
> > >>> application and the capabilities of the whole system, and optimize based
> > >>> on that.
> > >
> > > Thanks for the context, Tomi.
> > >
> > > Indeed it looks like the discussion died, but Laurent still brought up
> > > the u/s requirement and the patch was effectively dead until Daniel or
> > > Dave weighed in. I'd expect at least some outreach before pushing the
> > > patch directly 2+ years later. Has anything changed since then?
> >
> > There were new review rounds for the series this summer & autumn, but
> > no, nothing else. I haven't specifically pinged anyone about the UAPI
> > changes.
> >
> > This series introduces three new flags to an already existing UAPI, and,
> > for whatever reason, this didn't register to me as a new UAPI, even if
> > Laurent asked about it some years back.
> >
> > So, my mistake.
> >
> > The flags are added in a single patch, so I can easily push a revert for
> > that if this patch is not acceptable. The rest of the series is cleanup.
> >
>
> I'll wait for Dave or Daniel to weigh in on whether they want to take
> this, otherwise I'll revert before sending the next pull and we can
> have this conversation on the review.

I'd rather we revert it, just to stick to some semblance of the rules,
otherwise if we make execptions other people will drive trucks through
them.

Dave.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 5+ messages in thread