Re: [Intel-gfx] [PATCH i-g-t 1/1] lib/igt_gt: Add sentinel to intel_execution_engines2

2017-12-05 Thread Tvrtko Ursulin 


On 05/12/2017 10:16, Petri Latvala wrote:

The for_each_engine_class_instance macro stops at e__->name being
NULL, so add an object that is so.

Signed-off-by: Petri Latvala 
Cc: Tvrtko Ursulin 
Cc: Chris Wilson 
---
  lib/igt_gt.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/lib/igt_gt.c b/lib/igt_gt.c
index 4a8f541f..ad6e6205 100644
--- a/lib/igt_gt.c
+++ b/lib/igt_gt.c
@@ -615,6 +615,7 @@ const struct intel_execution_engine2 
intel_execution_engines2[] = {
{ "vcs0", I915_ENGINE_CLASS_VIDEO, 0 },
{ "vcs1", I915_ENGINE_CLASS_VIDEO, 1 },
{ "vecs0", I915_ENGINE_CLASS_VIDEO_ENHANCE, 0 },
+   { }
  };
  
  unsigned int




Oh dear, well spotted.

Reviewed-by: Tvrtko Ursulin 

Regards,

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

[Intel-gfx] [PATCH i-g-t 1/1] lib/igt_gt: Add sentinel to intel_execution_engines2

2017-12-05 Thread Petri Latvala 
The for_each_engine_class_instance macro stops at e__->name being
NULL, so add an object that is so.

Signed-off-by: Petri Latvala 
Cc: Tvrtko Ursulin 
Cc: Chris Wilson 
---
 lib/igt_gt.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/igt_gt.c b/lib/igt_gt.c
index 4a8f541f..ad6e6205 100644
--- a/lib/igt_gt.c
+++ b/lib/igt_gt.c
@@ -615,6 +615,7 @@ const struct intel_execution_engine2 
intel_execution_engines2[] = {
{ "vcs0", I915_ENGINE_CLASS_VIDEO, 0 },
{ "vcs1", I915_ENGINE_CLASS_VIDEO, 1 },
{ "vecs0", I915_ENGINE_CLASS_VIDEO_ENHANCE, 0 },
+   { }
 };
 
 unsigned int
-- 
2.14.1

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