Re: [PATCH] read() from driverfs files can read more bytes

From: Patrick Mochel (mochel@osdl.org)
Date: Thu Feb 07 2002 - 12:43:53 EST


> Can you also check for size >= PAGE_SIZE on enter to entry->show()
> procedure? It looks ugly to me that each driver has to check for this
> constant unless it wants to smash some innocent kernel memory.

Done. Thanks.

> And neither of driverfs_read_file nor driverfs_write_file supports
> semantic we use with other filesystems: If at least one byte was
> read/written, return byte count (even if error happens). Only if zero
> bytes was written, return error code.

I would think that you would want to return the error code. Say you did:

echo "action parameter" > file

and 'parameter' is an invalid parameter, as determined by the driver. It
would require another arbitrary check to determine if the command
succeeded or not if it returned the number of bytes written. Returning
-EINVAL lets userspace know that it made a boo-boo. Is that not good?

        -pat

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Thu Feb 07 2002 - 21:01:04 EST