Applied. Thanks!
On Mon, Dec 1, 2025 at 9:04 AM Christian König <[email protected]> wrote: > > On 12/1/25 15:00, Natalie Vock wrote: > > Otherwise userspace may be fooled into believing it has a reserved VMID > > when in reality it doesn't, ultimately leading to GPU hangs when SPM is > > used. > > > > Fixes: 80e709ee6ecc ("drm/amdgpu: add option params to enforce process > > isolation between graphics and compute") > > Cc: [email protected] > > Signed-off-by: Natalie Vock <[email protected]> > > Reviewed-by: Christian König <[email protected]> > > > --- > > drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > > index 61820166efbf6..1479742556991 100644 > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c > > @@ -2921,8 +2921,7 @@ int amdgpu_vm_ioctl(struct drm_device *dev, void > > *data, struct drm_file *filp) > > switch (args->in.op) { > > case AMDGPU_VM_OP_RESERVE_VMID: > > /* We only have requirement to reserve vmid from gfxhub */ > > - amdgpu_vmid_alloc_reserved(adev, vm, AMDGPU_GFXHUB(0)); > > - break; > > + return amdgpu_vmid_alloc_reserved(adev, vm, AMDGPU_GFXHUB(0)); > > case AMDGPU_VM_OP_UNRESERVE_VMID: > > amdgpu_vmid_free_reserved(adev, vm, AMDGPU_GFXHUB(0)); > > break; >
