[PATCH] tmpfs 2/4 mknod times

[PATCH] tmpfs 2/4 mknod times

[Hugh Dickins]

shmem_mknod should not update directory times if it cannot get an inode.

--- 2.4.19-pre10/mm/shmem.c	Tue Jun 11 19:02:30 2002
+++ linux/mm/shmem.c	Tue Jun 11 19:02:30 2002
@@ -987,8 +987,8 @@
 	struct inode * inode = shmem_get_inode(dir->i_sb, mode, dev);
 	int error = -ENOSPC;
 
-	dir->i_ctime = dir->i_mtime = CURRENT_TIME;
 	if (inode) {
+		dir->i_ctime = dir->i_mtime = CURRENT_TIME;
 		d_instantiate(dentry, inode);
 		dget(dentry); /* Extra count - pin the dentry in core */
 		error = 0;

Re: [PATCH] tmpfs 2/4 mknod times

[Ryan Cumming]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On June 11, 2002 20:25, Hugh Dickins wrote:
> On Tue, 11 Jun 2002, Ryan Cumming wrote:
> > On June 11, 2002 19:54, Hugh Dickins wrote:
> > > +               dir->i_ctime = dir->i_mtime = CURRENT_TIME;
> >
> > I'm probably misreading this, but why does shmem_mknod modify the
> > directory's ctime?
>
> Hmmm, good question.  Perhaps I'll have dreamt up an answer by morning.
Well, lets see if the list has any ideas while you're sleeping.

- -Ryan
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQE9BsCLLGMzRzbJfbQRAic3AJ9hh76od28Ic5OzB2PU8hLsV5vogACfbITB
8FyDd6i5BeMtk3xLzt1Y5ns=
=7+h9
-----END PGP SIGNATURE-----

Re: [PATCH] tmpfs 2/4 mknod times

[Jan Harkes]

On Tue, Jun 11, 2002 at 08:31:20PM -0700, Ryan Cumming wrote:
> On June 11, 2002 20:25, Hugh Dickins wrote:
> > On Tue, 11 Jun 2002, Ryan Cumming wrote:
> > > On June 11, 2002 19:54, Hugh Dickins wrote:
> > > > +               dir->i_ctime = dir->i_mtime = CURRENT_TIME;
> > >
> > > I'm probably misreading this, but why does shmem_mknod modify the
> > > directory's ctime?
> >
> > Hmmm, good question.  Perhaps I'll have dreamt up an answer by morning.
> Well, lets see if the list has any ideas while you're sleeping.

My guess would be that 'mtime' should be always updated because the
directory contents is changed (a new name is added). And 'ctime' has to
change whenever this new name cause a change in the size of the
directory because the i_size attribute has changed.

As tmpfs doesn't really have on-disk blocks for it's directories, every
create will change the directory size. (as well as every unlink).

Jan