firewire: new driver: nosy - IEEE 1394 traffic sniffer
in [Kernel]
|
Prev: [PATCH v4] gpio: sx150x: Add Semtech I2C sx150x gpio expander driver.
Next: Package Level Thermal Control and Power Limit Notification: pkgtemp doc
From: Stefan Richter on 22 Jul 2010 13:30 Randy Dunlap wrote: > On Thu, 22 Jul 2010 11:56:38 +0200 (CEST) Stefan Richter wrote: >> --- a/drivers/firewire/Kconfig >> +++ b/drivers/firewire/Kconfig >> @@ -66,4 +66,27 @@ config FIREWIRE_NET >> >> source "drivers/ieee1394/Kconfig" >> >> +config FIREWIRE_NOSY >> + tristate "Nosy - a FireWire traffic sniffer for PCILynx cards" >> + depends on PCI > > Just curious: why not depends on IEEE1394_PCILYNX ? This is a stand-alone driver that interacts neither with the old nor with the new FireWire kernel subsystem. It contains only the minimum of DMA programming that is required to get PCILynx' snoop mode (a catch-all reception DMA) up and running. IEEE1394_PCILYNX alias pcilynx on the other hand is a bigger card driver which implements several transmit and receive DMA modes as required by a low-level driver of the ieee1394 stack.� Thus, nosy does not use code from or share code with anything in drivers/ieee1394/. �) except for isochronous transmission and reception; this was initially also provided by pcilynx but became outdated early on in the history of the ieee1394 stack when video1394 was added; and except for physical DMA that the sbp2 driver requires If you have a PCILynx card, you can bind either pcilynx or nosy to it. (As the kernel's build system works, I think nosy is going to be bound before pcilynx if both are present.) However, nosy the bus analyzer provides better value than pcilynx the ieee1394 lowlevel driver that lacks 3/4 of the functionality that ohci1394 offers. >> --- /dev/null >> +++ b/drivers/firewire/nosy-user.h >> @@ -0,0 +1,25 @@ >> +#ifndef __nosy_user_h >> +#define __nosy_user_h >> + >> +#include <linux/ioctl.h> >> +#include <linux/types.h> >> + >> +#define NOSY_IOC_GET_STATS _IOR('&', 0, struct nosy_stats) >> +#define NOSY_IOC_START _IO('&', 1) >> +#define NOSY_IOC_STOP _IO('&', 2) >> +#define NOSY_IOC_FILTER _IOW('&', 2, __u32) > > Please add '&' to Documentation/ioctl/ioctl-number.txt. Right, I will send a follow-up patch which does this if there are no fundamental objections to this ioctl allocation. -- Stefan Richter -=====-==-=- -=== =-==- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |