Re: [PATCH 2/3] drm/tegra: Correct idr_alloc() minimum id

2017-05-14 Thread Mikko Perttunen
On 05/12/2017 10:00 PM, Dmitry Osipenko wrote: The start = 0 is invalid and causes weird CDMA channel timeouts, presumably some memory misuse/corruption is going on. What makes you think start = 0 is invalid? I can't see anything pointing to that in the idr code and there are many users in the

Re: [PATCH 2/3] drm/tegra: Correct idr_alloc() minimum id

2017-05-14 Thread Dmitry Osipenko
On 14.05.2017 14:53, Mikko Perttunen wrote: > On 05/12/2017 10:00 PM, Dmitry Osipenko wrote: >> The start = 0 is invalid and causes weird CDMA channel timeouts, presumably >> some memory misuse/corruption is going on. > > What makes you think start = 0 is invalid? I can't see anything pointing to

Re: [PATCH 2/3] drm/tegra: Correct idr_alloc() minimum id

2017-05-14 Thread Dmitry Osipenko
On 14.05.2017 16:02, Dmitry Osipenko wrote: > On 14.05.2017 14:53, Mikko Perttunen wrote: >> On 05/12/2017 10:00 PM, Dmitry Osipenko wrote: >>> The start = 0 is invalid and causes weird CDMA channel timeouts, presumably >>> some memory misuse/corruption is going on. >> >> What makes you think start

[PATCH 2/3] drm/tegra: Correct idr_alloc() minimum id

2017-05-12 Thread Dmitry Osipenko
The start = 0 is invalid and causes weird CDMA channel timeouts, presumably some memory misuse/corruption is going on. Fixes: bdd2f9cd10eb ("drm/tegra: Don't leak kernel pointer to userspace") Signed-off-by: Dmitry Osipenko --- drivers/gpu/drm/tegra/drm.c | 2 +- 1 file changed, 1 insertion(+),