[AMD Official Use Only - Internal Distribution Only] Ah not aware the enable patch before. But the already enable patch seems can't fallback to legacy gpuinfo FW load method when not support discovery and also may miss destroy the discovery_bin object when driver shut down. Thanks, Prike > -----Original Message----- > From: Alex Deucher <alexdeucher@xxxxxxxxx> > Sent: Tuesday, June 2, 2020 9:35 PM > To: Liang, Prike <Prike.Liang@xxxxxxx> > Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; Deucher, Alexander > <Alexander.Deucher@xxxxxxx>; Huang, Ray <Ray.Huang@xxxxxxx> > Subject: Re: [PATCH] drm/amdgpu: enable renoir discovery for gc info > retrieved > > On Mon, Jun 1, 2020 at 10:14 PM Liang, Prike <Prike.Liang@xxxxxxx> wrote: > > > > [AMD Official Use Only - Internal Distribution Only] > > > > Ping... > > Already enabled: > https://nam11.safelinks.protection.outlook.com/?url=https:%2F%2Fcgit.free > desktop.org%2F~agd5f%2Flinux%2Fcommit%2F%3Fh%3Damd-staging-drm- > next%26id%3De467ab869f5783cf93d4cf24c6ac647cc29d1fb5&data=02% > 7C01%7CPrike.Liang%40amd.com%7C5bcc45116bb042163cec08d806f9bd58 > %7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637267016987033 > 430&sdata=KJ0xmSPXwlZ4LEfhAYoFzAwaWyx3laoLAsQccMM0pcs%3D& > amp;reserved=0 > > Alex > > > > > Thanks, > > > -----Original Message----- > > > From: Liang, Prike <Prike.Liang@xxxxxxx> > > > Sent: Friday, May 29, 2020 11:28 AM > > > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > > > Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Huang, Ray > > > <Ray.Huang@xxxxxxx>; Liang, Prike <Prike.Liang@xxxxxxx> > > > Subject: [PATCH] drm/amdgpu: enable renoir discovery for gc info > > > retrieved > > > > > > Use ip discovery GC table instead of gpu info firmware for exporting > > > gpu info to inquire interface.As Renoir discovery has same version > > > with Navi1x therefore just enable it same way as Navi1x. > > > > > > Signed-off-by: Prike.Liang <Prike.Liang@xxxxxxx> > > > --- > > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 23 > > > ++++++++++++++++++++--- > > > 1 file changed, 20 insertions(+), 3 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > > index 2f0e8da..bff740ccd 100644 > > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > > @@ -1528,7 +1528,7 @@ static int > > > amdgpu_device_parse_gpu_info_fw(struct amdgpu_device *adev) { > > > const char *chip_name; char fw_name[30]; -int err; > > > +int err, r; > > > const struct gpu_info_firmware_header_v1_0 *hdr; > > > > > > adev->firmware.gpu_info_fw = NULL; > > > @@ -1578,6 +1578,23 @@ static int > > > amdgpu_device_parse_gpu_info_fw(struct amdgpu_device *adev) > > > chip_name = "arcturus"; break; case CHIP_RENOIR: > > > +if (amdgpu_discovery) { > > > +/** > > > + * For RENOIR series seems needn't reinitialize the reg base > > > again as it already set during > > > + * early init,if any concern here will need export > > > amdgpu_discovery_init() for this case. > > > + */ > > > +r = amdgpu_discovery_reg_base_init(adev); > > > +if (r) { > > > +DRM_WARN("failed to get ip discovery table, > > > " > > > +"fallback to get gpu info in legacy > > > method\n"); > > > +goto legacy_gpuinfo; > > > +} > > > + > > > +amdgpu_discovery_get_gfx_info(adev); > > > + > > > +return 0; > > > +} > > > +legacy_gpuinfo: > > > chip_name = "renoir"; > > > break; > > > case CHIP_NAVI10: > > > @@ -1617,7 +1634,7 @@ static int > > > amdgpu_device_parse_gpu_info_fw(struct amdgpu_device *adev) (const > > > struct gpu_info_firmware_v1_0 *)(adev- > > > >firmware.gpu_info_fw->data + > > > > > > le32_to_cpu(hdr->header.ucode_array_offset_bytes)); > > > > > > -if (amdgpu_discovery && adev->asic_type >= CHIP_NAVI10) { > > > +if (amdgpu_discovery && adev->asic_type >= CHIP_RENOIR > > > && !r) { > > > amdgpu_discovery_get_gfx_info(adev); > > > goto parse_soc_bounding_box; > > > } > > > @@ -3364,7 +3381,7 @@ void amdgpu_device_fini(struct > amdgpu_device > > > *adev) > > > sysfs_remove_files(&adev->dev->kobj, amdgpu_dev_attributes); if > > > (IS_ENABLED(CONFIG_PERF_EVENTS)) amdgpu_pmu_fini(adev); -if > > > (amdgpu_discovery && adev->asic_type >= CHIP_NAVI10) > > > +if (amdgpu_discovery && adev->asic_type >= CHIP_RENOIR) > > > amdgpu_discovery_fini(adev); > > > } > > > > > > -- > > > 2.7.4 > > > > _______________________________________________ > > amd-gfx mailing list > > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flist > > s.freedesktop.org%2Fmailman%2Flistinfo%2Famd- > gfx&data=02%7C01%7CPr > > > ike.Liang%40amd.com%7C5bcc45116bb042163cec08d806f9bd58%7C3dd896 > 1fe4884 > > > e608e11a82d994e183d%7C0%7C0%7C637267016987033430&sdata=R% > 2F%2BY%2B > > > z%2BKHh09WazkQqS%2FbwH%2BeBM97%2Fx5hvqWAjUYEtM%3D&res > erved=0 _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx