[Click] BUG: using smp_processor_id() in preemptible

[Jason,Park]

Hi

I'm running Click on preempt-SMP on linux 2.6.16.13 with
CONFIG_DEBUG_PREEMPT option.
But ToHost element makes many same message like this :
=====================================================================
BUG: using smp_processor_id() in preemptible [00000001] code: kclick/1293
caller is netif_receive_skb+0x5f/0x1ce
 [<c01036c2>] show_trace+0x13/0x15
 [<c01037a4>] dump_stack+0x16/0x18
 [<c01ec8c9>] debug_smp_processor_id+0x89/0xa0
 [<c02a02fe>] netif_receive_skb+0x5f/0x1ce
 [<f8e71b3c>] _ZN6ToHost4pushEiP6Packet+0xf4/0x15e [click]
 [<f8e6b72f>] _ZN10FromDevice8run_taskEv+0x63/0x172 [click]
 [<f8dc3fa2>] _ZN12RouterThread6driverEv+0x23e/0x51c [click]
 [<f8e93f82>] _Z11click_schedPv+0x8a/0x158 [click]
 [<c0100f45>] kernel_thread_helper+0x5/0
======================================================================

And here is some infos about click & linux
======================================================================
tohosttest.click
FromDevice(eth3) -> ToHost;

# cat /click/version
1.5.0

# uname -a
Linux 2.6.16.13 #21 SMP PREEMPT Wed May 31 12:37:49 Local time zone must be
set- i686 unknown

[CENTER][jason]ehus:~/ALDER/trunk/linux-2.6/linux-2.6.16.13>cat .config |
grep CONFIG_DEBUG_PREEMPT
CONFIG_DEBUG_PREEMPT=y
[CENTER][jason]ehus:~/ALDER/trunk/linux-2.6/linux-2.6.16.13>
======================================================================

For me this patch seems fix it.
I'll be please if this patch assist in project.

[CENTER][jason]ehus:~/ALDER/trunk/src/kernel/v15>cvs diff
elements/linuxmodule/tohost.cc
Index: elements/linuxmodule/tohost.cc
===================================================================
RCS file: /cvs/click/release/one/elements/linuxmodule/tohost.cc,v
retrieving revision 1.47
diff -r1.47 tohost.cc
170a171
>       local_bh_disable();
171a173
>       local_bh_enable(); 


Jason Park.