cvs commit: src/sys/kern sys_generic.c

[Jacques A. Vidrine]

On Tue, Jan 25, 2005 at 04:39:44PM +0100, Poul-Henning Kamp wrote:
> In message <20050125153729.GF5662 at lum.celabo.org>, "Jacques A. Vidrine" writes:
> >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.
> 
> Explain to me then why open+mmap doesn't affect atime isn't a bigger problem ?

Er. (a) Sorry I missed the rest of the thread before posting--- I do
not really have anything to add at this point. (b) I put one too many
negatives in the sentence above: s/do not think/think/.   (^_^;)

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