netdev - Re: panic 2.6.27-rc3-git2, qdisc_dequeue

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20080817223250.GC2546@ami.dom.local>
Date:	Mon, 18 Aug 2008 00:32:50 +0200
From:	Jarek Poplawski <jarkao2@...il.com>
To:	Denys Fedoryshchenko <denys@...p.net.lb>
Cc:	netdev@...r.kernel.org
Subject: Re: panic 2.6.27-rc3-git2, qdisc_dequeue_head

On Mon, Aug 18, 2008 at 01:21:40AM +0300, Denys Fedoryshchenko wrote:
> Short testing - TBF is working fine. No warnings, no oops, nothing yet. 
> Before - intensive changes was causing oops easily. I will test for 1 day 
> more with TBF.
> 
> P.S. New shaper (based on flow classifier + HTB classes, instead of prio+tbf) 
> crashing very fast on loaded server. Probably it is another bug.
> Here is message:
> http://www.nuclearcat.com/files/crash-newshaper.txt

I'll try to look at this "tomorrow".

Thanks,
Jarek P.

> 
> On Monday 18 August 2008, Jarek Poplawski wrote:
> > On Sun, Aug 17, 2008 at 10:36:17PM +0300, Denys Fedoryshchenko wrote:
> > > I am lost in patches :-)
> > > When you will be back, let me know if order and set of patches is
> > > correct. Here is what i am using:
> > >
> > > http://www.nuclearcat.com/patches/00-prevfixes1.patch
> > > http://www.nuclearcat.com/patches/00-prevfixes2.patch
> > > http://www.nuclearcat.com/patches/01-take3.patch
> > > http://www.nuclearcat.com/patches/02-fix1.patch
> > > http://www.nuclearcat.com/patches/03-fix_patch_2.patch
> > >
> > > And thanks for your help!
> > > I cannot take risk and put to wide production, since it is crashing
> > > sometimes, but i guess in few days we can make it ready for production.
> >
> > OK, so please try to test with all these patches above plus one new
> > patch below.
> >
> > Thanks,
> > Jarek P.
> >
> > (04-fix_patch_3.patch)
> > ---
> >
> >  net/sched/sch_api.c |    4 ++--
> >  1 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/net/sched/sch_api.c b/net/sched/sch_api.c
> > index c25465e..c8dc72e 100644
> > --- a/net/sched/sch_api.c
> > +++ b/net/sched/sch_api.c
> > @@ -638,9 +638,9 @@ static void notify_and_destroy(struct sk_buff *skb,
> > struct nlmsghdr *n, u32 clid qdisc_notify(skb, n, clid, old, new);
> >
> >  	if (old) {
> > -		spin_lock_bh(&old->q.lock);
> > +		sch_tree_lock(old);
> >  		qdisc_destroy(old);
> > -		spin_unlock_bh(&old->q.lock);
> > +		sch_tree_unlock(old);
> >  	}
> >  }
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe netdev" in
> > the body of a message to majordomo@...r.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html