Re: Active URB submitted twice in pegasus driver

2013-04-08 Thread Sarah Sharp
On Fri, Apr 05, 2013 at 08:01:03PM +0300, Petko Manolov wrote: > On Wed, 27 Mar 2013, Petko Manolov wrote: > >On Tue, 26 Mar 2013, Sarah Sharp wrote: > The new driver actually compiles and runs fine with Pegasus-II based > device. Please let me know if you run into problems. I'm away for a week,

Re: Active URB submitted twice in pegasus driver

2013-04-05 Thread Petko Manolov
On Wed, 27 Mar 2013, Petko Manolov wrote: On Tue, 26 Mar 2013, Sarah Sharp wrote: ctrl_callback is still reading the URB status, and using it in the switch statement. It's also using the urb->context as well, to dig out a pointer (pegasus_t) that the pegasus_set_multicast already has access t

Re: Active URB submitted twice in pegasus driver

2013-03-26 Thread Petko Manolov
On Tue, 26 Mar 2013, Sarah Sharp wrote: On Tue, Mar 26, 2013 at 10:54:25PM +0200, Petko Manolov wrote: On Tue, 26 Mar 2013, Sarah Sharp wrote: But considering the multicast code is pretty old, I bet I'm running into a different bug... The fix is rather small. Approximately one gigabyte was

Re: Active URB submitted twice in pegasus driver

2013-03-26 Thread Sarah Sharp
On Tue, Mar 26, 2013 at 10:54:25PM +0200, Petko Manolov wrote: > On Tue, 26 Mar 2013, Sarah Sharp wrote: > > >But considering the multicast code is pretty old, I bet I'm running into > >a different bug... > > The fix is rather small. Approximately one gigabyte was transferred > while alternating

Re: Active URB submitted twice in pegasus driver

2013-03-26 Thread Petko Manolov
On Tue, 26 Mar 2013, Sarah Sharp wrote: But considering the multicast code is pretty old, I bet I'm running into a different bug... The fix is rather small. Approximately one gigabyte was transferred while alternating in and out of promiscuous mode. No issues. I guess this should move the

Re: Active URB submitted twice in pegasus driver

2013-03-26 Thread Sarah Sharp
On Tue, Mar 26, 2013 at 07:28:53PM +0200, Petko Manolov wrote: > On Tue, 26 Mar 2013, Sarah Sharp wrote: > > >On Tue, Mar 26, 2013 at 05:22:07PM +0200, Petko Manolov wrote: > >>On Mon, 25 Mar 2013, Sarah Sharp wrote: > >After taking a brief glance at the pegasus code, > >pegasus_set_multicast look

Re: Active URB submitted twice in pegasus driver

2013-03-26 Thread Petko Manolov
On Tue, 26 Mar 2013, Sarah Sharp wrote: On Tue, Mar 26, 2013 at 05:22:07PM +0200, Petko Manolov wrote: On Mon, 25 Mar 2013, Sarah Sharp wrote: Hi Petko, I'm testing a USB to ethernet adapter with Greg's usb-linus branch (based on 3.9-rc4). I'm seeing an odd behavior, and I'm suspicious that

Re: Active URB submitted twice in pegasus driver

2013-03-26 Thread Sarah Sharp
On Tue, Mar 26, 2013 at 05:22:07PM +0200, Petko Manolov wrote: > On Mon, 25 Mar 2013, Sarah Sharp wrote: > > >Hi Petko, > > > >I'm testing a USB to ethernet adapter with Greg's usb-linus branch (based > >on 3.9-rc4). I'm seeing an odd behavior, and I'm suspicious that a > >second behavior found b

Re: Active URB submitted twice in pegasus driver

2013-03-26 Thread Petko Manolov
On Mon, 25 Mar 2013, Sarah Sharp wrote: Hi Petko, I'm testing a USB to ethernet adapter with Greg's usb-linus branch (based on 3.9-rc4). I'm seeing an odd behavior, and I'm suspicious that a second behavior found by Stephen Hemminger may also be related: http://marc.info/?l=linux-usb&m=136364