Re: [linux-usb-devel] [Bugme-new] [Bug 8310] New: USB device names are not sanitized for UTF-8

2007-06-05 Thread Oliver Neukum
Am Montag, 4. Juni 2007 22:52 schrieb Alan Stern: Regarding: http://bugzilla.kernel.org/show_bug.cgi?id=8310 Does anybody think it would be worthwhile to convert string descriptors from UCS-16 to UTF-8 (instead of Latin1) when we read them in? Yes, I do. We should either pass them

[linux-usb-devel] file_storage gadget requires a sync...

2007-06-05 Thread Guennadi Liakhovetski
Hi I see a ~100% reproducible problem under 2.6.20 with g_file_storage: the script cp fs.image /tmp/ modprobe g_file_storage file=/tmp/fs.image comes with no error back, but the enumeration fails. Whereas inserting a sync between the cp and modprobe fixes it... It worked without sync under

Re: [linux-usb-devel] [Bugme-new] [Bug 8564] New: ftdi_sio: BUG: unable to handle kernel NULL pointer dereference at virtual address

2007-06-05 Thread Oliver Neukum
Am Montag, 4. Juni 2007 21:57 schrieb John H.: Sorry, I can't really try it. I have fedora's 2.6.21 kernel and I have to leave the machine working for 2 months, so I can't experiment:) However, I did try 2.6.21.3 and it had same problem. Can you please post the oops with that version?

[linux-usb-devel] sleep with spinlock held detected by automatic tool

2007-06-05 Thread Oliver Neukum
Hi, this fixes the sleep found with the automatic tool. Regards Oliver Signed-off-by: Oliver Neukum [EMAIL PROTECTED] -- --- a/drivers/usb/serial/io_ti.c2007-06-05 10:35:26.0 +0200 +++ b/drivers/usb/serial/io_ti.c2007-06-05 10:35:34.0

Re: [linux-usb-devel] [PATCH] bugfix GFP_KERNEL - GFP_ATOMIC in spin_locked region

2007-06-05 Thread Oliver Neukum
Am Dienstag, 5. Juni 2007 06:08 schrieb Andrew Morton: Everything in USB appears to already be fixed, apart from the io_ti.c bug. Yes, that's a bug. I've queued a patch. Regards Oliver - This SF.net

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Oliver Neukum
Am Montag, 4. Juni 2007 16:44 schrieb Marcel Holtmann: Hi Oliver, Hello, an alternate way would be to extend the usb_kill_anchored_urbs() with its own complete handler that gets called for every anchored URB. This would make it possible to cleanup the allocated buffers. I attached a

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Marcel Holtmann
Hi Oliver, an alternate way would be to extend the usb_kill_anchored_urbs() with its own complete handler that gets called for every anchored URB. This would make it possible to cleanup the allocated buffers. I attached a patch for that, too. No, that would mean an URB

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Oliver Neukum
Am Dienstag, 5. Juni 2007 14:43 schrieb Marcel Holtmann: Hello, To avoid confusion we could rename it to usb_release_t or something similar. One usage is a bit weak to add common code. this is why I wanna have a general solution for one-shot buffers. I understand and concurr, but

Re: [linux-usb-devel] [PATCH] Fix NEC OHCI chip silicon bug

2007-06-05 Thread Alan Stern
On Mon, 4 Jun 2007, Michael Hanselmann wrote: On Sat, Jun 02, 2007 at 10:49:44AM -0400, Alan Stern wrote: Then later on, when the hub driver is resumed it will see the flag and disconnect all the devices below the root hub. For instance, you could force ohci-hcd to report connect-change

Re: [linux-usb-devel] file_storage gadget requires a sync...

