From: Takashi Sakamoto <o-takashi@sakamocchi.jp>
To: Edmund Raile <edmund.raile@protonmail.com>
Cc: tiwai@suse.com, linux-sound@vger.kernel.org, stable@vger.kernel.org
Subject: Re: [PATCH v2 0/2] ALSA: firewire-lib: restore process context workqueue to prevent deadlock
Date: Mon, 29 Jul 2024 09:27:25 +0900 [thread overview]
Message-ID: <20240729002725.GA33722@workstation.local> (raw)
In-Reply-To: <20240728122614.329544-1-edmund.raile@protonmail.com>
Hi,
On Sun, Jul 28, 2024 at 12:26:21PM +0000, Edmund Raile wrote:
> This patchset serves to prevent a deadlock between
> process context and softIRQ context:
(snip)
> Edmund Raile (2):
> ALSA: firewire-lib: restore workqueue for process context
> ALSA: firewire-lib: prevent deadlock between process and softIRQ
> context
>
> sound/firewire/amdtp-stream.c | 36 ++++++++++++++++++++++-------------
> sound/firewire/amdtp-stream.h | 1 +
> 2 files changed, 24 insertions(+), 13 deletions(-)
Thank you for your sending the revised patches, it looks better than the
previous one. However, I have an additional request.
In this case, it is enough to execute 'revert' subcommand[1] of git(1),
like:
$ git revert -s b5b519965c4c
$ git revert -s 7ba5ca32fe6e
It is permitted to add postscript to the commit comment generated by the
above command. You see my recent post as an example[2].
Just for safe, it is preferable to execute 'scripts/checkpatch.pl' in
kernel tree to check the patchset generated by send-email subcommand[3].
[1] https://git-scm.com/docs/git-revert
[2] https://lore.kernel.org/lkml/20240725161648.130404-1-o-takashi@sakamocchi.jp/
[3] https://git-scm.com/docs/git-send-email
Thanks
Takashi Sakamoto
next prev parent reply other threads:[~2024-07-29 0:27 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-28 12:26 [PATCH v2 0/2] ALSA: firewire-lib: restore process context workqueue to prevent deadlock Edmund Raile
2024-07-28 12:26 ` [PATCH v2 1/2] ALSA: firewire-lib: restore workqueue for process context Edmund Raile
2024-07-28 12:26 ` [PATCH v2 2/2] ALSA: firewire-lib: prevent deadlock between process and softIRQ context Edmund Raile
2024-07-29 0:27 ` Takashi Sakamoto [this message]
-- strict thread matches above, loose matches on Subject: below --
2024-07-28 12:30 [PATCH v2 0/2] ALSA: firewire-lib: restore process context workqueue to prevent deadlock Edmund Raile
2024-07-29 10:16 edmund.raile
2024-07-29 13:33 ` Takashi Sakamoto
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=20240729002725.GA33722@workstation.local \
--to=o-takashi@sakamocchi.jp \
--cc=edmund.raile@protonmail.com \
--cc=linux-sound@vger.kernel.org \
--cc=stable@vger.kernel.org \
--cc=tiwai@suse.com \
/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.