All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm: Avoid printing negative values for unsigned variables.
@ 2016-08-03 16:53 Rodrigo Vivi
  2016-08-03 17:23 ` ✗ Ro.CI.BAT: failure for " Patchwork
  2016-08-03 19:04 ` [Intel-gfx] [PATCH] " Ville Syrjälä
  0 siblings, 2 replies; 4+ messages in thread
From: Rodrigo Vivi @ 2016-08-03 16:53 UTC (permalink / raw)
  To: intel-gfx; +Cc: dri-devel, Rodrigo Vivi

It was really strange to see negative vblank seqs on debug
messages. It is rare to have that big number, but when it
happens it is confusing and misleading.

Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 drivers/gpu/drm/drm_irq.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c
index 77f357b..01a5079 100644
--- a/drivers/gpu/drm/drm_irq.c
+++ b/drivers/gpu/drm/drm_irq.c
@@ -1295,7 +1295,7 @@ void drm_vblank_off(struct drm_device *dev, unsigned int pipe)
 		if (e->pipe != pipe)
 			continue;
 		DRM_DEBUG("Sending premature vblank event on disable: "
-			  "wanted %d, current %d\n",
+			  "wanted %u, current %u\n",
 			  e->event.sequence, seq);
 		list_del(&e->base.link);
 		drm_vblank_put(dev, pipe);
