Topaz mistakenly reported as vf

Deucher, Alexander Alexander.Deucher at
Tue Dec 19 14:20:44 UTC 2017

Please make sure we only check that register on asics where it is relevant.  There are a bunch of VI asics.  It should probably be restricted to tonga and fiji only.


From: Yu, Xiangliang
Sent: Tuesday, December 19, 2017 2:44 AM
To: José Pekkarinen
Cc: amd-gfx at; Deucher, Alexander; Koenig, Christian
Subject: RE: Topaz mistakenly reported as vf

We can add ASIC check in vi_detect_hw_virtualization function to avoid to entry error path.
I'll submit patch later.


> -----Original Message-----
> From: José Pekkarinen [mailto:jose.pekkarinen at]
> Sent: Tuesday, December 19, 2017 3:27 PM
> To: Yu, Xiangliang <Xiangliang.Yu at>
> Cc: amd-gfx at; Deucher, Alexander
> <Alexander.Deucher at>; Koenig, Christian
> <Christian.Koenig at>
> Subject: Re: Topaz mistakenly reported as vf
> On Tuesday, 19 December 2017 09:19:02 EET Yu, Xiangliang wrote:
> > Topaz doesn't support SRIOV.
> >
>        Hi Xiangliang,
>        Allow me to ask for some attention, as I'm trying to say that given
> that Topaz is not having support, amdgpu_sriov_vf(adev) may return false
> and ignore the vi.c code path. Instead of it, it enters the code path and
> reaches the BUG_ON that states this is not supported.
>        Thanks for coming back!
>        José.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the amd-gfx mailing list