<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="#0563C1" vlink="#954F72"> <div class="WordSection1"> <p class="MsoNormal">Hi Christian,<o:p></o:p></p> <p class="MsoNormal"> Before unloading driver, user must sure there is not any userspace to use of amdgpu, if not, it will report driver is in use. And the unloading driver is a feature about amdgpu driver which will be easier to replace driver without rebooting VM. Do you think it has any issue about driver unloading path?<o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal">Best wishes<o:p></o:p></p> <p class="MsoNormal">Emily Deng<o:p></o:p></p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> <p class="MsoNormal"><b>From:</b> Koenig, Christian <Christian.Koenig@amd.com> <br> <b>Sent:</b> Wednesday, September 18, 2019 3:54 PM<br> <b>To:</b> Zhang, Jack (Jian) <Jack.Zhang1@amd.com><br> <b>Cc:</b> amd-gfx@lists.freedesktop.org; Teng, Rui <Rui.Teng@amd.com>; Deng, Emily <Emily.Deng@amd.com>; Cui, Flora <Flora.Cui@amd.com><br> <b>Subject:</b> RE: [PATCH] drm/amdgpu/sriov: omit fbcon error under sriov or passthrough<o:p></o:p></p> </div> </div> <p class="MsoNormal"><o:p> </o:p></p> <div> <div> <p class="MsoNormal">Hi Jack,<o:p></o:p></p> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">Well that believe is unfortunately completely wrong.<o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">The point is that ANY use of amdgpu by userspace will prevent correct driver unload, that qxl is used for the fbcon doesn't change anything here.<o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">So the patch is a clear NAK. Driver unload is not supposed to work even under SRIOV.<o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">Regards,<o:p></o:p></p> </div> <div> <p class="MsoNormal">Christian.<o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> <div> <p class="MsoNormal">Am 18.09.2019 09:32 schrieb "Zhang, Jack (Jian)" <<a href="mailto:Jack.Zhang1@amd.com">Jack.Zhang1@amd.com</a>>:<o:p></o:p></p> </div> </div> </div> <div> <p class="MsoNormal" style="margin-bottom:12.0pt">Hi, Christian and folks,<br> <br> In virtual machines(such virt-manager), there's always a virtual graphics device existed like "qxl" as the default gfx device.<br> So under such condition, we believe it's reasonable to unload amdgpu driver as it is not treated as the default fbcon device.<br> <br> Would you please help to review this patch?<br> <br> Best wishes,<br> Jack<br> <br> -----Original Message-----<br> From: Jack Zhang <<a href="mailto:Jack.Zhang1@amd.com">Jack.Zhang1@amd.com</a>> <br> Sent: Wednesday, September 18, 2019 3:25 PM<br> To: <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a><br> Cc: Zhang, Jack (Jian) <<a href="mailto:Jack.Zhang1@amd.com">Jack.Zhang1@amd.com</a>><br> Subject: [PATCH] drm/amdgpu/sriov: omit fbcon error under sriov or passthrough<br> <br> In virtual machine, there would be a qxl or cirrus graphics device as the default master fbcon device.<br> <br> So for PF(passthrough mode) or SRIOV VF, it is reasonable to unload amdgpu driver. Amdgpu doesn't have to be the only fbcon device under this condition.<br> <br> Signed-off-by: Jack Zhang <<a href="mailto:Jack.Zhang1@amd.com">Jack.Zhang1@amd.com</a>><br> ---<br> drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 5 +++--<br> 1 file changed, 3 insertions(+), 2 deletions(-)<br> <br> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c<br> index 420888e..ada2b25 100644<br> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c<br> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c<br> @@ -1103,8 +1103,9 @@ static void<br> amdgpu_pci_remove(struct pci_dev *pdev) {<br> struct drm_device *dev = pci_get_drvdata(pdev);<br> -<br> - DRM_ERROR("Device removal is currently not supported outside of fbcon\n");<br> + struct amdgpu_device *adev = dev->dev_private;<br> + if (!amdgpu_sriov_vf(adev) && !amdgpu_passthrough(adev))<br> + DRM_ERROR("Device removal is currently not supported outside of <br> +fbcon\n");<br> drm_dev_unplug(dev);<br> drm_dev_put(dev);<br> pci_disable_device(pdev);<br> --<br> 2.7.4<o:p></o:p></p> </div> </div> </div> </body> </html>