Re: [PATCH 6/6] drm: Add four ioctls for managing drm mode object leases [v4]

2017-10-10 Thread Keith Packard
Dave Airlie writes: >> +*/ >> + ret = idr_alloc(&leases, &drm_lease_idr_object , object_id, >> object_id + 1, GFP_KERNEL); > > In current kernels, the idr_alloc warns if start < 0, so if object_id > is a negative signed integer, which your > igt tests actually make

Re: [PATCH 6/6] drm: Add four ioctls for managing drm mode object leases [v4]

2017-10-10 Thread Dave Airlie
> +*/ > + ret = idr_alloc(&leases, &drm_lease_idr_object , object_id, > object_id + 1, GFP_KERNEL); In current kernels, the idr_alloc warns if start < 0, so if object_id is a negative signed integer, which your igt tests actually make it. So we get an ugly debug spl

[PATCH 6/6] drm: Add four ioctls for managing drm mode object leases [v4]

2017-10-04 Thread Keith Packard
drm_mode_create_lease Creates a lease for a list of drm mode objects, returning an fd for the new drm_master and a 64-bit identifier for the lessee drm_mode_list_lesees List the identifiers of the lessees for a master file drm_mode_get_lease List the leased obje