public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
* [PATCH] drm/i915: Fix sink crc connector iteration
@ 2015-03-13 23:13 Rodrigo Vivi
  2015-03-14  7:08 ` shuang.he
  2015-03-16  7:19 ` Ander Conselvan De Oliveira
  0 siblings, 2 replies; 9+ messages in thread
From: Rodrigo Vivi @ 2015-03-13 23:13 UTC (permalink / raw)
  To: intel-gfx; +Cc: Ander Conselvan de Oliveira, Rodrigo Vivi

Regressed by this commit:

commit 3455454e18ca3f92c565700539e744c620d8276b
Author: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Date:   Tue Mar 3 15:21:56 2015 +0200

    drm/i915: Add a for_each_intel_connector macro

Cc: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 drivers/gpu/drm/i915/i915_debugfs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index 1f7ec06..daf948e 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -2305,7 +2305,7 @@ static int i915_sink_crc(struct seq_file *m, void *data)
 	u8 crc[6];
 
 	drm_modeset_lock_all(dev);
-	for_each_intel_encoder(dev, connector) {
+	for_each_intel_connector(dev, connector) {
 
 		if (connector->base.dpms != DRM_MODE_DPMS_ON)
 			continue;
-- 
2.1.0

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915: Fix sink crc connector iteration
  2015-03-13 23:13 [PATCH] drm/i915: Fix sink crc connector iteration Rodrigo Vivi
@ 2015-03-14  7:08 ` shuang.he
  2015-03-16  7:19 ` Ander Conselvan De Oliveira
  1 sibling, 0 replies; 9+ messages in thread
From: shuang.he @ 2015-03-14  7:08 UTC (permalink / raw)
  To: shuang.he, ethan.gao, intel-gfx, rodrigo.vivi

Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang.he@intel.com)
Task id: 5950
-------------------------------------Summary-------------------------------------
Platform          Delta          drm-intel-nightly          Series Applied
PNV                                  276/276              276/276
ILK                                  303/303              303/303
SNB                 -1              279/279              278/279
IVB                                  343/343              343/343
BYT                                  287/287              287/287
HSW                                  363/363              363/363
BDW                                  308/308              308/308
-------------------------------------Detailed-------------------------------------
Platform  Test                                drm-intel-nightly          Series Applied
*SNB  igt_gem_persistent_relocs_forked-interruptible      PASS(2)      DMESG_WARN(1)PASS(1)
Note: You need to pay more attention to line start with '*'
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915: Fix sink crc connector iteration
  2015-03-13 23:13 [PATCH] drm/i915: Fix sink crc connector iteration Rodrigo Vivi
  2015-03-14  7:08 ` shuang.he
@ 2015-03-16  7:19 ` Ander Conselvan De Oliveira
  2015-03-16  7:21   ` [PATCH] drm/i915: Make for_each_intel_* macros typesafe Ander Conselvan de Oliveira
  1 sibling, 1 reply; 9+ messages in thread
From: Ander Conselvan De Oliveira @ 2015-03-16  7:19 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-gfx

On Fri, 2015-03-13 at 16:13 -0700, Rodrigo Vivi wrote:
> Regressed by this commit:
> 
> commit 3455454e18ca3f92c565700539e744c620d8276b
> Author: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
> Date:   Tue Mar 3 15:21:56 2015 +0200
> 
>     drm/i915: Add a for_each_intel_connector macro
> 
> Cc: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>  drivers/gpu/drm/i915/i915_debugfs.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index 1f7ec06..daf948e 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -2305,7 +2305,7 @@ static int i915_sink_crc(struct seq_file *m, void *data)
>  	u8 crc[6];
>  
>  	drm_modeset_lock_all(dev);
> -	for_each_intel_encoder(dev, connector) {
> +	for_each_intel_connector(dev, connector) {
>  
>  		if (connector->base.dpms != DRM_MODE_DPMS_ON)
>  			continue;

Yeah, oops. 

Reviewed-by: Ander Conselvan de Oliveira <conselvan2@gmail.com>


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

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] drm/i915: Make for_each_intel_* macros typesafe
  2015-03-16  7:19 ` Ander Conselvan De Oliveira
@ 2015-03-16  7:21   ` Ander Conselvan de Oliveira
  2015-03-16  9:38     ` shuang.he
  2015-03-16 10:00     ` Jani Nikula
  0 siblings, 2 replies; 9+ messages in thread
From: Ander Conselvan de Oliveira @ 2015-03-16  7:21 UTC (permalink / raw)
  To: intel-gfx; +Cc: Ander Conselvan de Oliveira, rodrigo.vivi

So that the compiler warns if we get something wrong.

Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
---
 drivers/gpu/drm/i915/i915_drv.h | 39 ++++++++++++++++++++++++++-------------
 1 file changed, 26 insertions(+), 13 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index ae4ba6a..871d77a 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -232,30 +232,43 @@ enum hpd_pin {
 	     (__s) < INTEL_INFO(__dev_priv)->num_sprites[(__p)];	\
 	     (__s)++)
 
-#define for_each_crtc(dev, crtc) \
+#define type_check(variable, type)				\
+	{							\
+		typeof(variable) __dummy_ptr = NULL;		\
+		const type *__type_check = __dummy_ptr;		\
+		(void) __type_check;				\
+	}
+
+#define for_each_crtc(dev, crtc)				\
+	type_check(crtc, struct drm_crtc)			\
 	list_for_each_entry(crtc, &dev->mode_config.crtc_list, head)
 
-#define for_each_intel_crtc(dev, intel_crtc) \
-	list_for_each_entry(intel_crtc, &dev->mode_config.crtc_list, base.head)
+#define for_each_intel_crtc(dev, crtc)				\
+	type_check(crtc, struct intel_crtc)			\
+	list_for_each_entry(crtc, &dev->mode_config.crtc_list, base.head)
 
-#define for_each_intel_encoder(dev, intel_encoder)		\
-	list_for_each_entry(intel_encoder,			\
+#define for_each_intel_encoder(dev, encoder)			\
+	type_check(encoder, struct intel_encoder)		\
+	list_for_each_entry(encoder,				\
 			    &(dev)->mode_config.encoder_list,	\
 			    base.head)
 
-#define for_each_intel_connector(dev, intel_connector)		\
-	list_for_each_entry(intel_connector,			\
+#define for_each_intel_connector(dev, connector)		\
+	type_check(connector, struct intel_connector);		\
+	list_for_each_entry(connector,				\
 			    &dev->mode_config.connector_list,	\
 			    base.head)
 
 
-#define for_each_encoder_on_crtc(dev, __crtc, intel_encoder) \
-	list_for_each_entry((intel_encoder), &(dev)->mode_config.encoder_list, base.head) \
-		if ((intel_encoder)->base.crtc == (__crtc))
+#define for_each_encoder_on_crtc(dev, __crtc, encoder)		\
+	type_check(encoder, struct intel_encoder)		\
+	list_for_each_entry((encoder), &(dev)->mode_config.encoder_list, base.head) \
+		if ((encoder)->base.crtc == (__crtc))
 
-#define for_each_connector_on_encoder(dev, __encoder, intel_connector) \
-	list_for_each_entry((intel_connector), &(dev)->mode_config.connector_list, base.head) \
-		if ((intel_connector)->base.encoder == (__encoder))
+#define for_each_connector_on_encoder(dev, __encoder, connector) \
+	type_check(connector, struct intel_connector)		\
+	list_for_each_entry((connector), &(dev)->mode_config.connector_list, base.head) \
+		if ((connector)->base.encoder == (__encoder))
 
 #define for_each_power_domain(domain, mask)				\
 	for ((domain) = 0; (domain) < POWER_DOMAIN_NUM; (domain)++)	\
-- 
2.1.0

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915: Make for_each_intel_* macros typesafe
  2015-03-16  7:21   ` [PATCH] drm/i915: Make for_each_intel_* macros typesafe Ander Conselvan de Oliveira
@ 2015-03-16  9:38     ` shuang.he
  2015-03-16 10:00     ` Jani Nikula
  1 sibling, 0 replies; 9+ messages in thread
From: shuang.he @ 2015-03-16  9:38 UTC (permalink / raw)
  To: shuang.he, ethan.gao, intel-gfx, ander.conselvan.de.oliveira

Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang.he@intel.com)
Task id: 5954
-------------------------------------Summary-------------------------------------
Platform          Delta          drm-intel-nightly          Series Applied
PNV                                  268/268              268/268
ILK                 -1              303/303              302/303
SNB                                  283/283              283/283
IVB                                  342/342              342/342
BYT                                  287/287              287/287
HSW                 -1              363/363              362/363
BDW                                  308/308              308/308
-------------------------------------Detailed-------------------------------------
Platform  Test                                drm-intel-nightly          Series Applied
*ILK  igt_gem_fenced_exec_thrash_no-spare-fences-busy      PASS(2)      DMESG_WARN(1)PASS(1)
*HSW  igt_gem_storedw_loop_vebox      PASS(2)      DMESG_WARN(1)PASS(1)
Note: You need to pay more attention to line start with '*'
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915: Make for_each_intel_* macros typesafe
  2015-03-16  7:21   ` [PATCH] drm/i915: Make for_each_intel_* macros typesafe Ander Conselvan de Oliveira
  2015-03-16  9:38     ` shuang.he
