CHECK /home/lunn/batman/trunk/batman-adv-kernelland/gateway_common.c /home/lunn/batman/trunk/batman-adv-kernelland/gateway_common.c:28:6: warning: symbol 'kbit_to_gw_srv_class' was not declared. Should it be static? CHECK /home/lunn/batman/trunk/batman-adv-kernelland/gateway_client.c /home/lunn/batman/trunk/batman-adv-kernelland/gateway_client.c:27:1: warning: symbol 'gw_list' was not declared. Should it be static? /home/lunn/batman/trunk/batman-adv-kernelland/gateway_client.c:28:1: warning: symbol 'curr_gw_lock' was not declared. Should it be static? /home/lunn/batman/trunk/batman-adv-kernelland/gateway_client.c:29:1: warning: symbol 'gw_list_lock' was not declared. Should it be static? /home/lunn/batman/trunk/batman-adv-kernelland/gateway_client.c:227:6: warning: context imbalance in 'gw_node_update' - wrong count at exit
Ah, just took a closer look. These are probably from Marek, not Simon.
gw_node_update() is missing a rcu_read_unlock(); before the return inside the list_for_each_entry_rcu() loop. The others are less serious, but should be corrected as well.
Andrew