2007-06-05 Thread Alan Stern
On Tue, 5 Jun 2007, Guennadi Liakhovetski wrote: Hi I see a ~100% reproducible problem under 2.6.20 with g_file_storage: the script cp fs.image /tmp/ modprobe g_file_storage file=/tmp/fs.image comes with no error back, but the enumeration fails. Whereas inserting a sync between the

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Pete Zaitcev
On Tue, 5 Jun 2007 15:29:12 +0200, Oliver Neukum [EMAIL PROTECTED] wrote: Perhaps you need to have two flags, for freeing unconditionally and on condition of no errors having happened. If you free unconditionally handling errors, eg. clearing a stall becomes hard. This assertion makes no

Re: [linux-usb-devel] file_storage gadget requires a sync...

2007-06-05 Thread Uncle George
Alan Stern wrote: Yes, of course we always write device drivers so that they will fail in unexpected ways unless the user does something unintuitive and undocumented. :-) Ya, we all knew that. Just so long as you understand that we users all read the hardware docs from cover to cover,

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Pete Zaitcev
On Mon, 04 Jun 2007 10:38:44 +0200, Marcel Holtmann [EMAIL PROTECTED] wrote: static void urb_destroy(struct kref *kref) { struct urb *urb = to_urb(kref); + + if (urb-transfer_flags URB_FREE_BUFFER) + usb_buffer_free(urb-dev, urb-transfer_buffer_length, +

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Marcel Holtmann
Hi Pete, static void urb_destroy(struct kref *kref) { struct urb *urb = to_urb(kref); + + if (urb-transfer_flags URB_FREE_BUFFER) + usb_buffer_free(urb-dev, urb-transfer_buffer_length, + urb-transfer_buffer, urb-transfer_dma); +

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Marcel Holtmann
Hi Pete, Perhaps you need to have two flags, for freeing unconditionally and on condition of no errors having happened. If you free unconditionally handling errors, eg. clearing a stall becomes hard. This assertion makes no sense to me. URBs are not freed or put by magic by the core

Re: [linux-usb-devel] file_storage gadget requires a sync...

2007-06-05 Thread Guennadi Liakhovetski
On Tue, 5 Jun 2007, Alan Stern wrote: On Tue, 5 Jun 2007, Guennadi Liakhovetski wrote: I see a ~100% reproducible problem under 2.6.20 with g_file_storage: the script cp fs.image /tmp/ modprobe g_file_storage file=/tmp/fs.image comes with no error back, but the enumeration

Re: [linux-usb-devel] [Bugme-new] [Bug 8561] New: list_add corruption. prev-next should be next (f7d28794), but was f0df8ed4

2007-06-05 Thread Alan Stern
On Mon, 4 Jun 2007, Paulo Pereira wrote: The patch that you send is not resolving the problem... :( I stil have Kernel panic after 45/60 min of work with Ktorrent/Amule... The Drump is: Call Trace: [c055fb36] usb_hcd_submit+0xb1/0x763 [f9276488] ipt_do_table+0x2c7/0x2ef [ip_tables]

[linux-usb-devel] (no subject)

2007-06-05 Thread Yury Kazakevich
- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Pete Zaitcev
On Tue, 05 Jun 2007 17:26:24 +0200, Marcel Holtmann [EMAIL PROTECTED] wrote: The usb_buffer_alloc() and usb_buffer_free() usage was taken from the skeleton example driver. No idea if that is a good idea or not. I was under the impression that using the provided helper function that take care

Re: [linux-usb-devel] file_storage gadget requires a sync...

2007-06-05 Thread Alan Stern
On Tue, 5 Jun 2007, Guennadi Liakhovetski wrote: When you say the enumeration fails, what exactly goes wrong? I've never tried using g_file_storage with a newly-created backing file. Ok, the host says: usb usb1: wakeup_rh (auto-start) hub 1-0:1.0: state 7 ports 2 chg evt 0004

Re: [linux-usb-devel] file_storage gadget requires a sync...

2007-06-05 Thread Guennadi Liakhovetski
On Tue, 5 Jun 2007, Alan Stern wrote: On Tue, 5 Jun 2007, Guennadi Liakhovetski wrote: usb usb1: wakeup_rh (auto-start) hub 1-0:1.0: state 7 ports 2 chg evt 0004 uhci_hcd :00:1f.2: port 2 portsc 0093,00 hub 1-0:1.0: port 2, status 0101, change 0001, 12 Mb/s hub 1-0:1.0:

Re: [linux-usb-devel] [RFC] URBs and buffer management

2007-06-05 Thread Marcel Holtmann
Hi Pete, The usb_buffer_alloc() and usb_buffer_free() usage was taken from the skeleton example driver. No idea if that is a good idea or not. I was under the impression that using the provided helper function that take care of DMA is a good thing. Just as I suspected. You know, if

Re: [linux-usb-devel] file_storage gadget requires a sync...

2007-06-05 Thread Alan Stern
On Tue, 5 Jun 2007, Guennadi Liakhovetski wrote: This looks like it has nothing to do with using sync or newly-created backing files. Instead it resembles a problem for which a patch was recently submitted: http://marc.info/?l=linux-usb-develm=118053809808501w=2 Emn, how

[linux-usb-devel] usb 2.0 driver performance issue

2007-06-05 Thread manoj mathai
hi, I am Manoj from Mumbai, India. I am having problems with my USB 2.0 pen drive on linux. i am having the same problem on almost all linux distros i tried and i think it might have something to do with the linux usb drivers. i am not sure though, please correct me if i am wrong. i don't mind a

[linux-usb-devel] You can find out how to do it.

2007-06-05 Thread Sales
When did you have sex last time? Yesterday? No?! Last week? No?! Can't have sex at all, because of the problem with an erection? So what are you waiting for, order our breakthrough medicine especially created for such desperate boys as you on our website: http://www.laserintl.com/, now and

Re: [linux-usb-devel] usb 2.0 driver performance issue

2007-06-05 Thread Alan Stern
On Tue, 5 Jun 2007, manoj mathai wrote: hi, I am Manoj from Mumbai, India. I am having problems with my USB 2.0 pen drive on linux. i am having the same problem on almost all linux distros i tried and i think it might have something to do with the linux usb drivers. i am not sure though,

Re: [linux-usb-devel] [4/5] 2.6.22-rc4: known regressions

2007-06-05 Thread Michal Piotrowski
Hi all, Here is a list of some known regressions in 2.6.22-rc4. Feel free to add new regressions/remove fixed etc. http://kernelnewbies.org/known_regressions SATA/PATA Subject: 22-rc3 broke the CDROM in Dell notebook References : http://lkml.org/lkml/2007/5/27/63 Submitter : Gregor

Re: [linux-usb-devel] [5/5] 2.6.22-rc4: known regressions

2007-06-05 Thread Justin Piszcz
Hello, The 2.6.22-rc3 USB issue was because I did not have the deprecated USB option enabled. Once I enabled it, the problem went away. That can be bug can be considered resolved. Justin. On Tue, 5 Jun 2007, Michal Piotrowski wrote: Hi all, Here is a list of some known regressions in

Re: [linux-usb-devel] [4/5] 2.6.22-rc4: known regressions

2007-06-05 Thread James Morris
On Tue, 5 Jun 2007, Michal Piotrowski wrote: SELinux Subject: very high non-preempt latency in context_struct_compute_av() References : http://lkml.org/lkml/2007/6/4/78 Submitter : Ingo Molnar [EMAIL PROTECTED] Handled-By : Stephen Smalley [EMAIL PROTECTED] James Morris

Re: [linux-usb-devel] [5/5] 2.6.22-rc4: known regressions

2007-06-05 Thread Greg KH
On Tue, Jun 05, 2007 at 04:44:09PM +0200, Michal Piotrowski wrote: Subject: usb hotplug/udev cannot correctly register usb/scanners References : http://lkml.org/lkml/2007/5/15/205 Submitter : [EMAIL PROTECTED] [EMAIL PROTECTED] Status : Unknown Art seems to not be responding

Re: [linux-usb-devel] [5/5] 2.6.22-rc4: known regressions

2007-06-05 Thread Michal Piotrowski
Hi all, Here is a list of some known regressions in 2.6.22-rc4. Feel free to add new regressions/remove fixed etc. http://kernelnewbies.org/known_regressions USB Subject: Unable to get HID descriptor (error sending control message: Operation not permitted) References :

Re: [linux-usb-devel] [5/5] 2.6.22-rc4: known regressions

2007-06-05 Thread Michal Piotrowski
On 05/06/07, Greg KH [EMAIL PROTECTED] wrote: On Tue, Jun 05, 2007 at 04:44:09PM +0200, Michal Piotrowski wrote: Subject: usb hotplug/udev cannot correctly register usb/scanners References : http://lkml.org/lkml/2007/5/15/205 Submitter : [EMAIL PROTECTED] [EMAIL PROTECTED]

Re: [linux-usb-devel] [5/5] 2.6.22-rc4: known regressions

2007-06-05 Thread Greg KH
On Tue, Jun 05, 2007 at 07:02:53PM +0200, Michal Piotrowski wrote: On 05/06/07, Greg KH [EMAIL PROTECTED] wrote: On Tue, Jun 05, 2007 at 04:44:09PM +0200, Michal Piotrowski wrote: Subject: usb hotplug/udev cannot correctly register usb/scanners References :

Re: [linux-usb-devel] [5/5] 2.6.22-rc4: known regressions

2007-06-05 Thread Michal Piotrowski
On 05/06/07, Greg KH [EMAIL PROTECTED] wrote: On Tue, Jun 05, 2007 at 07:02:53PM +0200, Michal Piotrowski wrote: On 05/06/07, Greg KH [EMAIL PROTECTED] wrote: On Tue, Jun 05, 2007 at 04:44:09PM +0200, Michal Piotrowski wrote: Subject: usb hotplug/udev cannot correctly register

Re: [linux-usb-devel] Large writes issue with Sierra Wireless modem

2007-06-05 Thread Alan Stern
On Tue, 5 Jun 2007, James Graves wrote: Hello all, We're seeing a strange problem when writing commands to a Sierra Wireless USB modem. It is a USB 1.1 device. We're sending some commands over the control channel for the modem. Almost everything works fine. However, when sending

[linux-usb-devel] Large writes issue with Sierra Wireless modem

2007-06-05 Thread James Graves
Hello all, We're seeing a strange problem when writing commands to a Sierra Wireless USB modem. It is a USB 1.1 device. We're sending some commands over the control channel for the modem. Almost everything works fine. However, when sending commands larger than 64 bytes, the rest of the

Re: [linux-usb-devel] [Bugme-new] [Bug 8310] New: USB device names are not sanitized for UTF-8

2007-06-05 Thread Alan Stern
On Mon, 4 Jun 2007, David Brownell wrote: On Monday 04 June 2007, Pete Zaitcev wrote: On Mon, 4 Jun 2007 16:52:01 -0400 (EDT), Alan Stern [EMAIL PROTECTED] wrote: Does anybody think it would be worthwhile to convert string descriptors from UCS-16 to UTF-8 (instead of Latin1) when

Re: [linux-usb-devel] [PATCH 11/12] drivers: PMC MSP71xx USB driver

2007-06-05 Thread Alan Stern
On Mon, 4 Jun 2007, Marc St-Jean wrote: [PATCH 11/12] drivers: PMC MSP71xx USB driver Patch to add an USB driver for the PMC-Sierra MSP71xx devices. Patches 1 through 10 were posted to [EMAIL PROTECTED] as well as other sub-system lists/maintainers as appropriate. This patch has some

[linux-usb-devel] [PATCH 2.6.21.3] USB: add support for TRU-install and new VID/PIDs to Sierra Wireless driver

2007-06-05 Thread Kevin Lloyd
From: Kevin Lloyd [EMAIL PROTECTED] This patch is derived from the 2.6.21.3 kernel source and adds support for the new TRU-install feature (without this support new devices will not work), and add new UMTS device VID/PIDs. Signed-off-by: Kevin Lloyd [EMAIL PROTECTED] --- diff -uprN

Re: [linux-usb-devel] [Bugme-new] [Bug 8310] New: USB device names are not sanitized for UTF-8

2007-06-05 Thread David Brownell
Date: Tue, 5 Jun 2007 17:00:56 -0400 (EDT) From: Alan Stern [EMAIL PROTECTED] Does anybody think it would be worthwhile to convert string descriptors from UCS-16 to UTF-8 (instead of Latin1) when we read them in? Or even UTF-7 ... ? FWIW the input isn't UCS-16; it's UTF16-LE.

Re: [linux-usb-devel] [PATCH 11/12] drivers: PMC MSP71xx USB driver

2007-06-05 Thread David Brownell
This does far too much to be a single patch. It needs to be broken up. Agreed. 300+ KB is not digestible at all. Most maintainers like to see patches on the order of 10 KB; that much is easy to review. New drivers are rarely that small of course, but that should give you some understanding

Re: [linux-usb-devel] [PATCH 11/12] drivers: PMC MSP71xx USB driv er

2007-06-05 Thread Marc St-Jean
Alan Stern wrote: On Mon, 4 Jun 2007, Marc St-Jean wrote: [PATCH 11/12] drivers: PMC MSP71xx USB driver Patch to add an USB driver for the PMC-Sierra MSP71xx devices. Patches 1 through 10 were posted to [EMAIL PROTECTED] as well as other sub-system lists/maintainers as

[linux-usb-devel] [PATCH 11/12] drivers: PMC MSP71xx USB driver

2007-06-05 Thread Marc St-Jean
[PATCH 11/12] drivers: PMC MSP71xx USB driver Patch to add an USB driver for the PMC-Sierra MSP71xx devices. Patches 1 through 10 were posted to [EMAIL PROTECTED] as well as other sub-system lists/maintainers as appropriate. This patch has some dependencies on the first few patches in the set.

Re: [linux-usb-devel] [PATCH 11/12] drivers: PMC MSP71xx USB driver

2007-06-05 Thread David Brownell
[ $SUBJECT is getting less accurate... ] There are already big-endian support patches for EHCI in the USB queue. I strongly suspect this patch will clash with those. For example, they obviously don't cope with mixed configuration systems, where an SOC includes big-endian EHCI registers but

[linux-usb-devel] [patch 13/18] PS3: USB system-bus rework

2007-06-05 Thread Geoff Levand
USB HCD glue updates to reflect the new PS3 unifed device support. - Fixed remove() routine. - Added shutdown() routine. - Added request_mem_region() call. - Fixed MODULE_ALIAS(). - Made a proper fix for the hack done to support muti-platform in commit

Re: [linux-usb-devel] Grabbing characters from a Symbol barcode scanner

2007-06-05 Thread Andy Stewart
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Jiri Kosina wrote: On Wed, 30 May 2007, Andy Stewart wrote: I have a Symbol USB barcode scanner that appears to work with Linux. If I bring up a Konsole window and give it focus, characters are typed into that window as I scan barcodes.

[linux-usb-devel] The anchor patch critique

2007-06-05 Thread Pete Zaitcev
Hi, Oliver: I thought that the anchor would be very useful for the usblp cleanup, so I looked at the patch. Short summary is, I like the general idea, but not the cost of implementation. @@ -1161,6 +1176,8 @@ struct urb /* public: documented fields in the urb that can be used by drivers

Re: [linux-usb-devel] The anchor patch critique

2007-06-05 Thread Oliver Neukum
Am Mittwoch, 6. Juni 2007 06:18 schrieb Pete Zaitcev: Hi, Oliver: I thought that the anchor would be very useful for the usblp cleanup, so I looked at the patch. Short summary is, I like the general idea, but not the cost of implementation. @@ -1161,6 +1176,8 @@ struct urb /*