cvs commit: src/sys/kern sys_generic.c

[Jacques A. Vidrine]

On Tue, Jan 25, 2005 at 12:42:26PM +0300, Andrey Chernov wrote:
> On Tue, Jan 25, 2005 at 09:15:32AM +0000, Poul-Henning Kamp wrote:
> >     sys/kern             sys_generic.c 
> >   Log:
> >   Previously a read of zero bytes got handled in devfs:vop_read() but I
> >   missed that when the vnode bypass was introduced.
> >   
> >   Deal with zero length transfers before we even get to fo_ops->fo_read().
> 
> Please back it out. Zero bytes read may affect atime, now not.

I do not think that a zero byte read should not affect atime.

,----[ IEEE Std 1003.1, 2004 Edition ]
| Upon successful completion, where nbyte is greater than 0, read()
| shall mark for update the st_atime field of the file, and shall return
| the number of bytes read.
`----

Cheers,
-- 
Jacques A Vidrine / NTT/Verio
nectar at celabo.org / jvidrine at verio.net / nectar at FreeBSD.org