All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Marzinski <bmarzins@redhat.com>
To: Mike Snitzer <snitzer@kernel.org>
Cc: Mikulas Patocka <mpatocka@redhat.com>,
	Heinz Mauelshagen <mauelshagen@redhat.com>,
	dm-devel@lists.linux.dev,
	Joel Colledge <joel.colledge@linbit.com>
Subject: Re: [PATCH 0/3] dm-delay synchronization cleanup
Date: Thu, 9 May 2024 12:07:26 -0400	[thread overview]
Message-ID: <Zjz0vkYQSMPxHFqz@redhat.com> (raw)
In-Reply-To: <ZjzctGe27JkXsANR@redhat.com>

On Thu, May 09, 2024 at 10:24:52AM -0400, Mike Snitzer wrote:
> On Tue, May 07, 2024 at 05:16:22PM -0400, Benjamin Marzinski wrote:
> > This patchset fixes a possible crash in delay_dtr() caused by the
> > delay_timer queueing work on a destroyed workqueue. It also cuts down
> > on the amount of locking and waiting the dm-delay does.
> > 
> > Benjamin Marzinski (3):
> >   dm-delay: fix workqueue delay_timer race
> >   dm-delay: change locking to avoid contention
> >   dm-delay: remove timer_lock
> > 
> >  drivers/md/dm-delay.c | 52 +++++++++++++++++++++++--------------------
> >  1 file changed, 28 insertions(+), 24 deletions(-)
> > 
> > -- 
> > 2.45.0
> > 
> > 
> 
> Hey Ben,
> 
> I just staged 5 dm-delay patches in dm-6.10. 1 from Joel and 4 from
> you.  Please confirm this is what you expected and looks good:
> https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git/log/?h=dm-6.10

Well, I was half expecting to be told that it's kinda pointless to
optimize a target whoses purpose is to slow things down, but yeah, this
matches my dev branch.

Thanks.
-Ben

> 
> Thanks,
> Mike


      reply	other threads:[~2024-05-09 16:07 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-07 21:16 [PATCH 0/3] dm-delay synchronization cleanup Benjamin Marzinski
2024-05-07 21:16 ` [PATCH 1/3] dm-delay: fix workqueue delay_timer race Benjamin Marzinski
2024-05-07 21:16 ` [PATCH 2/3] dm-delay: change locking to avoid contention Benjamin Marzinski
2024-05-07 21:16 ` [PATCH 3/3] dm-delay: remove timer_lock Benjamin Marzinski
2024-05-09 14:24 ` [PATCH 0/3] dm-delay synchronization cleanup Mike Snitzer
2024-05-09 16:07   ` Benjamin Marzinski [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Zjz0vkYQSMPxHFqz@redhat.com \
    --to=bmarzins@redhat.com \
    --cc=dm-devel@lists.linux.dev \
    --cc=joel.colledge@linbit.com \
    --cc=mauelshagen@redhat.com \
    --cc=mpatocka@redhat.com \
    --cc=snitzer@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.