[Public] Hi Aaron,
Yes, the cache info are different. But this diff is intentional to workaround the discovery table lacks cache info in GC 11.0.1 issue. The workaround will be removed after discovery table finishes integrating cache info. Given they already have a test version, we can pend this patch until they finish integration. Best Regards, Yifan -----Original Message----- From: Liu, Aaron <aaron....@amd.com> Sent: Friday, September 2, 2022 9:44 AM To: Huang, Tim <tim.hu...@amd.com>; Zhang, Yifan <yifan1.zh...@amd.com>; amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Du, Xiaojian <xiaojian...@amd.com> Subject: RE: [PATCH] drm/amdkfd: Match GC 11.0.1 cache info to yellow carp [Public] Hi Yifan, Yellow carp's cache info cannot be duplicated to GC_11_0_1. Different point to GC_11_0_1: TCP L1 Cache size is 32 GL1 Data Cache size per SA is 256 Others looks good to me -- Best Regards Aaron Liu > -----Original Message----- > From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of > Huang, Tim > Sent: Friday, September 2, 2022 6:44 AM > To: Zhang, Yifan <yifan1.zh...@amd.com>; amd-gfx@lists.freedesktop.org > Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Du, Xiaojian > <xiaojian...@amd.com> > Subject: RE: [PATCH] drm/amdkfd: Match GC 11.0.1 cache info to yellow > carp > > [Public] > > [Public] > > Reviewed-by: Tim Huang <tim.hu...@amd.com> > > Best Regards, > Tim Huang > > > > -----Original Message----- > From: Zhang, Yifan <yifan1.zh...@amd.com> > Sent: Thursday, September 1, 2022 3:30 PM > To: amd-gfx@lists.freedesktop.org > Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Huang, Tim > <tim.hu...@amd.com>; Du, Xiaojian <xiaojian...@amd.com>; Zhang, Yifan > <yifan1.zh...@amd.com> > Subject: [PATCH] drm/amdkfd: Match GC 11.0.1 cache info to yellow carp > > Current discovery table doesn't have cache info for GC 11.0.1, thus > can't be parsed like other GC 11, this patch to match GC 11.0.1 cache > info to yellow carp > > Signed-off-by: Yifan Zhang <yifan1.zh...@amd.com> > --- > drivers/gpu/drm/amd/amdkfd/kfd_crat.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > b/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > index 24b414cff3ec..1c500bfb0b28 100644 > --- a/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > +++ b/drivers/gpu/drm/amd/amdkfd/kfd_crat.c > @@ -1516,11 +1516,11 @@ static int kfd_fill_gpu_cache_info(struct > kfd_dev *kdev, > case IP_VERSION(10, 3, 3): > case IP_VERSION(10, 3, 6): /* TODO: Double check these > on production silicon */ > case IP_VERSION(10, 3, 7): /* TODO: Double check these > on production silicon */ > + case IP_VERSION(11, 0, 1): /* TODO: Double check these > +on production silicon */ > pcache_info = yellow_carp_cache_info; > num_of_cache_types = > ARRAY_SIZE(yellow_carp_cache_info); > break; > case IP_VERSION(11, 0, 0): > - case IP_VERSION(11, 0, 1): > case IP_VERSION(11, 0, 2): > case IP_VERSION(11, 0, 3): > pcache_info = cache_info; > -- > 2.37.1