@ 2015-03-16 10:00     ` Jani Nikula
  2015-03-16 10:42       ` Chris Wilson
  1 sibling, 1 reply; 9+ messages in thread
From: Jani Nikula @ 2015-03-16 10:00 UTC (permalink / raw)
  To: intel-gfx; +Cc: Ander Conselvan de Oliveira, rodrigo.vivi

On Mon, 16 Mar 2015, Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com> wrote:
> So that the compiler warns if we get something wrong.
>
> Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
> ---
>  drivers/gpu/drm/i915/i915_drv.h | 39 ++++++++++++++++++++++++++-------------
>  1 file changed, 26 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index ae4ba6a..871d77a 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -232,30 +232,43 @@ enum hpd_pin {
>  	     (__s) < INTEL_INFO(__dev_priv)->num_sprites[(__p)];	\
>  	     (__s)++)
>  
> -#define for_each_crtc(dev, crtc) \
> +#define type_check(variable, type)				\
> +	{							\
> +		typeof(variable) __dummy_ptr = NULL;		\
> +		const type *__type_check = __dummy_ptr;		\
> +		(void) __type_check;				\
> +	}
> +
> +#define for_each_crtc(dev, crtc)				\
> +	type_check(crtc, struct drm_crtc)			\
>  	list_for_each_entry(crtc, &dev->mode_config.crtc_list, head)

