From: Tony Hoyle <tmh@magenta-netlogic.com>
To: linux-kernel@vger.kernel.org, Alan Cox <alan@lxorguk.ukuu.org.uk>
Subject: Resend - [PATCH] Fix SMP lockup in usbdevfs
Date: Wed, 04 Apr 2001 16:47:16 +0100 [thread overview]
Message-ID: <3ACB4204.9080108@magenta-netlogic.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 741 bytes --]
This one didn't quite make 2.4.3, this time I've CC'd to AC.
I've been using this fix for a few days now & it's cleared up a lot of
problems - although I'm not 100% sure why it worked (the memset should
do the same job as the spin_lock_init surely?).
Tony
-------- Original Message --------
Subject: [PATCH] Fix SMP lockup in usbdevfs
Date: Fri, 30 Mar 2001 02:36:47 +0100
From: Tony Hoyle <tmh@magenta-netlogic.com>
Organization: Magenta Logic
To: linux-kernel@vger.kernel.org
CC: linux-usb-devel@lists.sourceforge.net
This fixes a lockup when calling the USBDEVFS_SUBMITURB ioctl in an SMP
kernel.
Tony
--
Don't click on this sig - a cyberwoozle will eat your underwear.
tmh@magenta-netlogic.com http://www.nothing-on.tv
[-- Attachment #2: devio.patch --]
[-- Type: text/plain, Size: 552 bytes --]
--- devio.c.old Fri Mar 30 02:22:32 2001
+++ devio.c Fri Mar 30 02:12:09 2001
@@ -175,6 +175,7 @@
return NULL;
memset(as, 0, assize);
as->urb.number_of_packets = numisoframes;
+ spin_lock_init(&as->urb.lock);
return as;
}
@@ -250,7 +251,7 @@
struct dev_state *ps = as->ps;
struct siginfo sinfo;
-#if 1
+#if 0
printk(KERN_DEBUG "usbdevfs: async_completed: status %d errcount %d actlen %d pipe 0x%x\n",
urb->status, urb->error_count, urb->actual_length, urb->pipe);
#endif
reply other threads:[~2001-04-04 15:48 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=3ACB4204.9080108@magenta-netlogic.com \
--to=tmh@magenta-netlogic.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox