Hello Jan, On 07/23/2014 12:54 PM, Jan Chaloupka wrote: > From https://bugzilla.redhat.com/show_bug.cgi?id=1110401. > > unix.7 is not clear about socket type of ioctl_type argument of ioctl > function. The description of SIOCINQ is applicable only for SOCK_STREAM > socket. For SOCK_DGRAM, udp(7) man page gives correct description of > SIOCINQ. Again, the intent of this patch seems correct, but some details need fixing, I think. > --- > man7/unix.7 | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/man7/unix.7 b/man7/unix.7 > index b0ade5b..6bcc8ba 100644 > --- a/man7/unix.7 > +++ b/man7/unix.7 > @@ -299,7 +299,9 @@ The correct syntax is: > can be: > .TP > .B SIOCINQ > -Returns the amount of queued unread data in the receive buffer. > +For > +.B SOCK_STREAM > +socket the function returns the amount of queued unread data in the > receive buffer. > The socket must not be in LISTEN state, otherwise an error > .RB ( EINVAL ) > is returned. > @@ -319,6 +321,10 @@ defined in > .\" That number is normally larger than the number of bytes of pending > .\" output. Since this info is, from userland's point of view, imprecise, > .\" and it may well change, probably best not to document this now. > +For > +.B SOCK_DGRAM > +socket, see > +.BR udp (7). It feels a bit odd to simply point at udp(7), since that describes another socket domain (Internet). Maybe say something like: For SOCK_DRAM sockets, the returned value is the same as for Internet domain datagram sockets; see udp(7). What do you think? Cheers, Michael > .SH ERRORS > .TP > .B EADDRINUSE > @@ -450,4 +456,5 @@ see > .BR cmsg (3), > .BR capabilities (7), > .BR credentials (7), > -.BR socket (7) > +.BR socket (7), > +.BR udp (7) > -- 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
