mbox series

[v2,0/3] restore WaEnableFloatBlendOptimization

Message ID 1548983324-15344-1-git-send-email-talha.nassar@intel.com (mailing list archive)
Headers show
Series restore WaEnableFloatBlendOptimization | expand

Message

Talha Nassar Feb. 1, 2019, 1:08 a.m. UTC 
This is the v2 of my patch after taking the feed from Chris. I have also included the HSDES per Mika's suggestion.
I am attaching the two-patch series from Tvrtko as there is a dependency.
Also to note that git couldn't apply Tvrtko's first patch due to a patch by Daniele that touched the same file and was merged. I had to manually apply the patch and no code changes were necessary.

v1: enabled the wa and added the register as wo in igt test.
v2: also modified the wa mask

Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>

Talha Nassar (1):
  drm/i915/icl: restore WaEnableFloatBlendOptimization

Tvrtko Ursulin (2):
  drm/i915: Move workaround infrastructure code up
  drm/i915: Save some lines of source code in workarounds

 drivers/gpu/drm/i915/i915_reg.h          |  3 ++
 drivers/gpu/drm/i915/intel_workarounds.c | 70 +++++++++++++-------------------
 2 files changed, 31 insertions(+), 42 deletions(-)

Comments

Chris Wilson Feb. 1, 2019, 8:39 a.m. UTC | #1 
Quoting Patchwork (2019-02-01 05:05:57)
> == Series Details ==
> 
> Series: restore WaEnableFloatBlendOptimization
> URL   : https://patchwork.freedesktop.org/series/56071/
> State : success
> 
> == Summary ==
> 
> CI Bug Log - changes from CI_DRM_5521_full -> Patchwork_12113_full
> ====================================================
> 
> Summary
> -------
> 
>   **SUCCESS**
> 
>   No regressions found.

Which means either it failed completely (and had no effect) or it worked
and successfully hid itself from gem_workarounds.

Thanks for the patch, pushed!
-Chris