Re: [Mesa-dev] [Intel-gfx] [PATCH] drm/i915/gen9: Add WaClearHIZ_WM_CHICKEN3 for bxt and glk

2018-05-13 Thread Chris Wilson
Quoting Michel Thierry (2018-05-11 16:07:55)
> On 5/11/2018 5:43 AM, Mika Kuoppala wrote:
> > Chris Wilson  writes:
> > 
> >> Quoting Mika Kuoppala (2018-05-11 10:56:49)
> >>> Michel Thierry  writes:
> >>>
>  Factor in clear values wherever required while updating destination
>  min/max.
> 
>  References: HSDES#160184
>  Signed-off-by: Michel Thierry 
>  Cc: mesa-dev@lists.freedesktop.org
>  Cc: Mika Kuoppala 
>  Cc: Oscar Mateo 
> >>>
> >>> Reviewed-by: Mika Kuoppala 
> >>
> >> Cc: stable?
> > 
> > Yes, we should.
> > 
> 
> I think so too, although stable doesn't have the workaround refactoring 
> yet, the change will be in intel_engine_cs.c instead.

Added the cc for stable, and pushed. We may need to send a fixup patch
later if we want it backported all the way.

Thanks,
-Chris
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [Intel-gfx] [PATCH] drm/i915/gen9: Add WaClearHIZ_WM_CHICKEN3 for bxt and glk

2018-05-11 Thread Michel Thierry

On 5/11/2018 5:43 AM, Mika Kuoppala wrote:

Chris Wilson  writes:


Quoting Mika Kuoppala (2018-05-11 10:56:49)

Michel Thierry  writes:


Factor in clear values wherever required while updating destination
min/max.

References: HSDES#160184
Signed-off-by: Michel Thierry 
Cc: mesa-dev@lists.freedesktop.org
Cc: Mika Kuoppala 
Cc: Oscar Mateo 


Reviewed-by: Mika Kuoppala 


Cc: stable?


Yes, we should.



I think so too, although stable doesn't have the workaround refactoring 
yet, the change will be in intel_engine_cs.c instead.



-Mika


___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [Intel-gfx] [PATCH] drm/i915/gen9: Add WaClearHIZ_WM_CHICKEN3 for bxt and glk

2018-05-11 Thread Mika Kuoppala
Chris Wilson  writes:

> Quoting Mika Kuoppala (2018-05-11 10:56:49)
>> Michel Thierry  writes:
>> 
>> > Factor in clear values wherever required while updating destination
>> > min/max.
>> >
>> > References: HSDES#160184
>> > Signed-off-by: Michel Thierry 
>> > Cc: mesa-dev@lists.freedesktop.org
>> > Cc: Mika Kuoppala 
>> > Cc: Oscar Mateo 
>> 
>> Reviewed-by: Mika Kuoppala 
>
> Cc: stable?

Yes, we should.

-Mika

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [Intel-gfx] [PATCH] drm/i915/gen9: Add WaClearHIZ_WM_CHICKEN3 for bxt and glk

2018-05-11 Thread Chris Wilson
Quoting Mika Kuoppala (2018-05-11 10:56:49)
> Michel Thierry  writes:
> 
> > Factor in clear values wherever required while updating destination
> > min/max.
> >
> > References: HSDES#160184
> > Signed-off-by: Michel Thierry 
> > Cc: mesa-dev@lists.freedesktop.org
> > Cc: Mika Kuoppala 
> > Cc: Oscar Mateo 
> 
> Reviewed-by: Mika Kuoppala 

Cc: stable?
-Chris
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev