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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Daniel, and Willem,

On Sun, Jan 5, 2014 at 3:47 AM, Daniel Borkmann <dborkman@xxxxxxxxxx> wrote:
> 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@xxxxxxxxxx>
>>
>>
>> Acked-by: Daniel Borkmann <dborkman@xxxxxxxxxx>
>>
>> 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 for the ping, and sorry for the delay. I've applied the patch:
http://git.kernel.org/cgit/docs/man-pages/man-pages.git/commit/?id=dbb4f7516b22663da977ea5c37691c5940bb3eb7

I added kernel version numbers for two of the newly documented
options. You may want to check them:
http://git.kernel.org/cgit/docs/man-pages/man-pages.git/commit/?id=342639d9e9d766b7b3b3a03045144be64f1f556c

Cheers,

Michael

-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux