lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4C487F2B.4040405@s5r6.in-berlin.de>
Date:	Thu, 22 Jul 2010 19:26:03 +0200
From:	Stefan Richter <stefanr@...6.in-berlin.de>
To:	Randy Dunlap <randy.dunlap@...cle.com>
CC:	linux-kernel@...r.kernel.org,
	Kristian Hoegsberg <krh@...planet.net>,
	linux1394-devel@...ts.sourceforge.net
Subject: Re: [review patch 1/2] firewire: new driver: nosy - IEEE 1394 traffic
 sniffer

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