On Wed, Jan 8, 2020 at 1:15 AM Tiecheng Zhou <Tiecheng.Zhou@xxxxxxx> 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 <Tiecheng.Zhou@xxxxxxx> > --- > 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 <alexander.deucher@xxxxxxx> > adev->external_rev_id = adev->rev_id + 0xa; > break; > default: > -- > 2.17.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx