[Public] It is actually a bug of discovery table which needs to be identified in NPI phase. Hopefully we don't need neither dummy nor yellow carp cache info in the future. Best Regards, Yifan -----Original Message----- From: Liang, Prike <Prike.Liang@xxxxxxx> Sent: Monday, September 5, 2022 9:24 AM To: Zhang, Yifan <Yifan1.Zhang@xxxxxxx>; Liu, Aaron <Aaron.Liu@xxxxxxx>; Huang, Tim <Tim.Huang@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Du, Xiaojian <Xiaojian.Du@xxxxxxx>; Huang, Ray <Ray.Huang@xxxxxxx>; Kuehling, Felix <Felix.Kuehling@xxxxxxx> Subject: RE: [PATCH] drm/amdkfd: Match GC 11.0.1 cache info to yellow carp How about add a dummy cache info for the NPI product in the default case and notify user that's using the dummy cache configuration to make sure not miss correcting the HW info in the future? Regards, --Prike -----Original Message----- From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Zhang, Yifan Sent: Friday, September 2, 2022 10:28 AM To: Liu, Aaron <Aaron.Liu@xxxxxxx>; Huang, Tim <Tim.Huang@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Du, Xiaojian <Xiaojian.Du@xxxxxxx> Subject: RE: [PATCH] drm/amdkfd: Match GC 11.0.1 cache info to yellow carp [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.Liu@xxxxxxx> Sent: Friday, September 2, 2022 9:44 AM To: Huang, Tim <Tim.Huang@xxxxxxx>; Zhang, Yifan <Yifan1.Zhang@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Du, Xiaojian <Xiaojian.Du@xxxxxxx> 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-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of > Huang, Tim > Sent: Friday, September 2, 2022 6:44 AM > To: Zhang, Yifan <Yifan1.Zhang@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Du, Xiaojian > <Xiaojian.Du@xxxxxxx> > Subject: RE: [PATCH] drm/amdkfd: Match GC 11.0.1 cache info to yellow > carp > > [Public] > > [Public] > > Reviewed-by: Tim Huang <Tim.Huang@xxxxxxx> > > Best Regards, > Tim Huang > > > > -----Original Message----- > From: Zhang, Yifan <Yifan1.Zhang@xxxxxxx> > Sent: Thursday, September 1, 2022 3:30 PM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Huang, Tim > <Tim.Huang@xxxxxxx>; Du, Xiaojian <Xiaojian.Du@xxxxxxx>; Zhang, Yifan > <Yifan1.Zhang@xxxxxxx> > 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.zhang@xxxxxxx> > --- > 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
