[igt-dev] [PATCH 0/7] some cleanups and uapi clarification for leases

Daniel Vetter daniel at ffwll.ch
Wed Apr 24 09:31:40 UTC 2019


On Fri, Apr 05, 2019 at 12:40:14PM +1000, Dave Airlie wrote:
> On Thu, 14 Mar 2019 at 18:58, Boris Brezillon
> <boris.brezillon at collabora.com> wrote:
> >
> > On Thu, 28 Feb 2019 15:49:03 +0100
> > Daniel Vetter <daniel.vetter at ffwll.ch> wrote:
> >
> > > Hi all,
> > >
> > > Nothing too major, only things I did find in all my igt test extending for
> > > drm lease is some corner cases around implicit planes and atomic target
> > > crtcs. Review and comments very much appreciated.
> > >
> > > Cheers, Daniel
> > >
> > > Test-with: 20190228141918.26043-1-daniel.vetter at ffwll.ch
> > >
> > > Daniel Vetter (7):
> > >   drm/leases: Drop object_id validation for negative ids
> > >   drm/lease: Drop recursive leads checks
> > >   drm/leases: Don't init to 0 in drm_master_create
> > >   drm/lease: Check for lessor outside of locks
> > >   drm/lease: Make sure implicit planes are leased
> > >   drm/atomic: Wire file_priv through for property changes
> > >   drm/atomic: -EACCESS for lease-denied crtc lookup
> >
> > I had a look at patches 5-7 and they seem to do the right thing, but my
> > knowledge on DRM leases is quite limited, and given all the corner
> > cases of legacy CRTC/plane updates, I'm not comfortable adding my R-b
> > on these.
> 
> For the series:
> 
> Reviewed-by: Dave Airlie <airlied at redhat.com>

Finally gotten around to pulling it all in, thanks for all your reviews.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


More information about the igt-dev mailing list