SWDEV-196010 Calltrace caught in amdgpu_vm_sdma.c file

Submitted by Liu, Monk on July 16, 2019, 3:41 a.m.

Details

Message ID MN2PR12MB39336BD78BF1438F7BD1681984CE0@MN2PR12MB3933.namprd12.prod.outlook.com
State New
Headers show
Series "SWDEV-196010 Calltrace caught in amdgpu_vm_sdma.c file" ( rev: 2 ) in AMD X.Org drivers

Not browsing as part of any series.

Commit Message

Liu, Monk July 16, 2019, 3:41 a.m.
Please ignore this patch, looks 643d146c86c2f1e29cb18db93fbcd2ee43e6959f   already addressed the issue.

_____________________________________
Monk Liu|GPU Virtualization Team |AMD


-----Original Message-----
From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of Monk Liu

Sent: Tuesday, July 16, 2019 11:09 AM
To: amd-gfx@lists.freedesktop.org
Cc: Liu, Monk <Monk.Liu@amd.com>
Subject: [PATCH] SWDEV-196010 Calltrace caught in amdgpu_vm_sdma.c file

don't commit sdma vm job if no updates needed and free the ib

Signed-off-by: Monk Liu <Monk.Liu@amd.com>

---
 drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

--
2.7.4

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Patch hide | download patch | download mbox

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index aeba9e6..ff6d37e 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -1304,9 +1304,15 @@  int amdgpu_vm_update_directories(struct amdgpu_device *adev,
 			goto error;
 	}
 
-	r = vm->update_funcs->commit(&params, &vm->last_update);
-	if (r)
-		goto error;
+
+	if (params.ib->length_dw > 0) {
+		r = vm->update_funcs->commit(&params, &vm->last_update);
+		if (r)
+			goto error;
+	} else {
+		amdgpu_sa_bo_free(adev, &params.ib->sa_bo, NULL);
+	}
+
 	return 0;
 
 error:

Comments

Ah! Yeah that was a known issue. Please ignore my response as well :)

Sorry for the noise,
Christian.

Am 16.07.19 um 05:41 schrieb Liu, Monk:
> Please ignore this patch, looks 643d146c86c2f1e29cb18db93fbcd2ee43e6959f   already addressed the issue.
>
> _____________________________________
> Monk Liu|GPU Virtualization Team |AMD
>
>
> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of Monk Liu
> Sent: Tuesday, July 16, 2019 11:09 AM
> To: amd-gfx@lists.freedesktop.org
> Cc: Liu, Monk <Monk.Liu@amd.com>
> Subject: [PATCH] SWDEV-196010 Calltrace caught in amdgpu_vm_sdma.c file
>
> don't commit sdma vm job if no updates needed and free the ib
>
> Signed-off-by: Monk Liu <Monk.Liu@amd.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 12 +++++++++---
>   1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> index aeba9e6..ff6d37e 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> @@ -1304,9 +1304,15 @@ int amdgpu_vm_update_directories(struct amdgpu_device *adev,
>   			goto error;
>   	}
>   
> -	r = vm->update_funcs->commit(&params, &vm->last_update);
> -	if (r)
> -		goto error;
> +
> +	if (params.ib->length_dw > 0) {
> +		r = vm->update_funcs->commit(&params, &vm->last_update);
> +		if (r)
> +			goto error;
> +	} else {
> +		amdgpu_sa_bo_free(adev, &params.ib->sa_bo, NULL);
> +	}
> +
>   	return 0;
>   
>   error:
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx