drm/i915/guc: Fix doorbell id selection
authorMichel Thierry <michel.thierry@intel.com>
Wed, 31 May 2017 00:05:46 +0000 (17:05 -0700)
committerJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
Wed, 31 May 2017 07:34:03 +0000 (10:34 +0300)
commit908d238a9e78130dfbdc494fa232b2b56ac348dc
tree9cf32486ec656fd14eab9dedfbf120d28fc3c94b
parent4841be5fe68760ec7fedae0b6284787119c5e3b9
drm/i915/guc: Fix doorbell id selection

We are passing parameters in the wrong order to find next zero bit, and
when it doesn't find anything it returns size (offset in the code), which
is always zero.

For reference the function is defined as:
find_next_bit( *addr, size, offset )

The incorrect parameter order was added by commit 71e8142cf5471
("drm/i915/guc: Sanitize GuC client initialization"). Luckily, currently
we only use a single guc client and a single doorbell, which happens to be
zero; therefore it isn't necessary to backport this fix (which would be for
v4.12).

Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Oscar Mateo <oscar.mateo@intel.com>
Signed-off-by: Michel Thierry <michel.thierry@intel.com>
Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170531000546.30762-1-michel.thierry@intel.com
drivers/gpu/drm/i915/i915_guc_submission.c