Gidday,

Thanks for this patch.  I have a few questions:

> Andi Kleen wrote:
> > The man page was supposed to document the kernel, so it's probably
> > a bug in the manpage.  You should send a patch to the manpages
> > maintainers, with a warning in NOTES that the Linux behaviour
> > differs from other OS.
> 
> OK. Attached patch fixes this and adds comment to the NOTES. Also comment
> about SOL_IP portability added to the NOTES and duplicate IP_PKTINFO
> removed in the VERSIONS section.

[patch inlined...]

> -.I IP_RECVTTL
> +.I IP_TTL

So is it the case that this option was just wrongly named in the 
original page, or is the change here reflective of something that 
has changed in the kernel?  (It doesn't look like the latter is 
true, but I thought it better to check.)

>  control message with the time to live
>  field of the received packet as a byte. Not supported for
>  .B SOCK_STREAM
> @@ -789,6 +789,20 @@ received datagrams. Linux has the more g
>  .I IP_PKTINFO
>  for the same task.
>  .PP
> +Some BSD sockets implementations also provide
> +.I IP_RECVTTL
> +option, but ancillary message with type
> +.I IP_RECVTTL
> +is passed with incoming packet. It's different from
> +.I IP_TTL
> +used in Linux.

>From reading the sources, Linux appears to have both 
IP_RECVTTL and IP_TTL.  So, does there not also need
to be some documentation of the "real" IP_RECVTTL?

> +.PP
> +Using
> +.I SOL_IP
> +socket options level isn't portable, BSD-based stacks use
> +.I IPPROTO_IP
> +level.

Recently (not yet published), I went though ip(7), tcp(7), udp(7) 
etc, and changed SOL_IP to IPPROTO_IP, SOL_TCP to IPPROTO_TCP, 
etc, on the basis that

-- the IPPROTO_* constants are what appear in POSIX, and
-- glibc defines the IPPROTO_* constants with the same values
   as the corresponding SOL_* constants.

Does anyone see a problem with this change in the docs?

Cheers,

Michael

-- 
Michael Kerrisk
maintainer of Linux man pages Sections 2, 3, 4, 5, and 7 

Want to help with man page maintenance?  Grab the latest
tarball at ftp://ftp.win.tue.nl/pub/linux-local/manpages/
and grep the source files for 'FIXME'.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to