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
[-- Attachment #1 --]
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 )
>
[-- Attachment #2 --]
=== 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);
}
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201012101915.40878.hselasky>