[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090618072300.GA6243@ff.dom.local>
Date: Thu, 18 Jun 2009 07:23:00 +0000
From: Jarek Poplawski <jarkao2@...il.com>
To: David Miller <davem@...emloft.net>
Cc: robert.olsson@....uu.se, zheng.yan@...cle.com,
linux-kernel@...r.kernel.org, netdev@...r.kernel.org
Subject: [PATCH 3/1] Re: [BUG] fib_tries related Oops in 2.6.30
On Wed, Jun 17, 2009 at 06:56:58PM -0700, David Miller wrote:
> From: Jarek Poplawski <jarkao2@...il.com>
> Date: Mon, 15 Jun 2009 18:08:01 +0200
>
> > ipv4: Fix fib_trie rebalancing, part 2
> >
> > My previous patch, which explicitly delays freeing of tnodes by adding
> > them to the list to flush them after the update is finished, isn't
> > strict enough. It treats exceptionally tnodes without parent, assuming
> > they are newly created, so "invisible" for the read side yet.
> >
> > But the top tnode doesn't have parent as well, so we have to exclude
> > all exceptions (at least until a better way is found). Additionally we
> > need to move rcu assignment of this node before flushing, so the
> > return type of the trie_rebalance() function is changed.
> >
> > Reported-by: Yan Zheng <zheng.yan@...cle.com>
> > Signed-off-by: Jarek Poplawski <jarkao2@...il.com>
>
> Applied, thanks a lot Jarek.
Not at all, really :-( I definitely need more time to find out what's
going on here...
Sorry/thanks x2,
Jarek P.
------------------>
ipv4: Fix fib_trie rebalancing, part 3
Alas my fix, part 2, has one if too much again... (We can't repeat the
same test because tn is different.)
Reported-by: Yan Zheng <zheng.yan@...cle.com>
Signed-off-by: Jarek Poplawski <jarkao2@...il.com>
---
diff -Nurp a/net/ipv4/fib_trie.c b/net/ipv4/fib_trie.c
--- a/net/ipv4/fib_trie.c 2009-06-18 06:53:24.000000000 +0000
+++ b/net/ipv4/fib_trie.c 2009-06-18 06:58:00.000000000 +0000
@@ -1032,8 +1032,7 @@ static void trie_rebalance(struct trie *
tn = (struct tnode *)resize(t, (struct tnode *)tn);
rcu_assign_pointer(t->trie, (struct node *)tn);
- if (IS_TNODE(tn))
- tnode_free_flush();
+ tnode_free_flush();
return;
}
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists