Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-11-01 Thread PrasannaKumar Muralidharan
Hi Daniel, On 1 November 2017 at 14:23, Daniel Vetter wrote: > On Wed, Nov 01, 2017 at 09:48:28AM +0530, PrasannaKumar Muralidharan wrote: >> Hi Daniel, >> >> On 31 October 2017 at 21:57, Daniel Vetter wrote: >> > On Tue, Oct 31, 2017 at 08:37:21PM +0530, PrasannaKumar Muralidharan wrote: >> >>

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-11-01 Thread Daniel Vetter
On Wed, Nov 01, 2017 at 09:48:28AM +0530, PrasannaKumar Muralidharan wrote: > Hi Daniel, > > On 31 October 2017 at 21:57, Daniel Vetter wrote: > > On Tue, Oct 31, 2017 at 08:37:21PM +0530, PrasannaKumar Muralidharan wrote: > >> My patch is supposed to catch problem with drivers. It warns when > >

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-31 Thread PrasannaKumar Muralidharan
Hi Daniel, On 31 October 2017 at 21:57, Daniel Vetter wrote: > On Tue, Oct 31, 2017 at 08:37:21PM +0530, PrasannaKumar Muralidharan wrote: >> My patch is supposed to catch problem with drivers. It warns when >> vblank refcount is non-zero in drm_vblank_cleanup call. From CI log it >> can be seen

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-31 Thread Daniel Vetter
On Tue, Oct 31, 2017 at 08:37:21PM +0530, PrasannaKumar Muralidharan wrote: > Hi Daniel, > > On 31 October 2017 at 15:51, Daniel Vetter wrote: > > > > On Mon, Oct 30, 2017 at 06:01:12PM +0530, PrasannaKumar Muralidharan wrote: > > > Hi Daniel, > > > > > > On 30 October 2017 at 15:40, Daniel Vette

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-31 Thread PrasannaKumar Muralidharan
Hi Daniel, On 31 October 2017 at 15:51, Daniel Vetter wrote: > > On Mon, Oct 30, 2017 at 06:01:12PM +0530, PrasannaKumar Muralidharan wrote: > > Hi Daniel, > > > > On 30 October 2017 at 15:40, Daniel Vetter wrote: > > > On Wed, Oct 25, 2017 at 08:44:45PM +0530, PrasannaKumar Muralidharan > > >

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-31 Thread Daniel Vetter
On Mon, Oct 30, 2017 at 06:01:12PM +0530, PrasannaKumar Muralidharan wrote: > Hi Daniel, > > On 30 October 2017 at 15:40, Daniel Vetter wrote: > > On Wed, Oct 25, 2017 at 08:44:45PM +0530, PrasannaKumar Muralidharan wrote: > >> Hi All, > >> > >> On 24 October 2017 at 22:18, PrasannaKumar Muralidh

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-30 Thread PrasannaKumar Muralidharan
Hi Daniel, On 30 October 2017 at 15:40, Daniel Vetter wrote: > On Wed, Oct 25, 2017 at 08:44:45PM +0530, PrasannaKumar Muralidharan wrote: >> Hi All, >> >> On 24 October 2017 at 22:18, PrasannaKumar Muralidharan >> wrote: >> > In i915 driver unload drm_vblank_get is added to test whether >> > dr

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-30 Thread Daniel Vetter
On Wed, Oct 25, 2017 at 08:44:45PM +0530, PrasannaKumar Muralidharan wrote: > Hi All, > > On 24 October 2017 at 22:18, PrasannaKumar Muralidharan > wrote: > > In i915 driver unload drm_vblank_get is added to test whether > > drm_vblank_cleanup refcount validation patch is working. > > > > Signed-

Re: [Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-25 Thread PrasannaKumar Muralidharan
Hi All, On 24 October 2017 at 22:18, PrasannaKumar Muralidharan wrote: > In i915 driver unload drm_vblank_get is added to test whether > drm_vblank_cleanup refcount validation patch is working. > > Signed-off-by: PrasannaKumar Muralidharan > --- > Changes in v2: > Use drm_crtc_vblank_get instead

[Intel-gfx] [PATCH v2 2/2] Test case for drm_vblank_cleanup refcount validation patch

2017-10-24 Thread PrasannaKumar Muralidharan
In i915 driver unload drm_vblank_get is added to test whether drm_vblank_cleanup refcount validation patch is working. Signed-off-by: PrasannaKumar Muralidharan --- Changes in v2: Use drm_crtc_vblank_get instead of _put. In previous patch _put was wrongly used. drivers/gpu/drm/i915/i915_drv.c |