@@ -1585,7 +1585,7 @@ static int drm_queue_vblank_event(struct drm_device *dev, unsigned int pipe,
 
 	seq = drm_vblank_count_and_time(dev, pipe, &now);
 
-	DRM_DEBUG("event on vblank count %d, current %d, crtc %u\n",
+	DRM_DEBUG("event on vblank count %u, current %u, crtc %u\n",
 		  vblwait->request.sequence, seq, pipe);
 
 	trace_drm_vblank_event_queued(current->pid, pipe,
@@ -1693,7 +1693,7 @@ int drm_wait_vblank(struct drm_device *dev, void *data,
 		return drm_queue_vblank_event(dev, pipe, vblwait, file_priv);
 	}
 
-	DRM_DEBUG("waiting on vblank count %d, crtc %u\n",
+	DRM_DEBUG("waiting on vblank count %u, crtc %u\n",
 		  vblwait->request.sequence, pipe);
 	DRM_WAIT_ON(ret, vblank->queue, 3 * HZ,
 		    (((drm_vblank_count(dev, pipe) -
@@ -1708,7 +1708,7 @@ int drm_wait_vblank(struct drm_device *dev, void *data,
 		vblwait->reply.tval_sec = now.tv_sec;
 		vblwait->reply.tval_usec = now.tv_usec;
 
-		DRM_DEBUG("returning %d to client\n",
+		DRM_DEBUG("returning %u to client\n",
 			  vblwait->reply.sequence);
 	} else {
 		DRM_DEBUG("vblank wait interrupted by signal\n");
@@ -1735,7 +1735,7 @@ static void drm_handle_vblank_events(struct drm_device *dev, unsigned int pipe)
 		if ((seq - e->event.sequence) > (1<<23))
 			continue;
 
-		DRM_DEBUG("vblank event on %d, current %d\n",
+		DRM_DEBUG("vblank event on %u, current %u\n",
 			  e->event.sequence, seq);
 
 		list_del(&e->base.link);
-- 
2.5.5

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 4+ messages in thread
* ✗ Ro.CI.BAT: failure for drm: Avoid printing negative values for unsigned variables.
  2016-08-03 16:53 [PATCH] drm: Avoid printing negative values for unsigned variables Rodrigo Vivi
@ 2016-08-03 17:23 ` Patchwork
  2016-08-03 19:04 ` [Intel-gfx] [PATCH] " Ville Syrjälä
  1 sibling, 0 replies; 4+ messages in thread
From: Patchwork @ 2016-08-03 17:23 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-gfx

== Series Details ==

Series: drm: Avoid printing negative values for unsigned variables.
URL   : https://patchwork.freedesktop.org/series/10612/
State : failure

== Summary ==

Series 10612v1 drm: Avoid printing negative values for unsigned variables.
http://patchwork.freedesktop.org/api/1.0/series/10612/revisions/1/mbox

Test kms_cursor_legacy:
        Subgroup basic-flip-vs-cursor-legacy:
                fail       -> PASS       (ro-hsw-i7-4770r)
                fail       -> PASS       (ro-bdw-i7-5600u)
                pass       -> FAIL       (ro-bdw-i5-5250u)
        Subgroup basic-flip-vs-cursor-varying-size:
                pass       -> FAIL       (ro-bdw-i5-5250u)

ro-bdw-i5-5250u  total:240  pass:218  dwarn:4   dfail:0   fail:2   skip:16 
ro-bdw-i7-5557U  total:240  pass:224  dwarn:0   dfail:0   fail:0   skip:16 
ro-bdw-i7-5600u  total:240  pass:207  dwarn:0   dfail:0   fail:1   skip:32 
ro-bsw-n3050     total:240  pass:193  dwarn:0   dfail:0   fail:4   skip:43 
ro-byt-n2820     total:240  pass:197  dwarn:0   dfail:0   fail:3   skip:40 
ro-hsw-i3-4010u  total:240  pass:214  dwarn:0   dfail:0   fail:0   skip:26 
ro-hsw-i7-4770r  total:240  pass:214  dwarn:0   dfail:0   fail:0   skip:26 
ro-ilk-i7-620lm  total:240  pass:173  dwarn:1   dfail:0   fail:1   skip:65 
ro-ilk1-i5-650   total:235  pass:173  dwarn:0   dfail:0   fail:2   skip:60 
ro-ivb-i7-3770   total:240  pass:205  dwarn:0   dfail:0   fail:0   skip:35 
ro-ivb2-i7-3770  total:240  pass:209  dwarn:0   dfail:0   fail:0   skip:31 
ro-skl3-i5-6260u total:240  pass:222  dwarn:0   dfail:0   fail:4   skip:14 
ro-snb-i7-2620M  total:240  pass:198  dwarn:0   dfail:0   fail:1   skip:41 

Results at /archive/results/CI_IGT_test/RO_Patchwork_1687/

4d6f224 drm-intel-nightly: 2016y-08m-03d-15h-38m-31s UTC integration manifest
7e81331 drm: Avoid printing negative values for unsigned variables.

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [Intel-gfx] [PATCH] drm: Avoid printing negative values for unsigned variables.
  2016-08-03 16:53 [PATCH] drm: Avoid printing negative values for unsigned variables Rodrigo Vivi
  2016-08-03 17:23 ` ✗ Ro.CI.BAT: failure for " Patchwork
@ 2016-08-03 19:04 ` Ville Syrjälä
  2016-08-04  8:12   ` Daniel Vetter
  1 sibling, 1 reply; 4+ messages in thread
From: Ville Syrjälä @ 2016-08-03 19:04 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-gfx, dri-devel

On Wed, Aug 03, 2016 at 09:53:46AM -0700, Rodrigo Vivi wrote:
> It was really strange to see negative vblank seqs on debug
> messages. It is rare to have that big number, but when it
> happens it is confusing and misleading.
> 
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

> ---
>  drivers/gpu/drm/drm_irq.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c
> index 77f357b..01a5079 100644
> --- a/drivers/gpu/drm/drm_irq.c
> +++ b/drivers/gpu/drm/drm_irq.c
> @@ -1295,7 +1295,7 @@ void drm_vblank_off(struct drm_device *dev, unsigned int pipe)
>  		if (e->pipe != pipe)
>  			continue;
>  		DRM_DEBUG("Sending premature vblank event on disable: "
> -			  "wanted %d, current %d\n",
> +			  "wanted %u, current %u\n",
>  			  e->event.sequence, seq);
>  		list_del(&e->base.link);
>  		drm_vblank_put(dev, pipe);
> @@ -1585,7 +1585,7 @@ static int drm_queue_vblank_event(struct drm_device *dev, unsigned int pipe,
>  
>  	seq = drm_vblank_count_and_time(dev, pipe, &now);
>  
> -	DRM_DEBUG("event on vblank count %d, current %d, crtc %u\n",
> +	DRM_DEBUG("event on vblank count %u, current %u, crtc %u\n",
>  		  vblwait->request.sequence, seq, pipe);
>  
>  	trace_drm_vblank_event_queued(current->pid, pipe,
> @@ -1693,7 +1693,7 @@ int drm_wait_vblank(struct drm_device *dev, void *data,
>  		return drm_queue_vblank_event(dev, pipe, vblwait, file_priv);
>  	}
>  
> -	DRM_DEBUG("waiting on vblank count %d, crtc %u\n",
> +	DRM_DEBUG("waiting on vblank count %u, crtc %u\n",
>  		  vblwait->request.sequence, pipe);
>  	DRM_WAIT_ON(ret, vblank->queue, 3 * HZ,
>  		    (((drm_vblank_count(dev, pipe) -
> @@ -1708,7 +1708,7 @@ int drm_wait_vblank(struct drm_device *dev, void *data,
>  		vblwait->reply.tval_sec = now.tv_sec;
>  		vblwait->reply.tval_usec = now.tv_usec;
>  
> -		DRM_DEBUG("returning %d to client\n",
> +		DRM_DEBUG("returning %u to client\n",
>  			  vblwait->reply.sequence);
>  	} else {
>  		DRM_DEBUG("vblank wait interrupted by signal\n");
> @@ -1735,7 +1735,7 @@ static void drm_handle_vblank_events(struct drm_device *dev, unsigned int pipe)
>  		if ((seq - e->event.sequence) > (1<<23))
>  			continue;
>  
> -		DRM_DEBUG("vblank event on %d, current %d\n",
> +		DRM_DEBUG("vblank event on %u, current %u\n",
>  			  e->event.sequence, seq);
>  
>  		list_del(&e->base.link);
> -- 
> 2.5.5
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrjälä
Intel OTC
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] drm: Avoid printing negative values for unsigned variables.
  2016-08-03 19:04 ` [Intel-gfx] [PATCH] " Ville Syrjälä
@ 2016-08-04  8:12   ` Daniel Vetter
  0 siblings, 0 replies; 4+ messages in thread
From: Daniel Vetter @ 2016-08-04  8:12 UTC (permalink / raw)
  To: Ville Syrjälä; +Cc: intel-gfx, dri-devel, Rodrigo Vivi

On Wed, Aug 03, 2016 at 10:04:36PM +0300, Ville Syrjälä wrote:
> On Wed, Aug 03, 2016 at 09:53:46AM -0700, Rodrigo Vivi wrote:
> > It was really strange to see negative vblank seqs on debug
> > messages. It is rare to have that big number, but when it
> > happens it is confusing and misleading.
> > 
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> 
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Put into my -misc queue, will push out once -rc1 hits.
-Daniel

> 
> > ---
> >  drivers/gpu/drm/drm_irq.c | 10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c
> > index 77f357b..01a5079 100644
> > --- a/drivers/gpu/drm/drm_irq.c
> > +++ b/drivers/gpu/drm/drm_irq.c
> > @@ -1295,7 +1295,7 @@ void drm_vblank_off(struct drm_device *dev, unsigned int pipe)
> >  		if (e->pipe != pipe)
> >  			continue;
> >  		DRM_DEBUG("Sending premature vblank event on disable: "
> > -			  "wanted %d, current %d\n",
> > +			  "wanted %u, current %u\n",
> >  			  e->event.sequence, seq);
> >  		list_del(&e->base.link);
> >  		drm_vblank_put(dev, pipe);
> > @@ -1585,7 +1585,7 @@ static int drm_queue_vblank_event(struct drm_device *dev, unsigned int pipe,
> >  
> >  	seq = drm_vblank_count_and_time(dev, pipe, &now);
> >  
> > -	DRM_DEBUG("event on vblank count %d, current %d, crtc %u\n",
> > +	DRM_DEBUG("event on vblank count %u, current %u, crtc %u\n",
> >  		  vblwait->request.sequence, seq, pipe);
> >  
> >  	trace_drm_vblank_event_queued(current->pid, pipe,
> > @@ -1693,7 +1693,7 @@ int drm_wait_vblank(struct drm_device *dev, void *data,
> >  		return drm_queue_vblank_event(dev, pipe, vblwait, file_priv);
> >  	}
> >  
> > -	DRM_DEBUG("waiting on vblank count %d, crtc %u\n",
> > +	DRM_DEBUG("waiting on vblank count %u, crtc %u\n",
> >  		  vblwait->request.sequence, pipe);
> >  	DRM_WAIT_ON(ret, vblank->queue, 3 * HZ,
> >  		    (((drm_vblank_count(dev, pipe) -
> > @@ -1708,7 +1708,7 @@ int drm_wait_vblank(struct drm_device *dev, void *data,
> >  		vblwait->reply.tval_sec = now.tv_sec;
> >  		vblwait->reply.tval_usec = now.tv_usec;
> >  
> > -		DRM_DEBUG("returning %d to client\n",
> > +		DRM_DEBUG("returning %u to client\n",
> >  			  vblwait->reply.sequence);
> >  	} else {
> >  		DRM_DEBUG("vblank wait interrupted by signal\n");
> > @@ -1735,7 +1735,7 @@ static void drm_handle_vblank_events(struct drm_device *dev, unsigned int pipe)
> >  		if ((seq - e->event.sequence) > (1<<23))
> >  			continue;
> >  
> > -		DRM_DEBUG("vblank event on %d, current %d\n",
> > +		DRM_DEBUG("vblank event on %u, current %u\n",
> >  			  e->event.sequence, seq);
> >  
> >  		list_del(&e->base.link);
> > -- 
> > 2.5.5
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
> -- 
> Ville Syrjälä
> Intel OTC
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-08-04  8:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-03 16:53 [PATCH] drm: Avoid printing negative values for unsigned variables Rodrigo Vivi
2016-08-03 17:23 ` ✗ Ro.CI.BAT: failure for " Patchwork
2016-08-03 19:04 ` [Intel-gfx] [PATCH] " Ville Syrjälä
2016-08-04  8:12   ` Daniel Vetter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.