Libpuffs: use struct timespec instead of time_t

diff --git a/lib/libpuffs/link.c b/lib/libpuffs/link.c
index e797f2102920d276bce48df78cb8e32208335a61..039e820b1760a668d344f6e72c3ebb1e11d8731c 100644 (file)
--- a/lib/libpuffs/link.c
+++ b/lib/libpuffs/link.c
@@ -59,7 +59,7 @@ int fs_ftrunc(void)
        if (r) return(EINVAL);
   }
 
-  update_times(pn, CTIME | MTIME, 0);
+  update_timens(pn, CTIME | MTIME, NULL);
 
   return(r);
 }
@@ -76,7 +76,7 @@ int fs_link()
   char string[NAME_MAX + 1];
   phys_bytes len;
   struct puffs_node *pn, *pn_dir, *new_pn;
-  time_t cur_time;
+  struct timespec cur_time;
   struct puffs_kcn pkcnp;
   PUFFS_MAKECRED(pcr, &global_kcred);
   struct puffs_cn pcn = {&pkcnp, (struct puffs_cred *) __UNCONST(pcr), {0,0,0}};
@@ -141,9 +141,9 @@ int fs_link()
   
   if (r != OK) return(EINVAL);
 
-  cur_time = clock_time();
-  update_times(pn, CTIME, cur_time);
-  update_times(pn_dir, MTIME | CTIME, cur_time);
+  cur_time = clock_timespec();
+  update_timens(pn, CTIME, &cur_time);
+  update_timens(pn_dir, MTIME | CTIME, &cur_time);
 
   return(OK);
 }
@@ -208,7 +208,7 @@ int fs_rename()
   int odir, ndir;                       /* TRUE iff {old|new} file is dir */
   int same_pdir;                        /* TRUE iff parent dirs are the same */
   phys_bytes len;
-  time_t cur_time;
+  struct timespec cur_time;
 
   if (global_pu->pu_ops.puffs_node_rename == NULL)
        return(EINVAL);
@@ -373,9 +373,9 @@ int fs_rename()
   }
 
 rename_out: 
-  cur_time = clock_time();
-  update_times(old_dirp, MTIME | CTIME, cur_time);
-  update_times(new_dirp, MTIME | CTIME, cur_time);
+  cur_time = clock_timespec();
+  update_timens(old_dirp, MTIME | CTIME, &cur_time);
+  update_timens(new_dirp, MTIME | CTIME, &cur_time);
 
   /* XXX see release_node comment in fs_unlink */
   if (new_ip && new_ip->pn_count == 0) {
@@ -401,7 +401,7 @@ int fs_unlink()
  */
   int r;
   struct puffs_node *pn, *pn_dir;
-  time_t cur_time;
+  struct timespec cur_time;
   struct puffs_kcn pkcnp;
   struct puffs_cn pcn = {&pkcnp, 0, {0,0,0}};
   PUFFS_KCREDTOCRED(pcn.pcn_cred, &global_kcred);
@@ -447,9 +447,9 @@ int fs_unlink()
   }
 
   if (pn->pn_va.va_nlink != 0) {
-       cur_time = clock_time();
-       update_times(pn, CTIME, cur_time);
-       update_times(pn_dir, MTIME | CTIME, cur_time);
+       cur_time = clock_timespec();
+       update_timens(pn, CTIME, &cur_time);
+       update_timens(pn_dir, MTIME | CTIME, &cur_time);
   }
 
   /* XXX Ideally, we should check pn->pn_flags & PUFFS_NODE_REMOVED, but

diff --git a/lib/libpuffs/open.c b/lib/libpuffs/open.c
index d520ac9732f5aa9a5cf2cd6e0349e0eba9fe0beb..0a858ce1b9adfc61ae5a93d71fe3d5442e6d5a0d 100644 (file)
--- a/lib/libpuffs/open.c
+++ b/lib/libpuffs/open.c
@@ -27,7 +27,7 @@ int fs_create()
   PUFFS_MAKECRED(pcr, &global_kcred);
   struct puffs_cn pcn = {&pkcnp, (struct puffs_cred *) __UNCONST(pcr), {0,0,0}};
   struct vattr va;
-  time_t cur_time;
+  struct timespec cur_time;
   int len;
 
   if (global_pu->pu_ops.puffs_node_create == NULL) {
@@ -59,14 +59,14 @@ int fs_create()
   memset(&pni, 0, sizeof(pni));
   pni.pni_cookie = (void** )&pn;
 
-  cur_time = clock_time();
+  cur_time = clock_timespec();
   
   memset(&va, 0, sizeof(va));
   va.va_type = VREG;
   va.va_mode = omode;
   va.va_uid = caller_uid;
   va.va_gid = caller_gid;
-  va.va_atime.tv_sec = va.va_mtime.tv_sec = va.va_ctime.tv_sec = cur_time;
+  va.va_atime = va.va_mtime = va.va_ctime = cur_time;
 
   if (buildpath) {
        r = puffs_path_pcnbuild(global_pu, &pcn, pn_dir);
@@ -96,7 +96,7 @@ int fs_create()
   /* Open pnode */
   pn->pn_count++;
 
