* [PATCH] drm/i915: fix broadwell EU computation
@ 2019-08-28 21:02 Rodrigo Vivi
2019-08-28 21:22 ` Lionel Landwerlin
2019-08-28 21:40 ` ✗ Fi.CI.BAT: failure for drm/i915: fix broadwell EU computation (rev2) Patchwork
0 siblings, 2 replies; 10+ messages in thread
From: Rodrigo Vivi @ 2019-08-28 21:02 UTC (permalink / raw)
To: stable
Cc: intel-gfx, dmitry.v.rogozhkin, lionel.g.landwerlin, Chris Wilson,
Tvrtko Ursulin, Joonas Lahtinen, Rodrigo Vivi
From: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
commit 6a67a20366f894c172734f28c5646bdbe48a46e3 upstream.
subslice_mask is an array indexed by slice, not subslice.
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181112123931.2815-1-lionel.g.landwerlin@intel.com
(cherry picked from commit 63ac3328f0d1d37f286e397b14d9596ed09d7ca5)
Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: <stable@vger.kernel.org> # 4.17
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
drivers/gpu/drm/i915/intel_device_info.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
index 0ef0c6448d53..01fa98299bae 100644
--- a/drivers/gpu/drm/i915/intel_device_info.c
+++ b/drivers/gpu/drm/i915/intel_device_info.c
@@ -474,7 +474,7 @@ static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
u8 eu_disabled_mask;
u32 n_disabled;
- if (!(sseu->subslice_mask[ss] & BIT(ss)))
+ if (!(sseu->subslice_mask[s] & BIT(ss)))
/* skip disabled subslice */
continue;
--
2.20.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/i915: fix broadwell EU computation
2019-08-28 21:02 [PATCH] drm/i915: fix broadwell EU computation Rodrigo Vivi
@ 2019-08-28 21:22 ` Lionel Landwerlin
2019-08-28 21:40 ` ✗ Fi.CI.BAT: failure for drm/i915: fix broadwell EU computation (rev2) Patchwork
1 sibling, 0 replies; 10+ messages in thread
From: Lionel Landwerlin @ 2019-08-28 21:22 UTC (permalink / raw)
To: Rodrigo Vivi, stable
Cc: intel-gfx, dmitry.v.rogozhkin, Chris Wilson, Tvrtko Ursulin,
Joonas Lahtinen
On 29/08/2019 00:02, Rodrigo Vivi wrote:
> From: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>
> commit 6a67a20366f894c172734f28c5646bdbe48a46e3 upstream.
>
> subslice_mask is an array indexed by slice, not subslice.
>
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
> Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> Link: https://patchwork.freedesktop.org/patch/msgid/20181112123931.2815-1-lionel.g.landwerlin@intel.com
> (cherry picked from commit 63ac3328f0d1d37f286e397b14d9596ed09d7ca5)
> Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Cc: <stable@vger.kernel.org> # 4.17
Rodrigo pointed out I forgot the Cc: tag which is why this didn't make
it to stable.
The same bug showed up on a CentOS kernel :
https://github.com/intel/compute-runtime/issues/200
My bad... :(
Thanks for resending!
-Lionel
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
> drivers/gpu/drm/i915/intel_device_info.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
> index 0ef0c6448d53..01fa98299bae 100644
> --- a/drivers/gpu/drm/i915/intel_device_info.c
> +++ b/drivers/gpu/drm/i915/intel_device_info.c
> @@ -474,7 +474,7 @@ static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
> u8 eu_disabled_mask;
> u32 n_disabled;
>
> - if (!(sseu->subslice_mask[ss] & BIT(ss)))
> + if (!(sseu->subslice_mask[s] & BIT(ss)))
> /* skip disabled subslice */
> continue;
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/i915: fix broadwell EU computation
@ 2019-08-28 21:22 ` Lionel Landwerlin
0 siblings, 0 replies; 10+ messages in thread
From: Lionel Landwerlin @ 2019-08-28 21:22 UTC (permalink / raw)
To: Rodrigo Vivi, stable; +Cc: intel-gfx
On 29/08/2019 00:02, Rodrigo Vivi wrote:
> From: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>
> commit 6a67a20366f894c172734f28c5646bdbe48a46e3 upstream.
>
> subslice_mask is an array indexed by slice, not subslice.
>
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
> Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> Link: https://patchwork.freedesktop.org/patch/msgid/20181112123931.2815-1-lionel.g.landwerlin@intel.com
> (cherry picked from commit 63ac3328f0d1d37f286e397b14d9596ed09d7ca5)
> Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Cc: <stable@vger.kernel.org> # 4.17
Rodrigo pointed out I forgot the Cc: tag which is why this didn't make
it to stable.
The same bug showed up on a CentOS kernel :
https://github.com/intel/compute-runtime/issues/200
My bad... :(
Thanks for resending!
-Lionel
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
> drivers/gpu/drm/i915/intel_device_info.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
> index 0ef0c6448d53..01fa98299bae 100644
> --- a/drivers/gpu/drm/i915/intel_device_info.c
> +++ b/drivers/gpu/drm/i915/intel_device_info.c
> @@ -474,7 +474,7 @@ static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
> u8 eu_disabled_mask;
> u32 n_disabled;
>
> - if (!(sseu->subslice_mask[ss] & BIT(ss)))
> + if (!(sseu->subslice_mask[s] & BIT(ss)))
> /* skip disabled subslice */
> continue;
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/i915: fix broadwell EU computation
2019-08-28 21:22 ` Lionel Landwerlin
@ 2019-08-29 21:02 ` Sasha Levin
-1 siblings, 0 replies; 10+ messages in thread
From: Sasha Levin @ 2019-08-29 21:02 UTC (permalink / raw)
To: Lionel Landwerlin
Cc: Rodrigo Vivi, stable, intel-gfx, dmitry.v.rogozhkin,
Chris Wilson, Tvrtko Ursulin, Joonas Lahtinen
On Thu, Aug 29, 2019 at 12:22:27AM +0300, Lionel Landwerlin wrote:
>On 29/08/2019 00:02, Rodrigo Vivi wrote:
>>From: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>>
>>commit 6a67a20366f894c172734f28c5646bdbe48a46e3 upstream.
>>
>>subslice_mask is an array indexed by slice, not subslice.
>>
>>Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>>Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
>>Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
>>Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
>>Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>Link: https://patchwork.freedesktop.org/patch/msgid/20181112123931.2815-1-lionel.g.landwerlin@intel.com
>>(cherry picked from commit 63ac3328f0d1d37f286e397b14d9596ed09d7ca5)
>>Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
>>Cc: <stable@vger.kernel.org> # 4.17
>
>
>Rodrigo pointed out I forgot the Cc: tag which is why this didn't make
>it to stable.
>
>The same bug showed up on a CentOS kernel :
>https://github.com/intel/compute-runtime/issues/200
I just queued the upstream patch for 4.19, thanks!
--
Thanks,
Sasha
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/i915: fix broadwell EU computation
@ 2019-08-29 21:02 ` Sasha Levin
0 siblings, 0 replies; 10+ messages in thread
From: Sasha Levin @ 2019-08-29 21:02 UTC (permalink / raw)
To: Lionel Landwerlin
Cc: Rodrigo Vivi, stable, intel-gfx, dmitry.v.rogozhkin,
Chris Wilson, Tvrtko Ursulin, Joonas Lahtinen
On Thu, Aug 29, 2019 at 12:22:27AM +0300, Lionel Landwerlin wrote:
>On 29/08/2019 00:02, Rodrigo Vivi wrote:
>>From: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>>
>>commit 6a67a20366f894c172734f28c5646bdbe48a46e3 upstream.
>>
>>subslice_mask is an array indexed by slice, not subslice.
>>
>>Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>>Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
>>Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
>>Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
>>Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>Link: https://patchwork.freedesktop.org/patch/msgid/20181112123931.2815-1-lionel.g.landwerlin@intel.com
>>(cherry picked from commit 63ac3328f0d1d37f286e397b14d9596ed09d7ca5)
>>Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
>>Cc: <stable@vger.kernel.org> # 4.17
>
>
>Rodrigo pointed out I forgot the Cc: tag which is why this didn't make
>it to stable.
>
>The same bug showed up on a CentOS kernel :
>https://github.com/intel/compute-runtime/issues/200
I just queued the upstream patch for 4.19, thanks!
^ permalink raw reply [flat|nested] 10+ messages in thread
* ✗ Fi.CI.BAT: failure for drm/i915: fix broadwell EU computation (rev2)
2019-08-28 21:02 [PATCH] drm/i915: fix broadwell EU computation Rodrigo Vivi
2019-08-28 21:22 ` Lionel Landwerlin
@ 2019-08-28 21:40 ` Patchwork
1 sibling, 0 replies; 10+ messages in thread
From: Patchwork @ 2019-08-28 21:40 UTC (permalink / raw)
To: Lionel Landwerlin; +Cc: intel-gfx
== Series Details ==
Series: drm/i915: fix broadwell EU computation (rev2)
URL : https://patchwork.freedesktop.org/series/52355/
State : failure
== Summary ==
Applying: drm/i915: fix broadwell EU computation
Using index info to reconstruct a base tree...
M drivers/gpu/drm/i915/intel_device_info.c
Falling back to patching base and 3-way merge...
Auto-merging drivers/gpu/drm/i915/intel_device_info.c
CONFLICT (content): Merge conflict in drivers/gpu/drm/i915/intel_device_info.c
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch' to see the failed patch
Patch failed at 0001 drm/i915: fix broadwell EU computation
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] drm/i915: fix broadwell EU computation
@ 2018-11-12 12:39 Lionel Landwerlin
2018-11-12 15:44 ` Chris Wilson
2018-11-12 15:57 ` Tvrtko Ursulin
0 siblings, 2 replies; 10+ messages in thread
From: Lionel Landwerlin @ 2018-11-12 12:39 UTC (permalink / raw)
To: intel-gfx
subslice_mask is an array indexed by slice, not subslice.
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
---
drivers/gpu/drm/i915/intel_device_info.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
index 89ed3a84a4fa..ceecb5bd5226 100644
--- a/drivers/gpu/drm/i915/intel_device_info.c
+++ b/drivers/gpu/drm/i915/intel_device_info.c
@@ -474,7 +474,7 @@ static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
u8 eu_disabled_mask;
u32 n_disabled;
- if (!(sseu->subslice_mask[ss] & BIT(ss)))
+ if (!(sseu->subslice_mask[s] & BIT(ss)))
/* skip disabled subslice */
continue;
--
2.19.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/i915: fix broadwell EU computation
2018-11-12 12:39 [PATCH] drm/i915: fix broadwell EU computation Lionel Landwerlin
@ 2018-11-12 15:44 ` Chris Wilson
2018-11-12 15:57 ` Tvrtko Ursulin
1 sibling, 0 replies; 10+ messages in thread
From: Chris Wilson @ 2018-11-12 15:44 UTC (permalink / raw)
To: Lionel Landwerlin, intel-gfx
Quoting Lionel Landwerlin (2018-11-12 12:39:31)
> subslice_mask is an array indexed by slice, not subslice.
>
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
> Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
> ---
> drivers/gpu/drm/i915/intel_device_info.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
> index 89ed3a84a4fa..ceecb5bd5226 100644
> --- a/drivers/gpu/drm/i915/intel_device_info.c
> +++ b/drivers/gpu/drm/i915/intel_device_info.c
> @@ -474,7 +474,7 @@ static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
> u8 eu_disabled_mask;
> u32 n_disabled;
>
> - if (!(sseu->subslice_mask[ss] & BIT(ss)))
> + if (!(sseu->subslice_mask[s] & BIT(ss)))
Tallies up with the other users.
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/i915: fix broadwell EU computation
2018-11-12 12:39 [PATCH] drm/i915: fix broadwell EU computation Lionel Landwerlin
2018-11-12 15:44 ` Chris Wilson
@ 2018-11-12 15:57 ` Tvrtko Ursulin
2018-11-12 17:42 ` Lionel Landwerlin
1 sibling, 1 reply; 10+ messages in thread
From: Tvrtko Ursulin @ 2018-11-12 15:57 UTC (permalink / raw)
To: Lionel Landwerlin, intel-gfx
On 12/11/2018 12:39, Lionel Landwerlin wrote:
> subslice_mask is an array indexed by slice, not subslice.
>
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
> Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
> ---
> drivers/gpu/drm/i915/intel_device_info.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_device_info.c b/drivers/gpu/drm/i915/intel_device_info.c
> index 89ed3a84a4fa..ceecb5bd5226 100644
> --- a/drivers/gpu/drm/i915/intel_device_info.c
> +++ b/drivers/gpu/drm/i915/intel_device_info.c
> @@ -474,7 +474,7 @@ static void broadwell_sseu_info_init(struct drm_i915_private *dev_priv)
> u8 eu_disabled_mask;
> u32 n_disabled;
>
> - if (!(sseu->subslice_mask[ss] & BIT(ss)))
> + if (!(sseu->subslice_mask[s] & BIT(ss)))
> /* skip disabled subslice */
> continue;
>
>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/i915: fix broadwell EU computation
2018-11-12 15:57 ` Tvrtko Ursulin
@ 2018-11-12 17:42 ` Lionel Landwerlin
0 siblings, 0 replies; 10+ messages in thread
From: Lionel Landwerlin @ 2018-11-12 17:42 UTC (permalink / raw)
To: Tvrtko Ursulin, intel-gfx
On 12/11/2018 15:57, Tvrtko Ursulin wrote:
>
> On 12/11/2018 12:39, Lionel Landwerlin wrote:
>> subslice_mask is an array indexed by slice, not subslice.
>>
>> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>> Fixes: 8cc7669355136f ("drm/i915: store all subslice masks")
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108712
>> ---
>> drivers/gpu/drm/i915/intel_device_info.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_device_info.c
>> b/drivers/gpu/drm/i915/intel_device_info.c
>> index 89ed3a84a4fa..ceecb5bd5226 100644
>> --- a/drivers/gpu/drm/i915/intel_device_info.c
>> +++ b/drivers/gpu/drm/i915/intel_device_info.c
>> @@ -474,7 +474,7 @@ static void broadwell_sseu_info_init(struct
>> drm_i915_private *dev_priv)
>> u8 eu_disabled_mask;
>> u32 n_disabled;
>> - if (!(sseu->subslice_mask[ss] & BIT(ss)))
>> + if (!(sseu->subslice_mask[s] & BIT(ss)))
>> /* skip disabled subslice */
>> continue;
>>
> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>
> Regards,
>
> Tvrtko
>
Thanks all, pushed.
-
Lionel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2019-08-29 21:02 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-28 21:02 [PATCH] drm/i915: fix broadwell EU computation Rodrigo Vivi
2019-08-28 21:22 ` Lionel Landwerlin
2019-08-28 21:22 ` Lionel Landwerlin
2019-08-29 21:02 ` Sasha Levin
2019-08-29 21:02 ` Sasha Levin
2019-08-28 21:40 ` ✗ Fi.CI.BAT: failure for drm/i915: fix broadwell EU computation (rev2) Patchwork
-- strict thread matches above, loose matches on Subject: below --
2018-11-12 12:39 [PATCH] drm/i915: fix broadwell EU computation Lionel Landwerlin
2018-11-12 15:44 ` Chris Wilson
2018-11-12 15:57 ` Tvrtko Ursulin
2018-11-12 17:42 ` Lionel Landwerlin
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.