On Wed, Jan 8, 2020 at 1:15 AM Tiecheng Zhou <[email protected]> wrote: > > guest vm gets 0xffffffff when reading RCC_DEV0_EPF0_STRAP0, > as a consequence, the rev_id and external_rev_id are wrong. > > workaround it by hardcoding the rev_id to 0, which is the default value. > > Signed-off-by: Tiecheng Zhou <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/nv.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/nv.c b/drivers/gpu/drm/amd/amdgpu/nv.c > index b0229543e887..63d54604ace6 100644 > --- a/drivers/gpu/drm/amd/amdgpu/nv.c > +++ b/drivers/gpu/drm/amd/amdgpu/nv.c > @@ -726,6 +726,8 @@ static int nv_common_early_init(void *handle) > AMD_PG_SUPPORT_VCN_DPG | > AMD_PG_SUPPORT_JPEG | > AMD_PG_SUPPORT_ATHUB; > + if (amdgpu_sriov_vf(adev)) > + adev->rev_id = 0;
Please add a comment explaining this here so it's clear why we are doing this. With that added: Reviewed-by: Alex Deucher <[email protected]> > adev->external_rev_id = adev->rev_id + 0xa; > break; > default: > -- > 2.17.1 > > _______________________________________________ > amd-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
