From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Borkmann Subject: Re: [PATCH man-pages] man: packet.7: document fanout, ring and auxiliary options Date: Sat, 04 Jan 2014 15:47:20 +0100 Message-ID: <52C81EF8.6090908@redhat.com> References: <52A1F7D7.6040305@redhat.com> <1386350329-32723-1-git-send-email-willemb@google.com> <52A22B78.8070109@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Willem de Bruijn , linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, David Miller To: Michael Kerrisk Return-path: In-Reply-To: <52A22B78.8070109-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> Sender: linux-man-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org 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 > > Acked-by: Daniel Borkmann > > 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 linux-man" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html