Hi Christoph, On 25 April 2016 at 10:47, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Mon, Apr 18, 2016 at 02:51:50PM +0100, Michael Kerrisk (man-pages) wrote: >> Thanks. I applied the patch, but I see one point where the doc >> and code differ, and I suspect that the code needs to be fixed. >> See below. > >> > .TP >> > .B EINVAL >> > The vector count \fIiovcnt\fP is less than zero or greater than the >> > -permitted maximum. >> > +permitted maximum. Or, an unknown flag is specified in \fIflags\fP. >> >> In the case described in the last sentence, the code currently appears >> to be returning EOPNOTSUPP. EINVAL is more usual here, so I think the >> code needs adjusting. Your thoughts? > > I'd rather update the man page - EOPNOTSUPP is a much more descriptive > error code for this case. I'll send you a patch. Unless I'm misunderstanding something here, you're proposing something very inconsistent. The standard error for unknown flag bits is EINVAL. This is so for dozens of systems calls (check the man pages; you might find a rare exception, but that's the point, they are exceptions). It seems to me here that it's really the implementation that needs fixing, not the man page! 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-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
