On Tue, Feb 28, 2017 at 03:52:31PM +0100, Michal Wajdeczko wrote: > On Tue, Feb 28, 2017 at 02:21:23PM +0000, Tvrtko Ursulin wrote: > > > > On 28/02/2017 14:00, Michal Wajdeczko wrote: > > > Additionally use runtime check to catch invalid engine indices. > > > > > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > > > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > > > --- > > > drivers/gpu/drm/i915/intel_engine_cs.c | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/drivers/gpu/drm/i915/intel_engine_cs.c b/drivers/gpu/drm/i915/intel_engine_cs.c > > > index a238304..8df53ae 100644 > > > --- a/drivers/gpu/drm/i915/intel_engine_cs.c > > > +++ b/drivers/gpu/drm/i915/intel_engine_cs.c > > > @@ -89,6 +89,8 @@ intel_engine_setup(struct drm_i915_private *dev_priv, > > > const struct engine_info *info = &intel_engines[id]; > > > struct intel_engine_cs *engine; > > > > > > + BUILD_BUG_ON(ARRAY_SIZE(intel_engines) != I915_NUM_ENGINES); > > > > For some reason I feel this is too strict. ;) > > It has to be strict to be useful. But is pointless if it doesn't apply to gen+1, or tomorrow's packing of sparse engines, which is where Tvrtko is coming from. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
