✓ CI.checkpatch: success for drm/xe/ptl: L3bank mask is not available on the media GT (rev2)
Patchwork
patchwork at emeril.freedesktop.org
Mon Oct 7 20:23:04 UTC 2024
== Series Details ==
Series: drm/xe/ptl: L3bank mask is not available on the media GT (rev2)
URL : https://patchwork.freedesktop.org/series/139586/
State : success
== Summary ==
+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
30ab6715fc09baee6cc14cb3c89ad8858688d474
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 37b15026d2df333f83c8b421cd5868a76543460d
Author: Shekhar Chauhan <shekhar.chauhan at intel.com>
Date: Mon Oct 7 08:41:44 2024 -0700
drm/xe/ptl: L3bank mask is not available on the media GT
On PTL platforms with media version 30.00, the fuse registers for
reporting L3 bank availability to the GT just read out as ~0 and do not
provide proper values. Xe does not use the L3 bank mask for anything
internally; it only passes the mask through to userspace via the GT
topology query.
Since we don't have any way to get the real L3 bank mask, we don't want
to pass garbage to userspace. Passing a zeroed mask or a copy of the
primary GT's L3 bank mask would also be inaccurate and likely to cause
confusion for userspace. The best approach is to simply not include L3
in the list of masks returned by the topology query in cases where we
aren't able to provide a meaningful value. This won't change the
behavior for any existing platforms (where we can always obtain L3 masks
successfully for all GTs), it will only prevent us from mis-reporting
bad information on upcoming platform(s).
There's a good chance this will become a formal workaround in the
future, but for now we don't have a lineage number so "no_media_l3" is
used in place of a lineage as the OOB workaround descriptor.
v2:
- Re-calculate query size to properly match data returned. (Gustavo)
- Update kerneldoc to clarify that the L3bank mask may not be included
in the query results if the hardware doesn't make it available.
(Gustavo)
Cc: Matt Atwood <matthew.s.atwood at intel.com>
Cc: Gustavo Sousa <gustavo.sousa at intel.com>
Signed-off-by: Shekhar Chauhan <shekhar.chauhan at intel.com>
Co-developed-by: Matt Roper <matthew.d.roper at intel.com>
Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
Reviewed-by: Jonathan Cavitt <jonathan.cavitt at intel.com>
Reviewed-by: Gustavo Sousa <gustavo.sousa at intel.com>
+ /mt/dim checkpatch 1f72b793ff063e3519fc58ca3f3e83854b990ee7 drm-intel
37b15026d2df drm/xe/ptl: L3bank mask is not available on the media GT
More information about the Intel-xe
mailing list