Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 10 Jan 2007 13:12:26 +0100
From:      "Julian H. Stacey" <jhs@flat.berklix.net>
To:        Atom Smasher <atom@smasher.org>
Cc:        freebsd-usb@freebsd.org
Subject:   Re: usb/107701: usbd ignores "detach" 
Message-ID:  <200701101212.l0ACCQfs050519@fire.jhs.private>
In-Reply-To: <200701100112.l0A1C2Qx046748@fire.jhs.private> 
References:  <200701100112.l0A1C2Qx046748@fire.jhs.private>

next in thread | previous in thread | raw e-mail | index | archive | help
People may have missed this ( hidden under a repost, sorry).

I can reproduce on 6.1-RELEASE error seen by Atom Smasher <atom@smasher.org>

> Loads of people Should have stumbled on this.
> I've found the bug in the code.
> This diff just marks the bug.
> 
> *** /host/fire/usr/src/usr.sbin/usbd/usbd.c	Fri Jul  1 17:49:52 2005
> --- usbd.c	Wed Jan 10 01:54:54 2007
> ***************
> *** 879,886 ****
> --- 879,891 ----
>   
>   		devinfo = &events.u.ue_device;
>   		for (i = 0; i < USB_MAX_DEVNAMES; i++) {
> + printf("  \nJHS1\n" );
>   			if (devinfo->udi_devnames[i][0] == '\0')
>   				break;
> + 				/* This break is a problem it prevents
> + 				   USB_EVENT_IS_DETACH being run at JHS3
> + 				*/
> + printf("  \nJHS2\n" );
>   
>   			memcpy(&the_event, &events, sizeof(the_event));
>   			the_devinfo = &the_event.u.ue_device;
> ***************
> *** 925,930 ****
> --- 930,936 ----
>   				if (USB_EVENT_IS_ATTACH(the_event.ue_type) &&
>   					action_match.action->attach) 
>   					execute_command(action_match.action->attach);
> + printf("  \nJHS3\n" );
>   				if (USB_EVENT_IS_DETACH(the_event.ue_type) &&
>   					action_match.action->detach)
>   					execute_command(action_match.action->detach);
> 
> 
> The error is still there in 6-Stable too.
> There is no src/usr.sbin/usbd/usbd.c in current.
> I don't know the code, & too tired & busy to fix this,
> Someone else here able to fix it & post a patch ?
> If not, Atom Smasher <atom@smasher.org> please use
> send-pr to report this identified bug.

-- 
Julian Stacey.  BSD Unix C Net Consultancy, Munich/Muenchen  http://berklix.com
Mail Ascii, not HTML.		Ihr Rauch = mein allergischer Kopfschmerz.
			http://berklix.org/free-software



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200701101212.l0ACCQfs050519>