Hello,
I am the person who received the patch. Sorry, but I still have the hang.
The last entries in my syslog:
Jul 16 16:56:16 (none) kernel: pwc Image is mirrored.
Jul 16 16:56:16 (none) kernel: pwc Frame buffer underflow (2 bytes);
discarded.
Jul 16 16:56:16 (none) kernel: pwc Image is
With kernel 2.5.70 and kernel 2.6.0-test1 I did exactly the same actions. Both kernels
have the same (as similar as possible) configuration. I tested in both cases directly
after booting into Runlevel 3 without doing anything before my test.
Here the /proc/bus/usb/devices of kernel 2.5.70 after
On 17 Jul 2003, Daniel Stöckner wrote:
With kernel 2.5.70 and kernel 2.6.0-test1 I did exactly the same
actions. Both kernels have the same (as similar as possible)
configuration. I tested in both cases directly after booting into
Runlevel 3 without doing anything before my test.
Here the
Alan Stern wrote:
Can you explain what's going on in this little extract from Harald's
syslog?
usb-storage: bus_reset called
hub 1-0:0: port 2 not reset yet, waiting 10ms
hub 1-0:0: port 2 not reset yet, waiting 10ms
hub 1-0:0: port 2 not reset yet, waiting 200ms
ehci_hcd :00:09.2: port 2
Hi,
On Thu, Jul 17, 2003 at 07:21:35PM +0400, Sergey Vlasov wrote:
On Tue, 15 Jul 2003 19:44:42 +0200
Henning Meier-Geinitz [EMAIL PROTECTED] wrote:
- kmalloc error handling in probe_scanner() does not unlink the IRQ
URB submitted previously.
True. Shouldn't the URB also be freed
On Tue, 15 Jul 2003 19:44:42 +0200
Henning Meier-Geinitz [EMAIL PROTECTED] wrote:
- kmalloc error handling in probe_scanner() does not unlink the IRQ
URB submitted previously.
True. Shouldn't the URB also be freed with usb_free_urb() here?
No - it was not allocated with usb_alloc_urb().
On Wed, 16 Jul 2003, Charles Lepple wrote:
The patch below will log some additional debugging information. It might
help pinpoint where things go wrong during your test. Unfortunately, it
will generate a good deal of output even when things go right. Try to
run just the test that
Alan Stern said:
The information in the -g4 log was helpful. Here's another patch, which
is my attempt to fix the bug. It actually changes two things:
Cool, thanks! No more kernel panics-- my filesystem thanks you. However,
the test still reports failure.
Try this patch and run those tests
Hi,
Clean up irq urb when not enough memory is available (Sergey Vlasov).
Bye,
Henning
--- linux-2.4.22-pre6.nofreeze/drivers/usb/scanner.c2003-07-15 18:31:26.0
+0200
+++ linux-2.4.22-pre6.irq-cleanup/drivers/usb/scanner.c 2003-07-17 17:56:41.0
+0200
@@ -376,6 +376,7 @@
Hi,
Clean up irq urb when not enough memory is available.
Bye,
Henning
--- linux-2.6.0-test1.newids8/drivers/usb/image/scanner.c 2003-07-15
11:41:02.0 +0200
+++ linux-2.6.0-test1.irq-cleanup/drivers/usb/image/scanner.c 2003-07-17
19:10:37.0 +0200
@@ -367,6 +367,7 @@
On Thu, 17 Jul 2003, Charles Lepple wrote:
Alan Stern said:
The information in the -g4 log was helpful. Here's another patch, which
is my attempt to fix the bug. It actually changes two things:
Cool, thanks! No more kernel panics-- my filesystem thanks you. However,
the test still
Alan Stern wrote:
I'm not sure about that last message from usbtest, though. Maybe David
can help explain what's going on.
David -- here's the result of one of Charles's tests with my debugging
output included (and some other stuff removed).
In short: Failed with status 0 means that a control
Hiya - I thought this mgiht interest you:
I am experiencing low memory and system hang with USB-STORAGE, when I try
to transfer bigger files (~400MB) to a USB HDD. At first everything seems
normal, but then transfer becomes very slow, hangs for some time,
continues, but ultimately system locks up.
On Thu, 17 Jul 2003, David Brownell wrote:
Alan Stern wrote:
Remove_list is a list of URBs that were unlinked by the user.
Complete_list is a list of URBs that are ready to be sent to
giveback_urb(), either because they completed or because they were
unlinked. The URB's hcd-private
Greg:
This patch for the UHCI driver changes the test for over current inputs,
which can cause false Resume indications with some of Intel's USB
controllers. Previously the code only checked for one specific controller
chip, but now another one has turned up that exhibits the same flaw. It
On Thu, Jul 17, 2003, Alan Stern [EMAIL PROTECTED] wrote:
This patch for the UHCI driver changes the test for over current inputs,
which can cause false Resume indications with some of Intel's USB
controllers. Previously the code only checked for one specific controller
chip, but now another
Johannes:
A bug has turned up in the UHCI driver. Briefly, when an URB completes,
uhci_transfer_result() doesn't store the status in urb-status but only in
urbp-status. urb-status is set just before calling hcd_giveback_urb().
However, the hcd glue layer uses urb-status == -EINPROGRESS as a
Hi !
I'm trying to write a driver for an usb device. I use usbdevfs to
communicate with it. Unfortunately, I get this message :
Jul 16 21:09:46 rtp kernel: usb-uhci.c: interrupt, status 3, frame# 290
Jul 16 21:09:46 rtp kernel: usbdevfs: USBDEVFS_BULK failed dev 2 ep 0x83
len 26 ret -75
Can
Hi there,
I am trying to make my USB HID device post absolute XY coordinates so it can
be recognized by Linux without writing any additional drivers. I saw an old
reference on this mailing list from Oliver Neukum which mentioned We have
standards on how to present absolute mice. Use them. I have
Alan Stern said:
Another theory would be that the EZ-USB chip is deeply broken,
but I thought they were more reasonable than that. (For all
that they're only 8 bit CPUs!)
Remember, Charles said that he was going through all this to test his
firmware. Isn't it possible that an error in his
The first four subtests (#0..#3) will run in a loop, 5000 times each.
At most four will be queued simultaneously: 0123012301230123.
- get device descriptor (18 bytes)
- get config descriptor (9 bytes, NOT including interfaces, etc)
- get altsetting for interface 0 (one byte, some
On Thu, Jul 17, 2003, Alan Stern [EMAIL PROTECTED] wrote:
A bug has turned up in the UHCI driver. Briefly, when an URB completes,
uhci_transfer_result() doesn't store the status in urb-status but only in
urbp-status. urb-status is set just before calling hcd_giveback_urb().
However, the
On Thu, Jul 17, 2003 at 07:25:08PM +0200, Henning Meier-Geinitz wrote:
Hi,
Clean up irq urb when not enough memory is available.
Applied, thanks.
greg k-h
---
This SF.net email is sponsored by: VM Ware
With VMware you can run multiple
On Thu, Jul 17, 2003 at 05:02:21PM -0400, Alan Stern wrote:
Greg:
This patch for the UHCI driver changes the test for over current inputs,
which can cause false Resume indications with some of Intel's USB
controllers. Previously the code only checked for one specific controller
chip, but
On Wed, Jul 16, 2003 at 08:56:10PM +0200, Oliver Neukum wrote:
Hi,
the driver is registering the device too early, so that open may
see a partially initialised device.
Applied, thanks.
greg k-h
---
This SF.net email is sponsored by: VM
Am Donnerstag, 17. Juli 2003 23:09 schrieb Alan Stern:
Johannes:
A bug has turned up in the UHCI driver. Briefly, when an URB completes,
uhci_transfer_result() doesn't store the status in urb-status but only in
urbp-status. urb-status is set just before calling hcd_giveback_urb().
A while ago I saw that halted, very bad message, and my system
then wedged when rmmod called a driver disconnect which waited
forever in the UHCI code.
What's supposed to happen is that HCDs mark themselves as dead,
and then usbcore shuts them down cleanly. OHCI and EHCI did,
or they did last
Oliver Neukum wrote:
Am Donnerstag, 17. Juli 2003 23:09 schrieb Alan Stern:
Johannes:
A bug has turned up in the UHCI driver. Briefly, when an URB completes,
uhci_transfer_result() doesn't store the status in urb-status but only in
urbp-status. urb-status is set just before calling
Am Freitag, 18. Juli 2003 00:38 schrieb David Brownell:
Oliver Neukum wrote:
Am Donnerstag, 17. Juli 2003 23:09 schrieb Alan Stern:
Johannes:
A bug has turned up in the UHCI driver. Briefly, when an URB completes,
uhci_transfer_result() doesn't store the status in urb-status but only
On Thu, Jul 17, 2003 at 07:23:44PM +0200, Henning Meier-Geinitz wrote:
Hi,
Clean up irq urb when not enough memory is available (Sergey Vlasov).
Applied, thanks.
greg k-h
---
This SF.net email is sponsored by: VM Ware
With VMware you can
From: David Brownell [EMAIL PROTECTED]
Date: Thu, 17 Jul 2003 15:38:11 -0700
A bug has turned up in the UHCI driver. Briefly, when an URB completes,
uhci_transfer_result() doesn't store the status in urb-status but only in
urbp-status. urb-status is set just before calling
Hi,
Here are some USB fixes for 2.6.0-test1. There are a lot of good race
condition fixes from Oliver Neukum, and some usb-storage fixes for some
devices. I've also included the change that flushes all in-flight urbs
_before_ disconnect() is called to fix some problems in the visor and
ftdi_sio
ChangeSet 1.1455.2.5, 2003/07/13 21:44:11-07:00, [EMAIL PROTECTED]
[PATCH] USB: Adding DSS-20 SyncStation to ftdi_sio
Patch for 2.4.22 ftdi_sio to add DSS-20 SyncStation to ftdi_sio - the
SyncStation is the usb cradle for the Sony Ericsson P800 phone.
drivers/usb/serial/ftdi_sio.c |3 +++
ChangeSet 1.1455.1.19, 2003/07/15 14:32:34-07:00, [EMAIL PROTECTED]
[PATCH] USB: flush all in-flight urbs _before_ disconnect() is called.
This solves the module unload problem for some usb-serial drivers
(like visor.c and ftdi_sio.c), and makes usb drivers much simpler.
ChangeSet 1.1455.1.29, 2003/07/15 14:59:26-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix race between open() and probe()
registering the device before setting the interface data can lead
to a null pointer being followed in open(). This patch fixes that.
drivers/usb/class/usblp.c | 34
ChangeSet 1.1455.1.27, 2003/07/15 14:55:31-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix open/probe race in scanner driver
This patch from Oliver Neukum fixes a race between open and
probe.
drivers/usb/image/scanner.c |8 +---
drivers/usb/image/scanner.h |2 +-
2 files changed, 6
ChangeSet 1.1455.1.20, 2003/07/15 14:33:02-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix up bluetty driver's tty and devfs names.
drivers/usb/class/bluetty.c |3 ++-
1 files changed, 2 insertions(+), 1 deletion(-)
diff -Nru a/drivers/usb/class/bluetty.c b/drivers/usb/class/bluetty.c
---
ChangeSet 1.1455.1.31, 2003/07/15 15:04:44-07:00, [EMAIL PROTECTED]
[PATCH] USB: More unusual_devs.h entry updates
Here are more user-reported updates for unusual_devs.h.
drivers/usb/storage/unusual_devs.h |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff -Nru
ChangeSet 1.1455.2.6, 2003/07/13 22:41:37-07:00, [EMAIL PROTECTED]
[PATCH] USB: more ids for ipaq
3 more device ids added to ipaq.c/ipaq.h
Added ids for the Rover P5, Toshiba E310 and E335. Thanks to
Matthijs van der Molen, Pavel Stoliarov, Tod B. Schmidt and
Matt Hartley.
ChangeSet 1.1455.1.23, 2003/07/15 14:53:33-07:00, [EMAIL PROTECTED]
[PATCH] USB: usb net drivers SET_NETDEV_DEV
I noticed that Viro's use alloc_etherdev patch left a
small problem: only usbnet did SET_NETDEV_DEV, and
that used the wrong driver model device.
This has all the usb network drivers
ChangeSet 1.1455.1.32, 2003/07/15 16:20:24-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix a nasty use-after-free bug in the usb-serial core.
drivers/usb/serial/usb-serial.c |2 ++
1 files changed, 2 insertions(+)
diff -Nru a/drivers/usb/serial/usb-serial.c b/drivers/usb/serial/usb-serial.c
ChangeSet 1.1455.1.38, 2003/07/16 10:16:22-07:00, [EMAIL PROTECTED]
[PATCH] USB: I/O buffering for sddr09
This patch makes the sddr09 subdriver use proper DMA I/O buffering.
drivers/usb/storage/sddr09.c | 101 ---
1 files changed, 58 insertions(+), 43
ChangeSet 1.1455.1.18, 2003/07/15 14:32:18-07:00, [EMAIL PROTECTED]
[PATCH] USB: remove some warnings when building the documentation.
drivers/usb/core/hcd-pci.c |1 +
include/linux/usb.h| 22 +-
2 files changed, 18 insertions(+), 5 deletions(-)
diff -Nru
ChangeSet 1.1455.2.7, 2003/07/14 16:07:51-07:00, [EMAIL PROTECTED]
[PATCH] USB: fixed up pci slot_name accesses in usb code
drivers/usb/core/hcd-pci.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff -Nru a/drivers/usb/core/hcd-pci.c b/drivers/usb/core/hcd-pci.c
---
ChangeSet 1.1455.1.34, 2003/07/16 10:11:48-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix irq urb in hpusbscsi
this fixes the completion handler for the interrupt urb in hpusbscsi.
drivers/usb/image/hpusbscsi.c |9 +++--
1 files changed, 7 insertions(+), 2 deletions(-)
diff -Nru
ChangeSet 1.1455.1.39, 2003/07/16 10:16:36-07:00, [EMAIL PROTECTED]
[PATCH] USB: Make sddr55 use proper I/O buffering
This patch makes the sddr55 subdriver use proper DMA I/O buffering.
drivers/usb/storage/sddr55.c | 41 -
1 files changed, 24
ChangeSet 1.1455.1.43, 2003/07/17 15:45:30-07:00, [EMAIL PROTECTED]
[PATCH] USB: better locking in hcd_endpoint_disable()
You'll recall there was a change to the locking in that code after
it was submitted. This is a better fix to that issue.
Obviously correct, though I've not run with this in
ChangeSet 1.1455.1.37, 2003/07/16 10:15:18-07:00, [EMAIL PROTECTED]
[PATCH] USB: usblcd: race between open and read/write
usblcd registers a device before all buffers are allocated leading
to a race resulting in NULL pointers being followed.
This fixes it.
drivers/usb/misc/usblcd.c | 22
ChangeSet 1.1455.1.22, 2003/07/15 14:53:20-07:00, [EMAIL PROTECTED]
[PATCH] USB: ohci minor tweaks
Two small updates:
- Report short control reads correctly in an exotic case
that our regression tests cover. (Haven't run them with
ohci for a long time, it seems...)
- IRQ
ChangeSet 1.1455.1.17, 2003/07/14 16:54:25-07:00, [EMAIL PROTECTED]
[PATCH] USB: remove now-dead mode-translation code
This patch removes all of the mode-translation logic. It's no longer
needed, as the upper-level drivers now all issue the proper 10-byte
commands. This is a large patch, but
ChangeSet 1.1455.1.28, 2003/07/15 14:55:44-07:00, [EMAIL PROTECTED]
[PATCH] USB: New vendor/product ids for scanner driver
this patch adds new vendor/product ids for Avision, Canon, HP,
Microtek and Relisys scanners.
drivers/usb/image/scanner.c |1 +
drivers/usb/image/scanner.h |6
ChangeSet 1.1455.1.21, 2003/07/15 14:33:37-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix up cdc-acm driver's tty and devfs names.
drivers/usb/class/cdc-acm.c |3 ++-
1 files changed, 2 insertions(+), 1 deletion(-)
diff -Nru a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c
---
ChangeSet 1.1455.1.41, 2003/07/17 14:46:56-07:00, [EMAIL PROTECTED]
[PATCH] USB: unlink interrupt URBs in scanner driver
Clean up irq urb when not enough memory is available.
drivers/usb/image/scanner.c |7 +++
1 files changed, 7 insertions(+)
diff -Nru a/drivers/usb/image/scanner.c
ChangeSet 1.1455.1.35, 2003/07/16 10:12:00-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix race between probe and open in skeleton
registering a device only partially initialised is quite bad an
idea.
drivers/usb/usb-skeleton.c | 37 +++--
1 files changed, 15
ChangeSet 1.1455.1.33, 2003/07/15 22:33:17-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix memory leak in the visor driver.
drivers/usb/serial/visor.c | 13 ++---
1 files changed, 6 insertions(+), 7 deletions(-)
diff -Nru a/drivers/usb/serial/visor.c b/drivers/usb/serial/visor.c
---
ChangeSet 1.1455.1.42, 2003/07/17 14:47:19-07:00, [EMAIL PROTECTED]
[PATCH] USB: fix race between probe and open in dabusb
the driver is registering the device too early, so that open may
see a partially initialised device.
drivers/usb/media/dabusb.c | 15 ++-
1 files changed,
ChangeSet 1.1455.1.40, 2003/07/17 14:46:26-07:00, [EMAIL PROTECTED]
[PATCH] USB: Handle over current inputs on all Intel controllers
This patch for the UHCI driver changes the test for over current inputs,
which can cause false Resume indications with some of Intel's USB
controllers. Previously
ChangeSet 1.1455.1.25, 2003/07/15 14:53:55-07:00, [EMAIL PROTECTED]
[PATCH] USB: usbtest, autoconfigure from descriptors
Not all gadget controllers support ep2in-bulk and ep2out-bulk,
unlike the EZ-USB (non-FX2) devices or the net2280.
So this patch teaches usbtest how to look at the
ChangeSet 1.1455.1.16, 2003/07/14 16:54:12-07:00, [EMAIL PROTECTED]
[PATCH] USB: convert ISD200 and Jumpshot to DMA-safe buffer
This patch converts the ISD200 and Jumpshot drivers to use the new DMA-safe
buffer.
I'd also like to take a moment to thank Alan Stern, who writes many of
these
ChangeSet 1.1455.1.36, 2003/07/16 10:12:14-07:00, [EMAIL PROTECTED]
[PATCH] USB: ethernet gadget, another pxa update
- #ifdefs out some code that never runs on the pxa
- catches alloc_etherdev Jihad changes I somehow missed
drivers/usb/gadget/ether.c | 18 +++---
1 files
ChangeSet 1.1455.1.24, 2003/07/15 14:53:43-07:00, [EMAIL PROTECTED]
[PATCH] USB: ethernet gadget learns about pxa2xx udc
The original code needed some updates to work properly with
the USB Device Controller found in Intel's PXA 2xx processors.
- SET_INTERFACE is effectively unusable. So the
ChangeSet 1.1455.2.8, 2003/07/14 16:08:00-07:00, [EMAIL PROTECTED]
[PATCH] USB: fixed up pci slot_name accesses in usb gadget code
drivers/usb/gadget/net2280.c |6 +++---
drivers/usb/gadget/net2280.h |2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff -Nru
ChangeSet 1.1455.1.26, 2003/07/15 14:54:08-07:00, [EMAIL PROTECTED]
[PATCH] USB: gadget zero learns about pxa2xx udc
The original code needed some updates to work properly with
the USB Device Controller found in Intel's PXA 2xx processors.
- SET_INTERFACE and SET_CONFIGURATION involve
However, the hcd glue layer uses urb-status == -EINPROGRESS as a test for
It shouldn't use it. The clean way to fix this issue is to have an explicit
state field.
But urb-status ** IS ** an explicit state field.
It holds the first fault experienced by that request. One of
those fault codes is a
On Thu, 17 Jul 2003, David Brownell wrote:
OK, so it's possible that some of these problems start because the
chip is making a nonsensical response ... perhaps not unexpected,
since it wasn't actually running firmware at that point.
Since the test program is intended for detecting
On Thu, 17 Jul 2003, Charles Lepple wrote:
The firmware that I am loading is actually the ep2_inout firmware. (I am
trying to establish that the USB physical layer is sound, and I assume
that the EZ-USB chip is functional.)
I just finished testing with several other chips, and I can confirm
Date: Thu, 17 Jul 2003 18:39:38 -0700
From: David Brownell [EMAIL PROTECTED]
So the completion uses an extra field to indicate that
urb was released by HC driver to the uppper level.
It's not an explicit field, it's an extra field.
This really makes no sense to me. There's a field,
On Thu, 17 Jul 2003, Johannes Erdfelt wrote:
On Thu, Jul 17, 2003, Alan Stern [EMAIL PROTECTED] wrote:
A bug has turned up in the UHCI driver. Briefly, when an URB completes,
uhci_transfer_result() doesn't store the status in urb-status but only in
urbp-status. urb-status is set just
On Thu, 17 Jul 2003, David Brownell wrote:
Oliver responded:
Right, it holds information about the status of the io operation the URB
describes. It doesn't hold information about the status of the URB itself,
such as unlinkable, ununlinkable, running completion handler, unlinked
These
On Thu, 17 Jul 2003, Pete Zaitcev wrote:
The urb-lock is situated in the same chunk of storage
as the urb-status, so it only can protect the status if
there is no hazard of deallocation of the urb. However, our
API makes no such guarantees across the call to the -complete.
Therefore,
Date: Thu, 17 Jul 2003 23:01:42 -0400 (EDT)
From: Alan Stern [EMAIL PROTECTED]
[...] But a properly-written driver won't
touch the URB _at all_ once it has been submitted until the completion
callback.
I said the same. It cannot look at the urb-status.
-- Pete
Alan Stern wrote:
On Thu, 17 Jul 2003, David Brownell wrote:
OK, so it's possible that some of these problems start because the
chip is making a nonsensical response ... perhaps not unexpected,
since it wasn't actually running firmware at that point.
Since the test program is intended for
Yes, sorry to have slowed down with replies. 2.4.20 (or maybe 21?) log
while copying:
http://bostoncoop.net/adam/pub/log_while_copying
when trying to unmount
http://bostoncoop.net/adam/pub/unmount_log
--Adam
On Mon, Jul 14, 2003 at 06:35:33PM -0700, Matthew Dharm wrote:
Did you ever get
Pete Zaitcev wrote:
Date: Thu, 17 Jul 2003 18:39:38 -0700
From: David Brownell [EMAIL PROTECTED]
So the completion uses an extra field to indicate that
urb was released by HC driver to the uppper level.
It's not an explicit field, it's an extra field.
This really makes no sense to me. There's a
ChangeSet 1.1455.1.18, 2003/07/15 14:32:18-07:00, [EMAIL PROTECTED]
[PATCH] USB: remove some warnings when building the documentation.
--- a/include/linux/usb.h Thu Jul 17 17:05:19 2003
+++ b/include/linux/usb.h Thu Jul 17 17:05:19 2003
@@ -575,6 +580,8 @@
* it likes with the URB, including
On Thu, Jul 17, 2003 at 09:06:00PM -0700, David Brownell wrote:
ChangeSet 1.1455.1.18, 2003/07/15 14:32:18-07:00, [EMAIL PROTECTED]
[PATCH] USB: remove some warnings when building the documentation.
--- a/include/linux/usb.hThu Jul 17 17:05:19 2003
+++ b/include/linux/usb.hThu
On Thu, Jul 17, 2003 at 11:49:08PM -0400, Adam Kessel wrote:
Yes, sorry to have slowed down with replies. 2.4.20 (or maybe 21?) log
while copying:
http://bostoncoop.net/adam/pub/log_while_copying
What command did you run to generate this log?
Matt
--
Matthew Dharm
77 matches
Mail list logo