[PATCH 0/2] Skip IH re-route on Vega SR-IOV

[Huang, Trigger]

Hi Christian,

On Vega10 SR-IOV VF, I injected a 'real' VMC page fault from user space, using the modified amdgpu_test.
[   19.127874] amdgpu 0000:00:08.0: [gfxhub] no-retry page fault (src_id:0 ring:174 vmid:1 pasid:32768, for process amdgpu_test pid 1071 thread amdgpu_test pid 1071)
[   19.130037] amdgpu 0000:00:08.0:   in page starting at address 0x0000000800000000 from 27

And see this interrupt is still from IH0 amdgpu_irq_handler, which can prove this feature is not working under SR-IOV.

I suggest to remove this feature from SR-IOV, as my concern is,  some weird bugs may be cased by it in the Virtualization heavy stress test.
In the future, maybe we can request PSP team to add this support for SR-IOV.

Thanks & Best Wishes,
Trigger Huang

-----Original Message-----
From: Christian König <ckoenig.leichtzumerken at gmail.com> 
Sent: Tuesday, May 07, 2019 3:37 PM
To: Huang, Trigger <Trigger.Huang at amd.com>; amd-gfx at lists.freedesktop.org
Subject: Re: [PATCH 0/2] Skip IH re-route on Vega SR-IOV

[CAUTION: External Email]

We intentionally didn't do this to make sure that the commands are ignored by the PSP firmware.

I have no strong opinion on if we should do this or not, but the PSP firmware guys might have.

Christian.

Am 07.05.19 um 06:08 schrieb Trigger Huang:
> IH re-route is not supported on Vega SR-IOV, need to be skipped
>
> Trigger Huang (2):
>    drm/amdgpu: Skip IH reroute in Vega10 SR-IOV VF
>    drm/amdgpu: Skip IH reroute in Vega20 SR-IOV VF
>
>   drivers/gpu/drm/amd/amdgpu/psp_v11_0.c | 4 ++++
>   drivers/gpu/drm/amd/amdgpu/psp_v3_1.c  | 4 ++++
>   2 files changed, 8 insertions(+)
>