if (foo)
	for_each_crtc(dev, crtc);

will do type_check if foo, but list_for_each_entry unconditionally.

I think this is too fragile.

I'm already pretty anxious about our loop macros with an if in them
blowing up due to the dangling else:

	if (foo)
        	list_for_each_encoder_on_crtc(dev, crtc, intel_encoder)
                	bar();
	else
        	baz();

where baz() gets called for each encoder *not* on crtc, instead of once
for !foo.


BR,
Jani.



>  
> -#define for_each_intel_crtc(dev, intel_crtc) \
> -	list_for_each_entry(intel_crtc, &dev->mode_config.crtc_list, base.head)
> +#define for_each_intel_crtc(dev, crtc)				\
> +	type_check(crtc, struct intel_crtc)			\
> +	list_for_each_entry(crtc, &dev->mode_config.crtc_list, base.head)
>  
> -#define for_each_intel_encoder(dev, intel_encoder)		\
> -	list_for_each_entry(intel_encoder,			\
> +#define for_each_intel_encoder(dev, encoder)			\
> +	type_check(encoder, struct intel_encoder)		\
> +	list_for_each_entry(encoder,				\
>  			    &(dev)->mode_config.encoder_list,	\
>  			    base.head)
>  
> -#define for_each_intel_connector(dev, intel_connector)		\
> -	list_for_each_entry(intel_connector,			\
> +#define for_each_intel_connector(dev, connector)		\
> +	type_check(connector, struct intel_connector);		\
> +	list_for_each_entry(connector,				\
>  			    &dev->mode_config.connector_list,	\
>  			    base.head)
>  
>  
> -#define for_each_encoder_on_crtc(dev, __crtc, intel_encoder) \
> -	list_for_each_entry((intel_encoder), &(dev)->mode_config.encoder_list, base.head) \
> -		if ((intel_encoder)->base.crtc == (__crtc))
> +#define for_each_encoder_on_crtc(dev, __crtc, encoder)		\
> +	type_check(encoder, struct intel_encoder)		\
> +	list_for_each_entry((encoder), &(dev)->mode_config.encoder_list, base.head) \
> +		if ((encoder)->base.crtc == (__crtc))
>  
> -#define for_each_connector_on_encoder(dev, __encoder, intel_connector) \
> -	list_for_each_entry((intel_connector), &(dev)->mode_config.connector_list, base.head) \
> -		if ((intel_connector)->base.encoder == (__encoder))
> +#define for_each_connector_on_encoder(dev, __encoder, connector) \
> +	type_check(connector, struct intel_connector)		\
> +	list_for_each_entry((connector), &(dev)->mode_config.connector_list, base.head) \
> +		if ((connector)->base.encoder == (__encoder))
>  
>  #define for_each_power_domain(domain, mask)				\
>  	for ((domain) = 0; (domain) < POWER_DOMAIN_NUM; (domain)++)	\
> -- 
> 2.1.0
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915: Make for_each_intel_* macros typesafe
  2015-03-16 10:00     ` Jani Nikula
@ 2015-03-16 10:42       ` Chris Wilson
  2015-03-16 12:51         ` Ander Conselvan de Oliveira
  0 siblings, 1 reply; 9+ messages in thread
From: Chris Wilson @ 2015-03-16 10:42 UTC (permalink / raw)
  To: Jani Nikula; +Cc: Ander Conselvan de Oliveira, intel-gfx, rodrigo.vivi

On Mon, Mar 16, 2015 at 12:00:35PM +0200, Jani Nikula wrote:
> On Mon, 16 Mar 2015, Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com> wrote:
> > So that the compiler warns if we get something wrong.
> >
> > Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
> > ---
> >  drivers/gpu/drm/i915/i915_drv.h | 39 ++++++++++++++++++++++++++-------------
> >  1 file changed, 26 insertions(+), 13 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> > index ae4ba6a..871d77a 100644
> > --- a/drivers/gpu/drm/i915/i915_drv.h
> > +++ b/drivers/gpu/drm/i915/i915_drv.h
> > @@ -232,30 +232,43 @@ enum hpd_pin {
> >  	     (__s) < INTEL_INFO(__dev_priv)->num_sprites[(__p)];	\
> >  	     (__s)++)
> >  
> > -#define for_each_crtc(dev, crtc) \
> > +#define type_check(variable, type)				\
> > +	{							\
> > +		typeof(variable) __dummy_ptr = NULL;		\
> > +		const type *__type_check = __dummy_ptr;		\
> > +		(void) __type_check;				\
> > +	}
> > +
> > +#define for_each_crtc(dev, crtc)				\
> > +	type_check(crtc, struct drm_crtc)			\
> >  	list_for_each_entry(crtc, &dev->mode_config.crtc_list, head)

#define __list_for_each_entry_type(pos, head, type, member) \
	for (pos = list_first_entry(head, type, member); \
	     &pos->member != (head); \
	     pos = list_next_entry(pos, member))

#define for_each_crtc(dev, crtc) \
	__list_for_each_entry_type(crtc, \
				   &dev->mode_config.crtc_list, \
				   struct drm_crtc, \
				   head)
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] drm/i915: Make for_each_intel_* macros typesafe
  2015-03-16 10:42       ` Chris Wilson
@ 2015-03-16 12:51         ` Ander Conselvan de Oliveira
  2015-03-16 17:24           ` shuang.he
  0 siblings, 1 reply; 9+ messages in thread
From: Ander Conselvan de Oliveira @ 2015-03-16 12:51 UTC (permalink / raw)
  To: intel-gfx; +Cc: Ander Conselvan de Oliveira, rodrigo.vivi

So that the compiler warns if we get something wrong.

v2: Don't break 'if (foo) for_each_*' usage. (Jani & Chris)
Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
---
 drivers/gpu/drm/i915/i915_drv.h | 63 ++++++++++++++++++++++++++---------------
 1 file changed, 40 insertions(+), 23 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index ae4ba6a..55fc10d 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -232,30 +232,47 @@ enum hpd_pin {
 	     (__s) < INTEL_INFO(__dev_priv)->num_sprites[(__p)];	\
 	     (__s)++)
 
-#define for_each_crtc(dev, crtc) \
-	list_for_each_entry(crtc, &dev->mode_config.crtc_list, head)
-
-#define for_each_intel_crtc(dev, intel_crtc) \
-	list_for_each_entry(intel_crtc, &dev->mode_config.crtc_list, base.head)
-
-#define for_each_intel_encoder(dev, intel_encoder)		\
-	list_for_each_entry(intel_encoder,			\
-			    &(dev)->mode_config.encoder_list,	\
-			    base.head)
-
-#define for_each_intel_connector(dev, intel_connector)		\
-	list_for_each_entry(intel_connector,			\
-			    &dev->mode_config.connector_list,	\
-			    base.head)
+#define __list_for_each_entry_type(pos, head, type, member) \
+	for (pos = list_first_entry(head, type, member); \
+	     &pos->member != (head); \
+	     pos = list_next_entry(pos, member))
 
-
-#define for_each_encoder_on_crtc(dev, __crtc, intel_encoder) \
-	list_for_each_entry((intel_encoder), &(dev)->mode_config.encoder_list, base.head) \
-		if ((intel_encoder)->base.crtc == (__crtc))
-
-#define for_each_connector_on_encoder(dev, __encoder, intel_connector) \
-	list_for_each_entry((intel_connector), &(dev)->mode_config.connector_list, base.head) \
-		if ((intel_connector)->base.encoder == (__encoder))
+#define for_each_crtc(dev, crtc) \
+	__list_for_each_entry_type(crtc, \
+				   &dev->mode_config.crtc_list, \
+				   struct drm_crtc, \
+				   head)
+
+#define for_each_intel_crtc(dev, crtc)				\
+	__list_for_each_entry_type(crtc,			\
+				   &dev->mode_config.crtc_list,	\
+				   struct intel_crtc, base.head)
+
+#define for_each_intel_encoder(dev, encoder)			\
+	__list_for_each_entry_type(encoder,			\
+				   &(dev)->mode_config.encoder_list, \
+				   struct intel_encoder,	\
+				   base.head)
+
+#define for_each_intel_connector(dev, connector)		\
+	__list_for_each_entry_type(connector,			\
+				   &dev->mode_config.connector_list, \
+				   struct intel_connector,	\
+				   base.head)
+
+
+#define for_each_encoder_on_crtc(dev, __crtc, encoder)		\
+	__list_for_each_entry_type((encoder),			\
+				   &(dev)->mode_config.encoder_list,\
+				   struct intel_encoder, base.head) \
+		if ((encoder)->base.crtc == (__crtc))
+
+#define for_each_connector_on_encoder(dev, __encoder, connector) \
+	__list_for_each_entry_type((connector),			\
+				    &(dev)->mode_config.connector_list, \
+				    struct intel_connector,	\
+				    base.head) \
+		if ((connector)->base.encoder == (__encoder))
 
 #define for_each_power_domain(domain, mask)				\
 	for ((domain) = 0; (domain) < POWER_DOMAIN_NUM; (domain)++)	\
-- 
2.1.0

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915: Make for_each_intel_* macros typesafe
  2015-03-16 12:51         ` Ander Conselvan de Oliveira
@ 2015-03-16 17:24           ` shuang.he
  0 siblings, 0 replies; 9+ messages in thread
From: shuang.he @ 2015-03-16 17:24 UTC (permalink / raw)
  To: shuang.he, ethan.gao, intel-gfx, ander.conselvan.de.oliveira

Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang.he@intel.com)
Task id: 5958
-------------------------------------Summary-------------------------------------
Platform          Delta          drm-intel-nightly          Series Applied
PNV                 -2              275/275              273/275
ILK                 -23              303/303              280/303
SNB                 -2              279/279              277/279
IVB                 -2              343/343              341/343
BYT                 -2              287/287              285/287
HSW                 -1              361/361              360/361
BDW                 -2              308/308              306/308
-------------------------------------Detailed-------------------------------------
Platform  Test                                drm-intel-nightly          Series Applied
*PNV  igt_drv_debugfs_reader      PASS(2)      DMESG_WARN(2)
*PNV  igt_drv_hangman_error-state-sysfs-entry      PASS(2)      TIMEOUT(2)
*ILK  igt_drv_debugfs_reader      PASS(1)      DMESG_WARN(1)
*ILK  igt_drv_hangman_error-state-sysfs-entry      PASS(1)      TIMEOUT(1)
*ILK  igt_drv_suspend_debugfs-reader      PASS(1)      DMESG_WARN(1)
*ILK  igt_gem_partial_pwrite_pread_write-snoop      PASS(1)      INIT(1)
*ILK  igt_gem_partial_pwrite_pread_write-uncached      PASS(1)      INIT(1)
*ILK  igt_gem_pwrite_pread_display-pwrite-blt-gtt_mmap-performance      PASS(1)      INIT(1)
*ILK  igt_gem_pwrite_pread_uncached-pwrite-blt-gtt_mmap-performance      PASS(1)      INIT(1)
*ILK  igt_gem_ringfill_blitter      PASS(1)      INIT(1)
*ILK  igt_gem_ringfill_blitter-interruptible      PASS(1)      INIT(1)
*ILK  igt_gem_tiled_pread_pwrite      PASS(1)      INIT(1)
*ILK  igt_gem_userptr_blits_forked-sync-mempressure-interruptible      PASS(1)      INIT(1)
*ILK  igt_gem_userptr_blits_forked-sync-mempressure-normal      PASS(1)      INIT(1)
*ILK  igt_gem_userptr_blits_forked-sync-multifd-mempressure-interruptible      PASS(1)      INIT(1)
*ILK  igt_gem_userptr_blits_forked-sync-multifd-mempressure-normal      PASS(1)      INIT(1)
*ILK  igt_gem_userptr_blits_forked-unsync-mempressure-interruptible      PASS(1)      INIT(1)
*ILK  igt_gem_userptr_blits_forked-unsync-mempressure-normal      PASS(1)      INIT(1)
*ILK  igt_gem_userptr_blits_forked-unsync-multifd-mempressure-interruptible      PASS(1)      INIT(1)
*ILK  igt_gem_userptr_blits_forked-unsync-multifd-mempressure-normal      PASS(1)      INIT(1)
*ILK  igt_kms_flip_flip-vs-dpms-interruptible      PASS(1)      INIT(1)
*ILK  igt_kms_flip_flip-vs-rmfb-interruptible      PASS(1)      INIT(1)
*ILK  igt_kms_flip_rcs-wf_vblank-vs-dpms-interruptible      PASS(1)      INIT(1)
*ILK  igt_kms_flip_vblank-vs-hang      PASS(1)      INIT(1)
*ILK  igt_kms_flip_wf_vblank-vs-modeset-interruptible      PASS(1)      NO_RESULT(1)
*SNB  igt_drv_debugfs_reader      PASS(2)      DMESG_WARN(2)
*SNB  igt_drv_hangman_error-state-sysfs-entry      PASS(2)      TIMEOUT(2)
*IVB  igt_drv_debugfs_reader      PASS(2)      DMESG_WARN(2)
*IVB  igt_drv_hangman_error-state-sysfs-entry      PASS(2)      TIMEOUT(2)
*BYT  igt_drv_debugfs_reader      PASS(2)      DMESG_WARN(2)
*BYT  igt_gem_bad_length      PASS(2)      TIMEOUT(2)
*HSW  igt_pm_rpm_debugfs-read      PASS(2)      TIMEOUT(2)
*BDW  igt_drv_debugfs_reader      PASS(2)      DMESG_WARN(2)
*BDW  igt_drv_hangman_error-state-sysfs-entry      PASS(2)      TIMEOUT(2)
Note: You need to pay more attention to line start with '*'
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2015-03-16 17:25 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-13 23:13 [PATCH] drm/i915: Fix sink crc connector iteration Rodrigo Vivi
2015-03-14  7:08 ` shuang.he
2015-03-16  7:19 ` Ander Conselvan De Oliveira
2015-03-16  7:21   ` [PATCH] drm/i915: Make for_each_intel_* macros typesafe Ander Conselvan de Oliveira
2015-03-16  9:38     ` shuang.he
2015-03-16 10:00     ` Jani Nikula
2015-03-16 10:42       ` Chris Wilson
2015-03-16 12:51         ` Ander Conselvan de Oliveira
2015-03-16 17:24           ` shuang.he

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox