Changeset 1579 for trunk/batman-adv-kernelland/vis.h

Timestamp:

03/04/10 11:28:10 (6 months ago)

Author:

marek

Message:

batman-adv: don't have interrupts disabled while sending.

send_vis_packets() would disable interrupts before calling
dev_queue_xmit() which resulting in a backtrace in local_bh_enable().
Fix this by using kref on the vis_info object so that we can call
send_vis_packets() without holding vis_hash_lock. vis_hash_lock also
used to protect recv_list, so we now need a new lock to protect that
instead of vis_hash_lock.

Also a few checkpatch cleanups.

Reported-by: Linus Luessing <linus.luessing@…>
Signed-off-by: Andrew Lunn <andrew@…>
Signed-off-by: Marek Lindner <lindner_marek@…>
Signed-off-by: Simon Wunderlich <siwu@…>

Files:

• trunk/batman-adv-kernelland/vis.h (modified) (1 diff)

trunk/batman-adv-kernelland/vis.h

@@ -30 +30 @@
                              * from.  we should not reply to them. */
         struct list_head send_list;
+        struct kref refcount;
         /* this packet might be part of the vis send queue. */
         struct vis_packet packet;