Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 10 Dec 2010 19:15:40 +0100
From:      Hans Petter Selasky <hselasky@c2i.net>
To:        Oleg Nauman <oleg.nauman@gmail.com>
Cc:        freebsd-current@freebsd.org, freebsd-usb@freebsd.org
Subject:   Re: USB related panic on 8.2-PRERELEASE
Message-ID:  <201012101915.40878.hselasky@c2i.net>
In-Reply-To: <AANLkTinVuEg=YicnOSXm02NEWbpT4dbfVvyLiiUXzqq-@mail.gmail.com>
References:  <AANLkTi=OxWODd0cQR%2BjKz7=Hxpnd52=HfLbuarfAiqbW@mail.gmail.com> <AANLkTikyNpfBMXKeDcy1PYfAasYovmwjKZqRt8uXk%2BE-@mail.gmail.com> <AANLkTinVuEg=YicnOSXm02NEWbpT4dbfVvyLiiUXzqq-@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
--Boundary-00=_M5mANEC8qZagE7a
Content-Type: Text/Plain;
  charset="iso-8859-15"
Content-Transfer-Encoding: 7bit

Hi,

I think this is a known issue which never got fixed. Please try the attached 
patch and report back.

XXX_SAFE != XXX_REAL_SAFE :-)

--HPS

On Thursday 09 December 2010 12:02:48 Oleg Nauman wrote:
> On Wed, Dec 8, 2010 at 7:05 PM, Oleg Nauman <oleg.nauman@gmail.com> wrote:
> >  Hello Hans,
> > 
> > On Wed, Dec 8, 2010 at 3:33 PM, Hans Petter Selasky <hselasky@c2i.net> 
wrote:
> >> On Wednesday 08 December 2010 11:41:28 Oleg Nauman wrote:
> >>> Hello,
> >>> 
> >>> Unfortunately my notebook experienced the crash during the attempts to
> >>> attach EVDO modem supplied with builtin MicroSD cardreader.
> >>> Related core.txt file is attached as well as 'usbconfig
> >>> dump_all_config_desc' output (all_config.txt)
> >>> USB subsystem reports endless USB_ERR_STALLED events during attempts
> >>> to attach umass device, but attaches it finally ( sometimes it
> >>> attached after two attempts, sometimes it trying to attach during
> >>> 15-20 minutes ).MicroSD is inserted there, without any effect  on
> >>> attachment attempts though.
> >> 
> >> Hi,
> >> 
> >> Can you reproduce the panic using a kernel built with INVARIANTS options
> >> and DEBUG_MEMGUARD .
> > 
> >  I rebuilt my kernel with options you mentioned ( have added
> > INVARIANT_SUPPORT required  by INVARIANTS though )
> > 
> > Waiting on panic..
> 
>  Got it finally ( core.txt file is attached )
> 

--Boundary-00=_M5mANEC8qZagE7a
Content-Type: text/x-patch; charset="iso-8859-15"; name="kern_conf.c.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
	filename="kern_conf.c.patch"

=== kern_conf.c
==================================================================
--- kern_conf.c	(revision 215787)
+++ kern_conf.c	(local)
@@ -926,7 +926,7 @@
 destroy_devl(struct cdev *dev)
 {
 	struct cdevsw *csw;
-	struct cdev_privdata *p, *p1;
+	struct cdev_privdata *p;
 
 	mtx_assert(&devmtx, MA_OWNED);
 	KASSERT(dev->si_flags & SI_NAMED,
@@ -974,7 +974,7 @@
 	dev_unlock();
 	notify_destroy(dev);
 	mtx_lock(&cdevpriv_mtx);
-	LIST_FOREACH_SAFE(p, &cdev2priv(dev)->cdp_fdpriv, cdpd_list, p1) {
+	while ((p = LIST_FIRST(&cdev2priv(dev)->cdp_fdpriv)) != NULL) {
 		devfs_destroy_cdevpriv(p);
 		mtx_lock(&cdevpriv_mtx);
 	}

--Boundary-00=_M5mANEC8qZagE7a--



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