netdev - Re: [PATCH man-pages] man: packet.7: document fanout, ring and auxiliary options

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <52C81EF8.6090908@redhat.com>
Date:	Sat, 04 Jan 2014 15:47:20 +0100
From:	Daniel Borkmann <dborkman@...hat.com>
To:	Michael Kerrisk <mtk.manpages@...il.com>
CC:	Willem de Bruijn <willemb@...gle.com>, linux-man@...r.kernel.org,
	netdev@...r.kernel.org, David Miller <davem@...emloft.net>
Subject: Re: [PATCH man-pages] man: packet.7: document fanout, ring and auxiliary
 options

On 12/06/2013 08:54 PM, Daniel Borkmann wrote:
> On 12/06/2013 06:18 PM, Willem de Bruijn wrote:
>> The packet socket manual page does not list all socket options.
>>
>> This patch adds descriptions of the common packet socket options
>>    PACKET_AUXDATA, PACKET_FANOUT, PACKET_RX_RING, PACKET_STATISTICS,
>>    PACKET_TX_RING
>>
>> and the ring-specific options
>>    PACKET_LOSS, PACKET_RESERVE, PACKET_TIMESTAMP, PACKET_VERSION
>>
>> It does not yet add descriptions for
>>    PACKET_COPY_THRESH, PACKET_HDRLEN, PACKET_ORIGDEV,
>>    PACKET_TX_HAS_OFF, PACKET_TX_TIMESTAMP, PACKET_VNET_HDR
>>
>> It tries to balance being informative with exposing kernel detail
>> that is unlikely to be used by most readers or that may change
>> frequently. For implementation details, the manpage points to the
>> documentation in kernel Documentation/networking. Let me know if
>> options should be added or removed.
>>
>> Source: PACKET_FANOUT, PACKET_RX_RING and PACKET_VERSION are in
>> /tools/testing/net/psock_fanout.c in the latest Linux kernel source
>> tree. PACKET_STATISTICS was in the first version of that test.
>> PACKET_TX_RING I have used elsewhere. The other options are based
>> on reading kernel code.
>>
>>   [Very minor fixups. -dborkman]
>>
>> Signed-off-by: Willem de Bruijn <willemb@...gle.com>
>
> Acked-by: Daniel Borkmann <dborkman@...hat.com>
>
> I went over the man page again, and it looks good to me.
>
> Great work and thanks for following up Willem!

Ping, Michael, any progress on this one? Lets not get this work
lost somewhere as it happened in March last year ...

Thanks,
Daniel
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html