lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aLHr5VAuXQZE0bBd@redhat.com>
Date: Fri, 29 Aug 2025 13:05:42 -0500
From: David Teigland <teigland@...hat.com>
To: Alessio Attilio <alessio.attilio.dev@...il.com>
Cc: Alexander Aring <aahringo@...hat.com>, gfs2@...ts.linux.dev,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] fix(dlm): handle unlock/cancel during deferred lock
 messages

On Fri, Aug 29, 2025 at 01:07:49PM +0200, Alessio Attilio wrote:
> ---
>  fs/dlm/lock.c | 8067 ++++++++++++++++++++++++-------------------------
>  1 file changed, 3944 insertions(+), 4123 deletions(-)

Thanks for working on this, lock cancel has always been sketchy, and I'd
be happy to see that improved.  An 8000 line patch is going to be
difficult to review, though!  It looks like diff may just be moving a lot
of lines around, and perhaps splitting this up could avoid some of that?
Also, I'm not seeing which upstream commit the patch is against.

Thanks,
Dave



Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