Hi Trigger,
And see this interrupt is still from IH0 amdgpu_irq_handler, which can prove this feature is not working under SR-IOV.
In this case this change is a clear NAK.
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.
And I really think we should keep it to make sure that we have the same
handling for bare metal as for SRIOV.
In the future, maybe we can request PSP team to add this support for SR-IOV.
We will never be able to use this under SRIOV because of limitation in
the hardware.
What we could maybe do is check the response code from the PSP firmware
if it correctly ignored the commands under SR-IOV, but I think the
response code is the same for ignoring as for executing the commands.
Regards,
Christian.
Am 07.05.19 um 10:54 schrieb 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@xxxxxxxxx>
Sent: Tuesday, May 07, 2019 3:37 PM
To: Huang, Trigger <Trigger.Huang@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
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(+)
_______________________________________________
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
