Re: [PATCH] update ctime and mtime for mmaped write

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Miklos Szeredi
Date: Thursday, February 22, 2007 - 2:28 pm

> > > This still does not address the situation where a file is 'permanently'

You're right.  In theory, at least.  But in practice I don't think
this matters.  Show me an application that writes to a shared mapping
then doesn't call either msync or munmap and doesn't even exit.

If there were lot of these apps, then this bug would have been fixed
lots of years earlier.  In fact there are _very_ few apps writing to
shared mappings at all.

Applications should be encouraged to call msync(MS_ASYNC) because:

  - it's very fast (basically a no-op) on recent linux kernels

  - it's the only portable way to guarantee, that the data you written
    will _ever_ hit the disk.

There's really no downside to using msync(MS_ASYNC) in your
application, so making an effort to support applications that don't do
this is stupid, IMO.

Thanks,
Miklos
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Wed Feb 21, 10:51 am)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Wed Feb 21, 11:07 am)
Re: [PATCH] update ctime and mtime for mmaped write, Trond Myklebust, (Wed Feb 21, 11:12 am)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Wed Feb 21, 11:23 am)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Wed Feb 21, 11:28 am)
Re: [PATCH] update ctime and mtime for mmaped write, Trond Myklebust, (Wed Feb 21, 11:36 am)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Wed Feb 21, 11:50 am)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Wed Feb 21, 11:50 am)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Wed Feb 21, 11:54 am)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Wed Feb 21, 12:07 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Andrew Morton, (Wed Feb 21, 9:26 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Thu Feb 22, 12:49 am)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Thu Feb 22, 10:36 am)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Thu Feb 22, 10:39 am)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Thu Feb 22, 11:08 am)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Thu Feb 22, 11:16 am)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Thu Feb 22, 1:11 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Thu Feb 22, 1:14 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Thu Feb 22, 1:43 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Thu Feb 22, 1:48 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Thu Feb 22, 1:50 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Thu Feb 22, 1:55 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Trond Myklebust, (Thu Feb 22, 2:04 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Thu Feb 22, 2:28 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Peter Staubach, (Thu Feb 22, 2:52 pm)
Re: [PATCH] update ctime and mtime for mmaped write, Miklos Szeredi, (Thu Feb 22, 3:08 pm)