2002-11-18 11:14:13

by Adrian Bunk

[permalink] [raw]
Subject: [2.5 patch] fix compile breakage in fs/ after nanosecond stat timefields patch

Hi Andi,

the patch below fixes the following compile errors in 2.5.48:

<-- snip -->

..
gcc -Wp,-MD,fs/adfs/.inode.o.d -D__KERNEL__ -Iinclude -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fomit-frame-pointer -fno-strict-aliasing -fno-common -pipe -mpreferred-stack-boundary=2 -march=k6 -Iarch/i386/mach-generic -nostdinc -iwithprefix include -DKBUILD_BASENAME=inode -DKBUILD_MODNAME=adfs -c -o fs/adfs/inode.o fs/adfs/inode.c
fs/adfs/inode.c: In function `adfs_adfs2unix_time':
fs/adfs/inode.c:188: incompatible types in return
make[2]: *** [fs/adfs/inode.o] Error 1
...
gcc -Wp,-MD,fs/afs/.vlclient.o.d -D__KERNEL__ -Iinclude -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fomit-frame-pointer -fno-strict-aliasing -fno-common -pipe -mpreferred-stack-boundary=2 -march=k6 -Iarch/i386/mach-generic -nostdinc -iwithprefix include -DKBUILD_BASENAME=vlclient -DKBUILD_MODNAME=kafs -c -o fs/afs/vlclient.o fs/afs/vlclient.c
fs/afs/vlclient.c: In function `afs_rxvl_get_entry_by_id_async2':
fs/afs/vlclient.c:587: incompatible types in assignment
make[2]: *** [fs/afs/vlclient.o] Error 1
...
gcc -Wp,-MD,fs/jffs2/.fs.o.d -D__KERNEL__ -Iinclude -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fomit-frame-pointer -fno-strict-aliasing -fno-common -pipe -mpreferred-stack-boundary=2 -march=k6 -Iarch/i386/mach-generic -nostdinc -iwithprefix include -DKBUILD_BASENAME=fs -DKBUILD_MODNAME=jffs2 -c -o fs/jffs2/fs.o fs/jffs2/fs.c
fs/jffs2/fs.c: In function `jffs2_new_inode':
fs/jffs2/fs.c:257: incompatible types in assignment
make[2]: *** [fs/jffs2/fs.o] Error 1

<-- snip -->


Please check whether the patch is correct.

TIA
Adrian


--- linux-2.5.48/fs/adfs/inode.c.old 2002-11-18 11:29:15.000000000 +0100
+++ linux-2.5.48/fs/adfs/inode.c 2002-11-18 11:30:43.000000000 +0100
@@ -185,7 +185,7 @@
unsigned int high, low;

if (ADFS_I(inode)->stamped == 0)
- return CURRENT_TIME;
+ return get_seconds();

high = ADFS_I(inode)->loadaddr << 24;
low = ADFS_I(inode)->execaddr;
--- linux-2.5.48/fs/afs/vlclient.c.old 2002-11-18 11:36:13.000000000 +0100
+++ linux-2.5.48/fs/afs/vlclient.c 2002-11-18 11:37:41.000000000 +0100
@@ -584,7 +584,7 @@
#endif

/* success */
- entry->ctime = CURRENT_TIME;
+ entry->ctime = get_seconds();
ret = 0;
goto done;
}
--- linux-2.5.48/fs/jffs2/fs.c.old 2002-11-18 11:49:23.000000000 +0100
+++ linux-2.5.48/fs/jffs2/fs.c 2002-11-18 11:55:43.000000000 +0100
@@ -253,7 +253,8 @@
inode->i_mode = ri->mode;
inode->i_gid = ri->gid;
inode->i_uid = ri->uid;
- inode->i_atime = inode->i_ctime = inode->i_mtime =
+ inode->i_atime.tv_sec = inode->i_ctime.tv_sec =
+ inode->i_mtime.tv_sec =
ri->atime = ri->mtime = ri->ctime = get_seconds();
inode->i_blksize = PAGE_SIZE;
inode->i_blocks = 0;


2002-11-18 13:53:06

by Andi Kleen

[permalink] [raw]
Subject: Re: [2.5 patch] fix compile breakage in fs/ after nanosecond stat timefields patch

On Mon, Nov 18, 2002 at 12:21:09PM +0100, Adrian Bunk wrote:
> Hi Andi,
>
> the patch below fixes the following compile errors in 2.5.48:

Seems correct.

Weird, I could swear I (compile) tested all these.

-Andi

2002-11-18 14:43:48

by Holger Waechtler

[permalink] [raw]
Subject: Re: [2.5 patch] fix compile breakage in fs/ after nanosecond stat timefields patch

--- fs/devfs/base.c-orig 2002-11-18 05:29:26.000000000 +0100
+++ fs/devfs/base.c 2002-11-18 15:35:45.000000000 +0100
@@ -799,9 +799,9 @@ struct symlink_type
struct devfs_inode /* This structure is for "persistent" inode storage */
{
struct dentry *dentry;
- time_t atime;
- time_t mtime;
- time_t ctime;
+ struct timespec atime;
+ struct timespec mtime;
+ struct timespec ctime;
unsigned int ino; /* Inode number as seen in the VFS */
uid_t uid;
gid_t gid;
@@ -2509,9 +2509,12 @@ static int devfs_notify_change (struct d
de->mode = inode->i_mode;
de->inode.uid = inode->i_uid;
de->inode.gid = inode->i_gid;
- de->inode.atime = inode->i_atime.tv_sec;
- de->inode.mtime = inode->i_mtime.tv_sec;
- de->inode.ctime = inode->i_ctime.tv_sec;
+ de->inode.atime.tv_sec = inode->i_atime.tv_sec;
+ de->inode.atime.tv_nsec = inode->i_atime.tv_nsec;
+ de->inode.mtime.tv_sec = inode->i_mtime.tv_sec;
+ de->inode.mtime.tv_nsec = inode->i_mtime.tv_nsec;
+ de->inode.ctime.tv_sec = inode->i_ctime.tv_sec;
+ de->inode.ctime.tv_nsec = inode->i_ctime.tv_nsec;
if ( ( iattr->ia_valid & (ATTR_MODE | ATTR_UID | ATTR_GID) ) &&
!is_devfsd_or_child (fs_info) )
devfsd_notify_de (de, DEVFSD_NOTIFY_CHANGE, inode->i_mode,
@@ -2610,12 +2613,12 @@ static struct inode *_devfs_get_vfs_inod
inode->i_mode = de->mode;
inode->i_uid = de->inode.uid;
inode->i_gid = de->inode.gid;
- inode->i_atime.tv_sec = de->inode.atime;
- inode->i_mtime.tv_sec = de->inode.mtime;
- inode->i_ctime.tv_sec = de->inode.ctime;
- inode->i_atime.tv_nsec = 0;
- inode->i_mtime.tv_nsec = 0;
- inode->i_ctime.tv_nsec = 0;
+ inode->i_atime.tv_sec = de->inode.atime.tv_sec;
+ inode->i_atime.tv_nsec = de->inode.atime.tv_nsec;
+ inode->i_mtime.tv_sec = de->inode.mtime.tv_sec;
+ inode->i_mtime.tv_nsec = de->inode.mtime.tv_nsec;
+ inode->i_ctime.tv_sec = de->inode.ctime.tv_sec;
+ inode->i_ctime.tv_nsec = de->inode.ctime.tv_nsec;
DPRINTK (DEBUG_I_GET, "(): mode: 0%o uid: %d gid: %d\n",
(int) inode->i_mode, (int) inode->i_uid, (int) inode->i_gid);
return inode;


Attachments:
devfs-timepatch.diff (2.14 kB)