-  update_times(pn_dir, MTIME | CTIME, cur_time);
+  update_timens(pn_dir, MTIME | CTIME, &cur_time);
 
   /* Reply message */
   fs_m_out.RES_INODE_NR = pn->pn_va.va_fileid;
@@ -124,7 +124,7 @@ int fs_mknod()
   PUFFS_MAKECRED(pcr, &global_kcred);
   struct puffs_cn pcn = {&pkcnp, (struct puffs_cred *) __UNCONST(pcr), {0,0,0}};
   struct vattr va;
-  time_t cur_time;
+  struct timespec cur_time;
   int len;
 
   if (global_pu->pu_ops.puffs_node_mknod == NULL) {
@@ -154,7 +154,7 @@ int fs_mknod()
   memset(&pni, 0, sizeof(pni));
   pni.pni_cookie = (void** )&pn;
 
-  cur_time = clock_time();
+  cur_time = clock_timespec();
 
   memset(&va, 0, sizeof(va));
   va.va_type = VDIR;
@@ -162,7 +162,7 @@ int fs_mknod()
   va.va_uid = caller_uid;
   va.va_gid = caller_gid;
   va.va_rdev = (dev_t) fs_m_in.REQ_DEV;
-  va.va_atime.tv_sec = va.va_mtime.tv_sec = va.va_ctime.tv_sec = cur_time;
+  va.va_atime = va.va_mtime = va.va_ctime = cur_time;
 
   if (buildpath) {
        if (puffs_path_pcnbuild(global_pu, &pcn, pn_dir) != 0) {
@@ -188,7 +188,7 @@ int fs_mknod()
        return(r);
   }
 
-  update_times(pn_dir, MTIME | CTIME, cur_time);
+  update_timens(pn_dir, MTIME | CTIME, &cur_time);
 
   return(OK);
 }
@@ -207,7 +207,7 @@ int fs_mkdir()
   PUFFS_MAKECRED(pcr, &global_kcred);
   struct puffs_cn pcn = {&pkcnp, (struct puffs_cred *) __UNCONST(pcr), {0,0,0}};
   struct vattr va;
-  time_t cur_time;
+  struct timespec cur_time;
   int len;
 
   if (global_pu->pu_ops.puffs_node_mkdir == NULL) {
@@ -234,7 +234,7 @@ int fs_mkdir()
   if ((pn_dir = puffs_pn_nodewalk(global_pu, 0, &fs_m_in.REQ_INODE_NR)) == NULL)
        return(ENOENT);
   
-  cur_time = clock_time();
+  cur_time = clock_timespec();
 
   memset(&pni, 0, sizeof(pni));
   pni.pni_cookie = (void** )&pn;
@@ -244,7 +244,7 @@ int fs_mkdir()
   va.va_mode = (mode_t) fs_m_in.REQ_MODE;
   va.va_uid = caller_uid;
   va.va_gid = caller_gid;
-  va.va_atime.tv_sec = va.va_mtime.tv_sec = va.va_ctime.tv_sec = cur_time;
+  va.va_atime = va.va_mtime = va.va_ctime = cur_time;
 
   if (buildpath) {
        r = puffs_path_pcnbuild(global_pu, &pcn, pn_dir);
@@ -271,7 +271,7 @@ int fs_mkdir()
        return(r);
   }
 
-  update_times(pn_dir, MTIME | CTIME, cur_time);
+  update_timens(pn_dir, MTIME | CTIME, &cur_time);
 
   return(OK);
 }
@@ -341,7 +341,7 @@ int fs_slink()
   va.va_mode = (mode_t) (I_SYMBOLIC_LINK | RWX_MODES);
   va.va_uid = caller_uid;
   va.va_gid = caller_gid;
-  va.va_atime.tv_sec = va.va_mtime.tv_sec = va.va_ctime.tv_sec = clock_time();
+  va.va_atime = va.va_mtime = va.va_ctime = clock_timespec();
 
   if (buildpath) {
        r = puffs_path_pcnbuild(global_pu, &pcn, pn_dir);

diff --git a/lib/libpuffs/protect.c b/lib/libpuffs/protect.c
index f4abe5b112d13be1317bda4b6a54e4aee204f2d8..0c46dc74d0a30278ff876d573b406fb18577608d 100644 (file)
--- a/lib/libpuffs/protect.c
+++ b/lib/libpuffs/protect.c
@@ -33,8 +33,7 @@ int fs_chmod()
   puffs_vattr_null(&va);
   /* Clear setgid bit if file is not in caller's grp */
   va.va_mode = (pn->pn_va.va_mode & ~ALL_MODES) | (mode & ALL_MODES);
-  va.va_ctime.tv_nsec = 0;
-  va.va_ctime.tv_sec = clock_time();
+  va.va_ctime = clock_timespec();
 
   if (global_pu->pu_ops.puffs_node_setattr(global_pu, pn, &va, pcr) != 0)
        return(EINVAL);
@@ -64,8 +63,7 @@ int fs_chown()
        va.va_uid = fs_m_in.REQ_UID;
        va.va_gid = fs_m_in.REQ_GID;
        va.va_mode = pn->pn_va.va_mode & ~(I_SET_UID_BIT | I_SET_GID_BIT);
-       va.va_ctime.tv_nsec = 0;
-       va.va_ctime.tv_sec = clock_time();
+       va.va_ctime = clock_timespec();
 
        if (global_pu->pu_ops.puffs_node_setattr(global_pu, pn, &va, pcr) != 0)
                return(EINVAL);

diff --git a/lib/libpuffs/proto.h b/lib/libpuffs/proto.h
index 41da711bea8ac8ab836935e3229b64d8100ed3e8..c3758a7b7c082a59c54ed890742460f2cc341c80 100644 (file)
--- a/lib/libpuffs/proto.h
+++ b/lib/libpuffs/proto.h
@@ -3,6 +3,7 @@
 
 struct puffs_usermount;
 struct puffs_node;
+struct timespec;
 
 /* Function prototypes. */
 
@@ -67,8 +68,8 @@ int fs_utime(void);
 int no_sys(void);
 void mfs_nul_f(const char *file, int line, char *str, unsigned int len,
        unsigned int maxlen);
-time_t clock_time(void);
-int update_times(struct puffs_node *pn, int fl, time_t t);
+struct timespec clock_timespec(void);
+int update_timens(struct puffs_node *pn, int fl, struct timespec *);
 void lpuffs_debug(const char *format, ...);
 
 #endif /* PUFFS_PROTO_H */

diff --git a/lib/libpuffs/read.c b/lib/libpuffs/read.c
index 10d83f5f03cf61b64a7e03779209bb085c18c929..641a41972cb383078c2cb1ca7fc7cf9aa82b7907 100644 (file)
--- a/lib/libpuffs/read.c
+++ b/lib/libpuffs/read.c
@@ -68,7 +68,7 @@ int fs_readwrite(void)
        if (bytes_done) {
                r = sys_safecopyto(VFS_PROC_NR, gid, (vir_bytes) 0,
                                   (vir_bytes) rw_buf, bytes_done);
-               update_times(pn, ATIME, 0);
+               update_timens(pn, ATIME, NULL);
        }
   } else if (rw_flag == WRITING) {
        /* At first try to change vattr */
@@ -78,8 +78,8 @@ int fs_readwrite(void)
        puffs_vattr_null(&va);
        if ( (pos + bytes_left) > pn->pn_va.va_size)
                va.va_size = bytes_left + pos;
-       va.va_ctime.tv_sec = va.va_mtime.tv_sec = clock_time();
-       va.va_atime.tv_sec = pn->pn_va.va_atime.tv_sec;
+       va.va_ctime = va.va_mtime = clock_timespec();
+       va.va_atime = pn->pn_va.va_atime;
 
        r = global_pu->pu_ops.puffs_node_setattr(global_pu, pn, &va, pcr);
        if (r) return(EINVAL);
@@ -169,7 +169,7 @@ int fs_getdents(void)
        if (r != OK) return(r);
   }
 
-  update_times(pn, ATIME, 0);
+  update_timens(pn, ATIME, NULL);
 
   fs_m_out.RES_NBYTES = written;
   fs_m_out.RES_SEEK_POS_LO = pos;

diff --git a/lib/libpuffs/stadir.c b/lib/libpuffs/stadir.c
index ddf9acf9b0bf3171b786d27ac8d2a33f1d6c9cb8..a22faa77348b2bbdc924bad4f204f3869015dfef 100644 (file)
--- a/lib/libpuffs/stadir.c
+++ b/lib/libpuffs/stadir.c
@@ -81,9 +81,15 @@ int fs_stat()
   statbuf.st_gid = va.va_gid;
   statbuf.st_rdev = (s ? va.va_rdev : NO_DEV);
   statbuf.st_size = va.va_size;
-  statbuf.st_atime = va.va_atime.tv_sec;
-  statbuf.st_mtime = va.va_mtime.tv_sec;
-  statbuf.st_ctime = va.va_ctime.tv_sec;
+  statbuf.st_atimespec = va.va_atime;
+  statbuf.st_mtimespec = va.va_mtime;
+  statbuf.st_ctimespec = va.va_ctime;
+
+  statbuf.st_birthtimespec = va.va_birthtime;
+  statbuf.st_blksize = va.va_blocksize;
+  statbuf.st_blocks = va.va_bytes / va.va_blocksize;
+  statbuf.st_flags = va.va_flags;
+  statbuf.st_gen = va.va_gen;
 
   /* Copy the struct to user space. */
   r = sys_safecopyto(fs_m_in.m_source, (cp_grant_id_t) fs_m_in.REQ_GRANT,

diff --git a/lib/libpuffs/subr.c b/lib/libpuffs/subr.c
index c48b2fe0fd00df8318754ea856f8389c71f5febd..3ccd7be331fc2ef6f482951b91f833e19c3d64c0 100644 (file)
--- a/lib/libpuffs/subr.c
+++ b/lib/libpuffs/subr.c
@@ -292,10 +292,9 @@ puffs_stat2vattr(struct vattr *va, const struct stat *sb)
        va->va_fsid = sb->st_dev;
        va->va_fileid = sb->st_ino;
        va->va_size = sb->st_size;
-        va->va_atime.tv_nsec = va->va_mtime.tv_nsec = va->va_ctime.tv_nsec = 0;
-       va->va_atime.tv_sec = sb->st_atime;
-       va->va_ctime.tv_sec = sb->st_ctime;
-       va->va_mtime.tv_sec = sb->st_mtime;
+       va->va_atime = sb->st_atimespec;
+       va->va_ctime = sb->st_ctimespec;
+       va->va_mtime = sb->st_mtimespec;
        va->va_blocksize = sb->st_blksize;
        va->va_birthtime = sb->st_birthtimespec;
        va->va_gen = sb->st_gen;

diff --git a/lib/libpuffs/time.c b/lib/libpuffs/time.c
index df986277ff0a4f6af8038e50978089dbc0be5461..a5a26727f477901fcee1573084694ec35e597ffe 100644 (file)
--- a/lib/libpuffs/time.c
+++ b/lib/libpuffs/time.c
@@ -30,10 +30,10 @@ int fs_utime()
         return(EINVAL);
   
   puffs_vattr_null(&va);
-  va.va_atime.tv_nsec = va.va_mtime.tv_nsec = va.va_ctime.tv_nsec = 0;
+  va.va_atime.tv_nsec = va.va_mtime.tv_nsec = 0;
   va.va_atime.tv_sec = fs_m_in.REQ_ACTIME;
   va.va_mtime.tv_sec = fs_m_in.REQ_MODTIME;
-  va.va_ctime.tv_sec = clock_time();
+  va.va_ctime = clock_timespec();
 
   if (global_pu->pu_ops.puffs_node_setattr(global_pu, pn, &va, pcr) != 0)
        return(EINVAL);

diff --git a/lib/libpuffs/utility.c b/lib/libpuffs/utility.c
index 401df9c2ec62b13b942245a3bed2368e77f69cf6..03667561aa997b3dd6c08f8721a182a6c947606d 100644 (file)
--- a/lib/libpuffs/utility.c
+++ b/lib/libpuffs/utility.c
@@ -4,6 +4,7 @@
 
 #include "fs.h"
 
+#include <assert.h>
 #include <stdarg.h>
 
 #include "puffs.h"
@@ -35,35 +36,42 @@ void mfs_nul_f(const char *file, int line, char *str, unsigned int len,
 
 
 /*===========================================================================*
- *                             clock_time                                   *
+ *                             clock_timespec                               *
  *===========================================================================*/
-time_t clock_time()
+struct timespec clock_timespec()
 {
 /* This routine returns the time in seconds since 1.1.1970.  MINIX is an
  * astrophysically naive system that assumes the earth rotates at a constant
  * rate and that such things as leap seconds do not exist.
  */
+  static long system_hz = 0;
 
   register int k;
+  struct timespec tv;
   clock_t uptime;
   clock_t realtime;
   time_t boottime;
 
+  if (system_hz == 0) system_hz = sys_hz();
   if ((k=getuptime(&uptime, &realtime, &boottime)) != OK)
-       panic("clock_time: getuptme2 failed: %d", k);
+       panic("clock_timespec: getuptime failed: %d", k);
 
-  return( (time_t) (boottime + (realtime/sys_hz())));
+  tv.tv_sec = (time_t) (boottime + (realtime/system_hz));
+  /* We do not want to overflow, and system_hz can be as high as 50kHz */
+  assert(system_hz < LONG_MAX/40000);
+  tv.tv_nsec = (realtime%system_hz) * 40000 / system_hz * 25000;
+  return tv;
 }
 
 
 /*===========================================================================*
- *                             update_times                                 *
+ *                             update_timens                                *
  *===========================================================================*/
-int update_times(struct puffs_node *pn, int flags, time_t t)
+int update_timens(struct puffs_node *pn, int flags, struct timespec *t)
 {
   int r;
   struct vattr va;
-  time_t new_time;
+  struct timespec new_time;
   PUFFS_MAKECRED(pcr, &global_kcred);
 
   if (!flags)
@@ -72,28 +80,22 @@ int update_times(struct puffs_node *pn, int flags, time_t t)
   if (global_pu->pu_ops.puffs_node_setattr == NULL)
        return EINVAL;
 
-  new_time = t != 0 ? t : clock_time();
+  new_time = t != NULL ? *t : clock_timespec();
   
   puffs_vattr_null(&va);
   /* librefuse modifies atime and mtime together,
    * so set old values to avoid setting either one
    * to PUFFS_VNOVAL (set by puffs_vattr_null).
    */
-  va.va_atime.tv_sec = pn->pn_va.va_atime.tv_sec;
-  va.va_mtime.tv_sec = pn->pn_va.va_mtime.tv_sec;
-
-  if (flags & ATIME) {
-       va.va_atime.tv_sec = new_time;
-       va.va_atime.tv_nsec = 0;
-  }
-  if (flags & MTIME) {
-       va.va_mtime.tv_sec = new_time;
-       va.va_mtime.tv_nsec = 0;
-  }
-  if (flags & CTIME) {
-       va.va_ctime.tv_sec = new_time;
-       va.va_ctime.tv_nsec = 0;
-  }
+  va.va_atime = pn->pn_va.va_atime;
+  va.va_mtime = pn->pn_va.va_mtime;
+
+  if (flags & ATIME)
+       va.va_atime = new_time;
+  if (flags & MTIME)
+       va.va_mtime = new_time;
+  if (flags & CTIME)
+       va.va_ctime = new_time;
 
   r = global_pu->pu_ops.puffs_node_setattr(global_pu, pn, &va, pcr);