From: Stefan Agner <stefan@agner.ch>
To: Oliver Neukum <oneukum@suse.com>
Cc: gregKH@linuxfoundation.org, linux-usb@vger.kernel.org
Subject: Re: [PATCH] CDC-ACM: fix poison/unpoison imbalance
Date: Thu, 22 Apr 2021 13:35:30 +0200 [thread overview]
Message-ID: <ffd33d14769dd51375c87d43fff9cc3d@agner.ch> (raw)
In-Reply-To: <20210421074513.4327-1-oneukum@suse.com>
On 2021-04-21 09:45, Oliver Neukum wrote:
> suspend() does its poisoning conditionally, resume() does it
> unconditionally. On a device with combined interfaces this
> will balance, on a device with two interfaces the counter will
> go negative and resubmission will fail.
>
> Both actions need to be done conditionally.
FWIW, this relates to the issue reported on the kernel bugzilla:
https://bugzilla.kernel.org/show_bug.cgi?id=212725
Thanks for addressing this!
>
> Fixes: 6069e3e927c8f ("USB: cdc-acm: untangle a circular dependency
> between callback and softint")
> Signed-off-by: Oliver Neukum <oneukum@suse.com>
Tested-by: Stefan Agner <stefan@agner.ch>
--
Stefan
> ---
> drivers/usb/class/cdc-acm.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c
> index 3fda1ec961d7..f5886c512fec 100644
> --- a/drivers/usb/class/cdc-acm.c
> +++ b/drivers/usb/class/cdc-acm.c
> @@ -1634,12 +1634,13 @@ static int acm_resume(struct usb_interface *intf)
> struct urb *urb;
> int rv = 0;
>
> - acm_unpoison_urbs(acm);
> spin_lock_irq(&acm->write_lock);
>
> if (--acm->susp_count)
> goto out;
>
> + acm_unpoison_urbs(acm);
> +
> if (tty_port_initialized(&acm->port)) {
> rv = usb_submit_urb(acm->ctrlurb, GFP_ATOMIC);
prev parent reply other threads:[~2021-04-22 11:42 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-21 7:45 [PATCH] CDC-ACM: fix poison/unpoison imbalance Oliver Neukum
2021-04-22 11:35 ` Stefan Agner [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=ffd33d14769dd51375c87d43fff9cc3d@agner.ch \
--to=stefan@agner.ch \
--cc=gregKH@linuxfoundation.org \
--cc=linux-usb@vger.kernel.org \
--cc=oneukum@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.