From: Greg KH <gregkh@linuxfoundation.org>
To: Pascal Giard <pascal.giard@etsmtl.ca>
Cc: Jiri Kosina <jikos@kernel.org>,
Benjamin Tissoires <benjamin.tissoires@redhat.com>,
linux-input@vger.kernel.org, linux-kernel@vger.kernel.org,
stable@vger.kernel.org,
Daniel Nguyen <daniel.nguyen.1@ens.etsmtl.ca>
Subject: Re: [PATCH] HID: sony: fix freeze when inserting ghlive ps3/wii dongles
Date: Sat, 5 Jun 2021 08:43:30 +0200 [thread overview]
Message-ID: <YLsdEtbAWJxLB+GF@kroah.com> (raw)
In-Reply-To: <20210604161023.1498582-1-pascal.giard@etsmtl.ca>
On Fri, Jun 04, 2021 at 12:10:23PM -0400, Pascal Giard wrote:
> This commit fixes a freeze on insertion of a Guitar Hero Live PS3/WiiU
> USB dongle. Indeed, with the current implementation, inserting one of
> those USB dongles will lead to a hard freeze. I apologize for not
> catching this earlier, it didn't occur on my old laptop.
>
> While the issue was isolated to memory alloc/free, I could not figure
> out why it causes a freeze. So this patch fixes this issue by
> simplifying memory allocation and usage.
>
> We remind that for the dongle to work properly, a control URB needs to
> be sent periodically. We used to alloc/free the URB each time this URB
> needed to be sent.
>
> With this patch, the memory for the URB is allocated on the probe, reused
> for as long as the dongle is plugged in, and freed once the dongle is
> unplugged.
>
> Signed-off-by: Pascal Giard <pascal.giard@etsmtl.ca>
> ---
> drivers/hid/hid-sony.c | 98 +++++++++++++++++++++---------------------
> 1 file changed, 49 insertions(+), 49 deletions(-)
<formletter>
This is not the correct way to submit patches for inclusion in the
stable kernel tree. Please read:
https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html
for how to do this properly.
</formletter>
next prev parent reply other threads:[~2021-06-05 6:43 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-04 16:10 [PATCH] HID: sony: fix freeze when inserting ghlive ps3/wii dongles Pascal Giard
2021-06-05 6:43 ` Greg KH [this message]
2021-06-10 0:25 ` Pascal Giard
2021-06-10 5:22 ` Greg KH
2021-06-10 13:53 ` Pascal Giard
2021-06-15 8:53 ` Jiri Kosina
2021-07-13 19:08 ` Pascal Giard
2021-07-15 19:43 ` Jiri Kosina
2021-07-20 8:17 ` Benjamin Tissoires
2021-07-20 14:04 ` Pascal Giard
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=YLsdEtbAWJxLB+GF@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=benjamin.tissoires@redhat.com \
--cc=daniel.nguyen.1@ens.etsmtl.ca \
--cc=jikos@kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pascal.giard@etsmtl.ca \
--cc=stable@vger.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.