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] [thread-next>] [day] [month] [year] [list]

Message-Id: <200808180121.40998.denys@visp.net.lb>
Date:	Mon, 18 Aug 2008 01:21:40 +0300
From:	Denys Fedoryshchenko <denys@...p.net.lb>
To:	Jarek Poplawski <jarkao2@...il.com>
Cc:	netdev@...r.kernel.org
Subject: Re: panic 2.6.27-rc3-git2, qdisc_dequeue_head

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

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