Well, let's start with the obvious: Remove the line of code you mention,
and instead do:
snprintf(onetouch->name, 128, "Maxtor Test String");If that doesn't fix the problem, then it's not the culprit and you need to keep looking. If that is the problem, then my guess is more that this is a udev-related issue than anything else, as the two strings you're sourcing data from are coming from udev. I would be interested in seeing what your debug printouts show at detach time, to see if you're really disconnecting the device properly. That plus the USB Storage debugging should be telling. Matt On Thu, Mar 24, 2005 at 08:19:51PM -0500, Nick Sillik wrote: > Okay, I have a serious problem. Everything goes fine when the device is > unplugged. When the device is plugged back in however, everything goes bad, > then > the system hangs. The only change that I can recall happening in the interim > is > that us_data no loger has a manufacturer or product feild, and I have to use > this: > snprintf(onetouch->name, 128, "%s %s", > udev->manufacturer, udev->product); > > I may be missing something huge, but that doesn't seem like it should hang the > system. I put all sorts of printks in the onetouch_release_input and the > onetouch_close functions. From what I can tell all of the code is being run. > After reinserting the drive however. It doesn't even get as far as telling me > that a device has been plugged in. (using tail -f /var/log/kern.log) I'm at a > loss. I've compared the working code to the new, non-working code, and I'm > unable to find anything that would cause this. > > Nick Sillik > [EMAIL PROTECTED] > > > Alan Stern wrote: > > On Mon, 21 Mar 2005, Nick Sillik wrote: > > > > > >>I changed it to use the unusual devs code. Now I get this error when > >>insmodding. > >> > >>usb-storage: This device (0d49,7010,0200 S 06 P 50) has an unneeded SubClass > >>entry in unusual_devs.h > >> Please send a copy of this message to > >> <linux-usb-devel@lists.sourceforge.net> > >> > >>hmmm? > > > > > > You probably didn't specify US_SC_DEVICE in your unusual_devs entry. > > Anyway, that message isn't an error -- it's just informational. > > > > Alan Stern > > > > P.S.: Did you also make the other changes I recommended? > > > > > > ------------------------------------------------------- > SF email is sponsored by - The IT Product Guide > Read honest & candid reviews on hundreds of IT Products from real users. > Discover which products truly live up to the hype. Start reading now. > http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click > _______________________________________________ > linux-usb-devel@lists.sourceforge.net > To unsubscribe, use the last form field at: > https://lists.sourceforge.net/lists/listinfo/linux-usb-devel -- Matthew Dharm Home: [EMAIL PROTECTED] Maintainer, Linux USB Mass Storage Driver It was a new hope. -- Dust Puppy User Friendly, 12/25/1998
pgpyBvhzk2aLG.pgp
Description: PGP signature
