Re: [PATCH] locking bug in fib_semantics.c

On Mon, Aug 21, 2006 at 01:02:01PM +0200, Jarek Poplawski wrote:

On 21-08-2006 10:17, David Miller wrote:

quoted

From: Jarek Poplawski <redacted>
Date: Mon, 21 Aug 2006 10:16:43 +0200

...

quoted

I was skeptical of this case too, until I checked how
fib_release_info() was called.

I overlooked this - so I've to sharpen my sight and look at it 
again - now knowing it's there.

...

Hello,
I've found it at last but on that occasion I've got some
doubt according to rcu_read_lock and rcu_call treatment:
isn't it "illegal to block while in an RCU read-side
section"? And I think it takes place in:

fib_lookup(): from tb_insert (fn_hash_insert() or
  fn_trie_insert()), fib_create_info(), fib_check_nh() 

fn_trie_lookup(): like above, inet_addr_type(),
  tb_lookup()

fib_rule_put(): like #1 above or #2 after tb_lookup(),
  fib_res_put()

Shouldn't there be _bh also?

Jarek P.

PS: linux-2.6.18-rc4