Re: [next-20170609] WARNING: CPU: 3 PID: 71167 at lib/idr.c:157 idr_replace

2017-06-22 Thread Tejun Heo
On Thu, Jun 22, 2017 at 12:31:35AM +0100, Chris Wilson wrote: > Quoting Tejun Heo (2017-06-13 14:58:49) > > Cc'ing David Airlie. > > > > This is from drm driver calling in idr_replace() w/ a negative id. > > Probably a silly bug in error handling path? > > No, this is the validation of an invalid

Re: [next-20170609] WARNING: CPU: 3 PID: 71167 at lib/idr.c:157 idr_replace

2017-06-21 Thread Chris Wilson
Quoting Tejun Heo (2017-06-13 14:58:49) > Cc'ing David Airlie. > > This is from drm driver calling in idr_replace() w/ a negative id. > Probably a silly bug in error handling path? No, this is the validation of an invalid userspace handle. The drm ABI for handles is supposed to be a full u32 rang

Re: [next-20170609] WARNING: CPU: 3 PID: 71167 at lib/idr.c:157 idr_replace

2017-06-21 Thread Dave Airlie
Cc'ing dri-devel. Dave. On Tue, 13 Jun 2017, Tejun Heo wrote: > Cc'ing David Airlie. > > This is from drm driver calling in idr_replace() w/ a negative id. > Probably a silly bug in error handling path? > > Thanks. > > On Mon, Jun 12, 2017 at 08:10:54PM +0530, Abdul Haleem wrote: > > Hi, > >

Re: [next-20170609] WARNING: CPU: 3 PID: 71167 at lib/idr.c:157 idr_replace

2017-06-13 Thread Tejun Heo
Cc'ing David Airlie. This is from drm driver calling in idr_replace() w/ a negative id. Probably a silly bug in error handling path? Thanks. On Mon, Jun 12, 2017 at 08:10:54PM +0530, Abdul Haleem wrote: > Hi, > > WARN_ON_ONCE is being called from idr_replace() function in file > lib/idr.c at li

[next-20170609] WARNING: CPU: 3 PID: 71167 at lib/idr.c:157 idr_replace

2017-06-12 Thread Abdul Haleem
Hi, WARN_ON_ONCE is being called from idr_replace() function in file lib/idr.c at line 157 struct radix_tree_node *node; void __rcu **slot = NULL; void *entry; if (WARN_ON_ONCE(id < 0)) return ERR_PTR(-EINVAL); if (WARN_ON_ONCE(radix_tree_is_internal_node(ptr)))