linux/drivers/ieee1394
Ingo Molnar d378834840 [PATCH] lockdep: annotate ieee1394 skb-queue-head locking
ieee1394 reuses the skb infrastructure of the networking code, and uses two
skb-head queues: ->pending_packet_queue and hpsbpkt_queue.  The latter is used
in the usual fashion: processed from a kernel thread.  The other one,
->pending_packet_queue is also processed from hardirq context (f.e.  in
hpsb_bus_reset()), which is not what the networking code usually does (which
completes from softirq or process context).  This locking assymetry can be
totally correct if done carefully, but it can also be dangerous if networking
helper functions are reused, which could assume traditional networking use.

It would probably be more robust to push this completion into a workqueue -
but technically the code can be 100% correct, and lockdep has to be taught
about it.  The solution is to split the ->pending_packet_queue skb-head->lock
class from the networking lock-class by using a private lock-validator key.

Has no effect on non-lockdep kernels.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
Cc: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: Jody McIntyre <scjody@modernduck.com>
Cc: Ben Collins <bcollins@debian.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
2006-07-03 15:27:08 -07:00
..
.gitignore
config_roms.c Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
config_roms.h
csr.c
csr.h
csr1212.c ieee1394: add preprocessor constant for invalid csr address 2006-06-12 18:15:31 -04:00
csr1212.h ieee1394: add preprocessor constant for invalid csr address 2006-06-12 18:15:31 -04:00
dma.c ieee1394: speed up of dma_region_sync_for_cpu 2006-06-12 17:52:59 -04:00
dma.h
dv1394-private.h
dv1394.c Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
dv1394.h
eth1394.c [PATCH] drivers: use list_move() 2006-06-26 09:58:18 -07:00
eth1394.h ieee1394: add preprocessor constant for invalid csr address 2006-06-12 18:15:31 -04:00
highlevel.c Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
highlevel.h
hosts.c [PATCH] lockdep: annotate ieee1394 skb-queue-head locking 2006-07-03 15:27:08 -07:00
hosts.h ieee1394: extend lowlevel API for address range properties 2006-06-12 18:13:21 -04:00
ieee1394-ioctl.h
ieee1394.h
ieee1394_core.c Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
ieee1394_core.h spelling fixes 2006-06-26 18:35:02 +02:00
ieee1394_hotplug.h
ieee1394_transactions.c ieee1394: save RAM by using a single tlabel for broadcast transactions 2006-06-12 18:12:49 -04:00
ieee1394_transactions.h
ieee1394_types.h
iso.c
iso.h
Kconfig [PATCH] ieee1394: nodemgr: do not peek into struct semaphore 2006-06-25 10:00:54 -07:00
Makefile
nodemgr.c Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
nodemgr.h
ohci1394.c [PATCH] irq-flags: firewire: Use the new IRQF_ constants 2006-07-02 13:58:50 -07:00
ohci1394.h ohci1394: set address range properties 2006-06-12 18:13:32 -04:00
oui.db
oui2c.sh Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
pcilynx.c [PATCH] irq-flags: firewire: Use the new IRQF_ constants 2006-07-02 13:58:50 -07:00
pcilynx.h Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
raw1394-private.h
raw1394.c [PATCH] drivers: use list_move() 2006-06-26 09:58:18 -07:00
raw1394.h
sbp2.c Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
sbp2.h sbp2: use __attribute__((packed)) for on-the-wire structures 2006-06-12 18:14:47 -04:00
video1394.c Remove obsolete #include <linux/config.h> 2006-06-30 19:25:36 +02:00
video1394.h