Topaz mistakenly reported as vf

Xiangliang.Yu@xxxxxxx (Yu, Xiangliang) · Tue, 19 Dec 2017 07:44:00 +0000

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

Thanks!

> -----Original Message-----
> From: JosÃ© Pekkarinen [mailto:jose.pekkarinen at canonical.com]
> Sent: Tuesday, December 19, 2017 3:27 PM
> To: Yu, Xiangliang <Xiangliang.Yu at amd.com>
> Cc: amd-gfx at lists.freedesktop.org; Deucher, Alexander
> <Alexander.Deucher at amd.com>; Koenig, Christian
> <Christian.Koenig at amd.com>
> 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Ã©.
> 