All of lore.kernel.org
 help / color / mirror / Atom feed
* [GIT PULL v2] dlm fixes for 6.9
@ 2024-03-18 21:25 David Teigland
  2024-03-18 22:44 ` Linus Torvalds
  2024-03-18 22:47 ` pr-tracker-bot
  0 siblings, 2 replies; 3+ messages in thread
From: David Teigland @ 2024-03-18 21:25 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: linux-kernel, gfs2

Hi Linus,

Please pull dlm fixes from tag:

git://git.kernel.org/pub/scm/linux/kernel/git/teigland/linux-dlm.git dlm-6.9

(For v2 I recreated the tag with the same name as before, but now realize
I probably should have left it and used a new tag name.)

I dropped the commit with the bad atomic usage, and replaced it with two
other commits: the first reverts the unnecessary change that began using
atomic_t for lkb_wait_count, and the second adds comments to the recovery
code that forcibly resets the wait_count state.

Thanks,
Dave


Alexander Aring (1):
      dlm: fix user space lkb refcounting

David Teigland (2):
      dlm: revert atomic_t lkb_wait_count
      dlm: add comments about forced waiters reset


 fs/dlm/dlm_internal.h |   2 +-
 fs/dlm/lock.c         | 114 ++++++++++++++++++++++++++++++++++----------------
 fs/dlm/user.c         |  10 ++---
 3 files changed, 84 insertions(+), 42 deletions(-)


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [GIT PULL v2] dlm fixes for 6.9
  2024-03-18 21:25 [GIT PULL v2] dlm fixes for 6.9 David Teigland
@ 2024-03-18 22:44 ` Linus Torvalds
  2024-03-18 22:47 ` pr-tracker-bot
  1 sibling, 0 replies; 3+ messages in thread
From: Linus Torvalds @ 2024-03-18 22:44 UTC (permalink / raw)
  To: David Teigland; +Cc: linux-kernel, gfs2

On Mon, 18 Mar 2024 at 14:25, David Teigland <teigland@redhat.com> wrote:
>
> I dropped the commit with the bad atomic usage, and replaced it with two
> other commits: the first reverts the unnecessary change that began using
> atomic_t for lkb_wait_count, and the second adds comments to the recovery
> code that forcibly resets the wait_count state.

Ok, the diff certainly looks saner. I didn't look at the code outside
the context of the diff, so that's literally just going by the patches
themselves, but I appreciate the comment ("The wait_count will almost
always be 1, but in case of an overlapping unlock/cancel it could be
2: see ..") and yes, it just makes the old atomic thing sound even
odder.

Thanks,

            Linus

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [GIT PULL v2] dlm fixes for 6.9
  2024-03-18 21:25 [GIT PULL v2] dlm fixes for 6.9 David Teigland
  2024-03-18 22:44 ` Linus Torvalds
@ 2024-03-18 22:47 ` pr-tracker-bot
  1 sibling, 0 replies; 3+ messages in thread
From: pr-tracker-bot @ 2024-03-18 22:47 UTC (permalink / raw)
  To: David Teigland; +Cc: Linus Torvalds, linux-kernel, gfs2

The pull request you sent on Mon, 18 Mar 2024 16:25:07 -0500:

> git://git.kernel.org/pub/scm/linux/kernel/git/teigland/linux-dlm.git dlm-6.9

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/b3603fcb79b1036acae10602bffc4855a4b9af80

Thank you!

-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-03-18 22:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-18 21:25 [GIT PULL v2] dlm fixes for 6.9 David Teigland
2024-03-18 22:44 ` Linus Torvalds
2024-03-18 22:47 ` pr-tracker-bot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.