[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 unde

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 +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 atta

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 w

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 co

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-

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 be

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 n

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, lo

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->transfe

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

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

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: > [] usb_hcd_submit+0xb1/0x763 > [] ipt_do_table+0x2c7/0x2ef [ip_tables] > [] nf_ct_d

[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 >

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

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-

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. > > Yo

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-devel&m=118053809808501&w=2 >

[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 d

[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 yo

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 thou

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 Jasny

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]> > Jame

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 respon

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 : http://lkml.org/lk

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 PROTECT

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 : http://l

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 co

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 se

[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 data

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 L

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 > so

[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 linux-2

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; i

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 o

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/maintain

[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. If

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 there

[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 48fda45120a819ca40cadc50144b55bff1c4c7

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 > >

Re: [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 Andrew Morton
On Tue, 05 Jun 2007 14:26:48 -0700 Kevin Lloyd <[EMAIL PROTECTED]> wrote: > 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. 2.6.21.3 is quite old, sor