* Lorenzo Stoakes <[email protected]> [260108 10:20]:
> On Wed, Jan 07, 2026 at 09:31:15PM +0100, Mikulas Patocka wrote:
> > This error may happen if mm_take_all_locks was interrupted by a signal -
> > the userspace will retry an ioctl that returned -EINTR, so there is no
> > need to report it to the syslog.
> >
> > Christian König suggested to remove this message at all, so let's do it.
> >
> > Signed-off-by: Mikulas Patocka <[email protected]>
>
> I don't really see the point in this being part of the series here... seems
> to me a patch that could be applied any time, so why not send that
> separately to the amd driver people?
>
> I mean maybe it's not al that important but still.
I really think changing this to info would be better. It was (and may
be) useful to some?
Obviously the amd folks need to work on signal handling and this would
help them..
But then again, as Lorenzo said, we don't really need to be involved in
this patch.
>
> >
> > ---
> > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 5 +----
> > 1 file changed, 1 insertion(+), 4 deletions(-)
> >
> > Index: mm/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
> > ===================================================================
> > --- mm.orig/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
> > 2026-01-07 20:09:51.000000000 +0100
> > +++ mm/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c 2026-01-07
> > 20:12:11.000000000 +0100
> > @@ -1069,11 +1069,8 @@ static int init_user_pages(struct kgd_me
> > }
> >
> > ret = amdgpu_hmm_register(bo, user_addr);
> > - if (ret) {
> > - pr_err("%s: Failed to register MMU notifier: %d\n",
> > - __func__, ret);
> > + if (ret)
> > goto out;
> > - }
> >
> > if (criu_resume) {
> > /*
> >
> >
