Intel-GFX Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
To: Anuj Phogat <anuj.phogat@intel.com>,
	Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
	Intel-gfx@lists.freedesktop.org,
	Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Cc: kevin.ma@intel.com, xiaogang.li@intel.com
Subject: Re: [PATCH v2] drm/i915/icl: Whitelist GEN9_SLICE_COMMON_ECO_CHICKEN1
Date: Mon, 29 Apr 2019 06:57:42 +0100	[thread overview]
Message-ID: <836034a4-4060-9f40-3a63-729e7b1d4a72@linux.intel.com> (raw)
In-Reply-To: <89f0b59e-29e3-1259-2549-f2daa0b4c00c@intel.com>


On 26/04/2019 17:58, Anuj Phogat wrote:
> 
> Joonas,
> 
> Mesa now applies this WA on ICL and we're not seeing any regressions in CI.
> I tested Mesa with and without this patch applied to kernel. I don't see 
> any
> performance impact to Manhattan from GfxBench5. I'm little surprised to
> see it's not really helping benchmark performance in Mesa. I'll dig bit 
> more
> to figure out a possible explanation. I haven't tried any other benchmarks
> with this patch.

I think the concern was, if user is running old Mesa (no WA) on new 
kernel (no WA) there wouldn't be any GPU hangs, just theoretical (yet 
unmeasured) perf drop?

Regards,

Tvrtko

> 
> Thanks
> Anuj
> On 04/26/2019 01:31 AM, Joonas Lahtinen wrote:
>> + Anuj
>>
>> Quoting Lionel Landwerlin (2019-04-26 11:13:58)
>>> On 18/04/2019 18:06, Tvrtko Ursulin wrote:
>>>> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>>>
>>>> WaEnableStateCacheRedirectToCS context workaround configures the L3 
>>>> cache
>>>> to benefit 3d workloads but media has different requirements.
>>>>
>>>> Remove the workaround and whitelist the register to allow any userspace
>>>> configure the behaviour to their liking.
>>>>
>>>> v2:
>>>>    * Remove the workaround apart from adding the whitelist.
>>>>
>>>> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>>>> Cc: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>>>> Cc: kevin.ma@intel.com
>>>> Cc: xiaogang.li@intel.com
>>>
>>> Acked-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>>>
>>>
>>> Mesa commits :
>>>
>>> commit db5b372bb9f5a0dfea86618f8f9832f25d9eaf71 (anv)
>>>
>>> commit eaadb62c9ea98f841d7ffc26c14341abdf84d2d6 (i965)
>>>
>>> commit d1be67db39463b48369cb71979ed18662b2c157e (iris)
>> Could somebody confirm that applying this patch does not cause hangs in
>> older mesa, and the performance drop (if any) is insignificant?
>>
>> Best Regards,
>> Joonas
> 
> 
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2019-04-29  5:57 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-15 11:43 [PATCH] drm/i915/icl: Whitelist GEN9_SLICE_COMMON_ECO_CHICKEN1 Tvrtko Ursulin
2019-04-15 11:45 ` Chris Wilson
2019-04-15 12:25   ` Tvrtko Ursulin
2019-04-15 16:17     ` Chris Wilson
2019-04-15 12:38 ` ✓ Fi.CI.BAT: success for " Patchwork
2019-04-15 14:24 ` ✓ Fi.CI.IGT: " Patchwork
2019-04-18 10:06 ` [PATCH v2] " Tvrtko Ursulin
2019-04-26  8:13   ` Lionel Landwerlin
2019-04-26  8:31     ` Joonas Lahtinen
2019-04-26 16:58       ` Anuj Phogat
2019-04-29  5:57         ` Tvrtko Ursulin [this message]
2019-04-29 17:16           ` Anuj Phogat
2019-04-30  1:25             ` Li, Xiaogang
2019-04-26 17:01   ` Anuj Phogat
2019-04-30  6:53   ` Joonas Lahtinen
2019-04-30  6:57     ` Tvrtko Ursulin
2019-04-18 11:28 ` ✓ Fi.CI.BAT: success for drm/i915/icl: Whitelist GEN9_SLICE_COMMON_ECO_CHICKEN1 (rev2) Patchwork
2019-04-18 13:12 ` ✓ Fi.CI.IGT: " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=836034a4-4060-9f40-3a63-729e7b1d4a72@linux.intel.com \
    --to=tvrtko.ursulin@linux.intel.com \
    --cc=Intel-gfx@lists.freedesktop.org \
    --cc=anuj.phogat@intel.com \
    --cc=joonas.lahtinen@linux.intel.com \
    --cc=kevin.ma@intel.com \
    --cc=lionel.g.landwerlin@intel.com \
    --cc=xiaogang.li@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox