Re: [Intel-gfx] [PATCH 47/47] drm/i915/guc: Unblock GuC submission on Gen11+

From: Pekka Paalanen <ppaalanen@gmail.com>
To: John Harrison <john.c.harrison@intel.com>
Cc: Martin Peres <martin.peres@free.fr>,
	intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH 47/47] drm/i915/guc: Unblock GuC submission on Gen11+
Date: Thu, 1 Jul 2021 11:14:10 +0300	[thread overview]
Message-ID: <20210701111410.3fc6551e@eldfell> (raw)
In-Reply-To: <05e1d462-57ae-888a-888c-3ad486150821@intel.com>

[-- Attachment #1.1: Type: text/plain, Size: 2471 bytes --]

On Wed, 30 Jun 2021 11:58:25 -0700
John Harrison <john.c.harrison@intel.com> wrote:

> On 6/30/2021 01:22, Martin Peres wrote:
> > On 24/06/2021 10:05, Matthew Brost wrote:  
> >> From: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> >>
> >> Unblock GuC submission on Gen11+ platforms.
> >>
> >> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
> >> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> >> Signed-off-by: Matthew Brost <matthew.brost@intel.com>
> >> ---
> >>   drivers/gpu/drm/i915/gt/uc/intel_guc.h            |  1 +
> >>   drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c |  8 ++++++++
> >>   drivers/gpu/drm/i915/gt/uc/intel_guc_submission.h |  3 +--
> >>   drivers/gpu/drm/i915/gt/uc/intel_uc.c             | 14 +++++++++-----
> >>   4 files changed, 19 insertions(+), 7 deletions(-)
> >>

...

> >> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc.c 
> >> b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
> >> index 7a69c3c027e9..61be0aa81492 100644
> >> --- a/drivers/gpu/drm/i915/gt/uc/intel_uc.c
> >> +++ b/drivers/gpu/drm/i915/gt/uc/intel_uc.c
> >> @@ -34,8 +34,15 @@ static void uc_expand_default_options(struct 
> >> intel_uc *uc)
> >>           return;
> >>       }
> >>   -    /* Default: enable HuC authentication only */
> >> -    i915->params.enable_guc = ENABLE_GUC_LOAD_HUC;
> >> +    /* Intermediate platforms are HuC authentication only */
> >> +    if (IS_DG1(i915) || IS_ALDERLAKE_S(i915)) {
> >> +        drm_dbg(&i915->drm, "Disabling GuC only due to old 
> >> platform\n");  
> >
> > This comment does not seem accurate, given that DG1 is barely out, and 
> > ADL is not out yet. How about:
> >
> > "Disabling GuC on untested platforms"?
> >  
> Just because something is not in the shops yet does not mean it is new. 
> Technology is always obsolete by the time it goes on sale.

That is a very good reason to not use terminology like "new", "old",
"current", "modern" etc. at all.

End users like me definitely do not share your interpretation of "old".

Thanks,
pq

> And the issue is not a lack of testing, it is a question of whether we 
> are allowed to change the default on something that has already started 
> being used by customers or not (including pre-release beta customers). 
> I.e. it is basically a political decision not an engineering decision.

[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx