On Thu, 8 Jul 2004, Rogério Brito wrote:

> Dear Alan Stern,
> 
> Ealier this year, we exchanged some messages regarding a problem that I had
> with my USB Drive, which didn't work correctly with my VIA southbridge. In
> the message quoted below, you advised me to use a patch for my future
> kernels and that's precisely what I'm doing right now.
> 
> With your patch, I am able to use my USB Drive to save files to the USB
> drive and carry them to work, but still, I have two fears:
> 
> 1 - I'm using kernel 2.6.7 right now and it seems that each time, the patch
>     listed below applies correctly, but with some offset and I fear that
>     the kernel infra-structure will change enough in recent kernels so that
>     I won't be able to use the patch anymore.
> 
>     So, in light of this, I'd like to ask you if it is possible to include
>     this patch upstream and, if not, if you would like me to test more
>     "purified" versions of the patch for inclusion in the main kernel.

I think there wouldn't be any problem having the standard kernel change
the size of that array to 8 bytes, rather than 64 as in the patch you're
using.  However it would be a good idea to make sure that your device
will work okay with that change.  Use the patch below (in addition to the
one you've been using all along) and post your system log for when you
plug in the USB drive.

> 2 - I still have those SCSI layer problems listed below. Can I send you
>     debbuging messages of the kernels that I compile so we can try to track
>     what the reason of the problem is? Or should I take this with someone
>     else?

Go ahead and post the debugging messages here.  Be sure to turn on the
usb-storage debugging option in the kernel configuration.

Alan Stern


--- 2.6.7/drivers/usb/core/hub.c        Thu Jul  8 11:06:15 2004
+++ 2.6.7/drivers/usb/core/hub.c        Thu Jul  8 11:07:24 2004
@@ -257,6 +257,9 @@
        
        /* let khubd handle things */
        case 0:                 /* we got data:  port status changed */
+               if (urb->actual_length > 1)
+                       printk(KERN_INFO "hub_irq: actual_length = %d\n",
+                                       urb->actual_length);
                bits = 0;
                for (i = 0; i < urb->actual_length; ++i)
                        bits |= ((unsigned long) ((*hub->buffer)[i]))



-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 - 
digital self defense, top technical experts, no vendor pitches, 
unmatched networking opportunities. Visit www.blackhat.com
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to