]> Zhao Yanbai Git Server - minix.git/commitdiff
VFS: Convert K&R C -> ANSI C 65/3365/1
authorRichard Sailer <richard@weltraumpflege.org>
Mon, 3 Oct 2016 22:27:02 +0000 (00:27 +0200)
committerDavid van Moolenbroek <david@minix3.org>
Tue, 18 Oct 2016 12:20:21 +0000 (14:20 +0200)
Aditionally this removes all trailing whitespaces
using: sed -i 's/[[:space:]]*$//' *.c

Change-Id: I88451fdb6f6e79e61f8aae5bd5a7f2e3538f9944

14 files changed:
minix/servers/vfs/exec.c
minix/servers/vfs/filedes.c
minix/servers/vfs/link.c
minix/servers/vfs/lock.c
minix/servers/vfs/main.c
minix/servers/vfs/misc.c
minix/servers/vfs/open.c
minix/servers/vfs/path.c
minix/servers/vfs/pipe.c
minix/servers/vfs/protect.c
minix/servers/vfs/request.c
minix/servers/vfs/select.c
minix/servers/vfs/vmnt.c
minix/servers/vfs/vnode.c

index 9f67a0e1db8d922c374d7f814395e0a9246ad251..24704f08acccb917810449503dc0dfd7c77d2228 100644 (file)
@@ -295,7 +295,7 @@ int pm_exec(vir_bytes path, size_t path_len, vir_bytes frame, size_t frame_len,
        }
 
        /* ld.so is linked at 0, but it can relocate itself; we
-        * want it higher to trap NULL pointer dereferences. 
+        * want it higher to trap NULL pointer dereferences.
         * Let's put it below the stack, and reserve 10MB for ld.so.
         */
        execi.args.load_offset =
@@ -435,12 +435,12 @@ static int stack_prepare_elf(struct vfs_exec_info *execi, char *frame, size_t *f
                return ENOEXEC;
        }
 
-       /* Find first Aux vector in the stack frame. */ 
+       /* Find first Aux vector in the stack frame. */
        vap = (vir_bytes)(psp->ps_envstr + (psp->ps_nenvstr + 1));
        aux_vec = (AuxInfo *) (frame + (vap - *vsp));
        aux_vec_end = aux_vec + PMEF_AUXVECTORS;
 
-       if (((char *)aux_vec < frame) || 
+       if (((char *)aux_vec < frame) ||
                ((char *)aux_vec > (frame + *frame_size))) {
                printf("VFS: malformed stack for exec(), first AuxVector is"
                       " not on the stack.\n");
index 13bd6c8b61be5b1fe9f41f99d42a35012ec606f9..1233dd9dd1b3ed231af58e4d5d14e9d800d0c75c 100644 (file)
@@ -137,9 +137,11 @@ int get_fd(struct fproc *rfp, int start, mode_t bits, int *k, struct filp **fpt)
 /*===========================================================================*
  *                             get_filp                                     *
  *===========================================================================*/
-struct filp *get_filp(fild, locktype)
-int fild;                      /* file descriptor */
-tll_access_t locktype;
+struct filp *
+get_filp(
+       int fild,                       /* file descriptor */
+       tll_access_t locktype
+)
 {
 /* See if 'fild' refers to a valid file descr.  If so, return its filp ptr. */
 
@@ -150,10 +152,12 @@ tll_access_t locktype;
 /*===========================================================================*
  *                             get_filp2                                    *
  *===========================================================================*/
-struct filp *get_filp2(rfp, fild, locktype)
-register struct fproc *rfp;
-int fild;                      /* file descriptor */
-tll_access_t locktype;
+struct filp *
+get_filp2(
+       register struct fproc *rfp,
+       int fild,                       /* file descriptor */
+       tll_access_t locktype
+)
 {
 /* See if 'fild' refers to a valid file descr.  If so, return its filp ptr. */
   struct filp *filp;
@@ -242,9 +246,8 @@ void invalidate_filp_by_endpt(endpoint_t proc_e)
 /*===========================================================================*
  *                             lock_filp                                    *
  *===========================================================================*/
-void lock_filp(filp, locktype)
-struct filp *filp;
-tll_access_t locktype;
+void
+lock_filp(struct filp *filp, tll_access_t locktype)
 {
   struct worker_thread *org_self;
   struct vnode *vp;
@@ -287,8 +290,8 @@ tll_access_t locktype;
 /*===========================================================================*
  *                             unlock_filp                                  *
  *===========================================================================*/
-void unlock_filp(filp)
-struct filp *filp;
+void
+unlock_filp(struct filp *filp)
 {
   /* If this filp holds a soft lock on the vnode, we must be the owner */
   if (filp->filp_softlock != NULL)
@@ -312,9 +315,8 @@ struct filp *filp;
 /*===========================================================================*
  *                             unlock_filps                                 *
  *===========================================================================*/
-void unlock_filps(filp1, filp2)
-struct filp *filp1;
-struct filp *filp2;
+void
+unlock_filps(struct filp *filp1, struct filp *filp2)
 {
 /* Unlock two filps that are tied to the same vnode. As a thread can lock a
  * vnode only once, unlocking the vnode twice would result in an error. */
@@ -344,8 +346,8 @@ struct filp *filp2;
 /*===========================================================================*
  *                             close_filp                                   *
  *===========================================================================*/
-void close_filp(f)
-struct filp *f;
+void
+close_filp(struct filp *f)
 {
 /* Close a file. Will also unlock filp when done */
 
index ea6124c391ad20d5a8920cc7ab3c49c8f5b2d0ee..129a805952cf4e37bde55bcbca69d5784a00bce9 100644 (file)
@@ -362,9 +362,8 @@ int do_ftruncate(void)
 /*===========================================================================*
  *                             truncate_vnode                               *
  *===========================================================================*/
-int truncate_vnode(vp, newsize)
-struct vnode *vp;
-off_t newsize;
+int
+truncate_vnode(struct vnode *vp, off_t newsize)
 {
 /* Truncate a regular file or a pipe */
   int r;
@@ -427,10 +426,12 @@ int do_slink(void)
 /*===========================================================================*
  *                              rdlink_direct                                *
  *===========================================================================*/
-int rdlink_direct(orig_path, link_path, rfp)
-char *orig_path;
-char link_path[PATH_MAX]; /* should have length PATH_MAX */
-struct fproc *rfp;
+int
+rdlink_direct(
+       char *orig_path,
+       char link_path[PATH_MAX], /* should have length PATH_MAX */
+       struct fproc *rfp
+)
 {
 /* Perform a readlink()-like call from within the VFS */
   int r;
index 8cf3419341f5747c67ee516c905ca1d1e837dcda..20d410265595b5f05088b17661b929c3c03e8893 100644 (file)
@@ -169,7 +169,8 @@ int lock_op(int fd, int req, vir_bytes arg)
 /*===========================================================================*
  *                             lock_revive                                  *
  *===========================================================================*/
-void lock_revive()
+void
+lock_revive(void)
 {
 /* Go find all the processes that are waiting for any kind of lock and
  * revive them all.  The ones that are still blocked will block again when
index b690ffd3330669cc3f8ad744d226470a26caa754..9be33245e0192ebcd3a3275ddbe8eb83788d552f 100644 (file)
@@ -915,8 +915,8 @@ static void service_pm(void)
 /*===========================================================================*
  *                             unblock                                      *
  *===========================================================================*/
-static int unblock(rfp)
-struct fproc *rfp;
+static int
+unblock(struct fproc *rfp)
 {
 /* Unblock a process that was previously blocked on a pipe or a lock.  This is
  * done by reconstructing the original request and continuing/repeating it.
index d7e30bb948529bd1b03190324923e5e14f1c8fcc..54ed7ad0fb9644ba801c89474c1b2d293a3e33b0 100644 (file)
@@ -496,7 +496,8 @@ reqdone:
 /*===========================================================================*
  *                             pm_reboot                                    *
  *===========================================================================*/
-void pm_reboot()
+void
+pm_reboot(void)
 {
 /* Perform the VFS side of the reboot call. This call is performed from the PM
  * process context.
@@ -709,10 +710,8 @@ void pm_exit(void)
 /*===========================================================================*
  *                             pm_setgid                                    *
  *===========================================================================*/
-void pm_setgid(proc_e, egid, rgid)
-endpoint_t proc_e;
-int egid;
-int rgid;
+void
+pm_setgid(endpoint_t proc_e, int egid, int rgid)
 {
   register struct fproc *tfp;
   int slot;
@@ -728,10 +727,8 @@ int rgid;
 /*===========================================================================*
  *                             pm_setgroups                                 *
  *===========================================================================*/
-void pm_setgroups(proc_e, ngroups, groups)
-endpoint_t proc_e;
-int ngroups;
-gid_t *groups;
+void
+pm_setgroups(endpoint_t proc_e, int ngroups, gid_t *groups)
 {
   struct fproc *rfp;
   int slot;
@@ -751,10 +748,8 @@ gid_t *groups;
 /*===========================================================================*
  *                             pm_setuid                                    *
  *===========================================================================*/
-void pm_setuid(proc_e, euid, ruid)
-endpoint_t proc_e;
-int euid;
-int ruid;
+void
+pm_setuid(endpoint_t proc_e, int euid, int ruid)
 {
   struct fproc *tfp;
   int slot;
@@ -973,11 +968,11 @@ ds_event(void)
 }
 
 /* A function to be called on panic(). */
-void panic_hook(void)   
-{               
+void panic_hook(void)
+{
   printf("VFS mthread stacktraces:\n");
-  mthread_stacktraces(); 
-}         
+  mthread_stacktraces();
+}
 
 /*===========================================================================*
  *                             do_getrusage                                 *
index 4ace3b985c89430ce575a7120e93bc93bbe3e7e8..b5e3f8513fe7dfcac6f24cdc9f0e7dc3bf45aa9d 100644 (file)
@@ -682,9 +682,8 @@ int do_close(void)
 /*===========================================================================*
  *                             close_fd                                     *
  *===========================================================================*/
-int close_fd(rfp, fd_nr)
-struct fproc *rfp;
-int fd_nr;
+int
+close_fd(struct fproc *rfp, int fd_nr)
 {
 /* Perform the close(fd) system call. */
   register struct filp *rfilp;
index ffbc67d3d9ed0c635ae656ebc0486f3b5b0cb8c3..21fab6f1a725589b825c38ebea00c324d77902cd 100644 (file)
@@ -39,10 +39,8 @@ static int check_perms(endpoint_t ep, cp_grant_id_t io_gr, size_t
 /*===========================================================================*
  *                             advance                                      *
  *===========================================================================*/
-struct vnode *advance(dirp, resolve, rfp)
-struct vnode *dirp;
-struct lookup *resolve;
-struct fproc *rfp;
+struct vnode *
+advance(struct vnode *dirp, struct lookup *resolve, struct fproc *rfp)
 {
 /* Resolve a path name starting at dirp to a vnode. */
   int r;
@@ -134,9 +132,8 @@ struct fproc *rfp;
 /*===========================================================================*
  *                             eat_path                                     *
  *===========================================================================*/
-struct vnode *eat_path(resolve, rfp)
-struct lookup *resolve;
-struct fproc *rfp;
+struct vnode *
+eat_path(struct lookup *resolve, struct fproc *rfp)
 {
 /* Resolve path to a vnode. advance does the actual work. */
   struct vnode *start_dir;
@@ -148,9 +145,8 @@ struct fproc *rfp;
 /*===========================================================================*
  *                             last_dir                                     *
  *===========================================================================*/
-struct vnode *last_dir(resolve, rfp)
-struct lookup *resolve;
-struct fproc *rfp;
+struct vnode *
+last_dir(struct lookup *resolve, struct fproc *rfp)
 {
 /* Parse a path, as far as the last directory, fetch the vnode
  * for the last directory into the vnode table, and return a pointer to the
@@ -387,11 +383,8 @@ struct fproc *rfp;
 /*===========================================================================*
  *                             lookup                                       *
  *===========================================================================*/
-static int lookup(start_node, resolve, result_node, rfp)
-struct vnode *start_node;
-struct lookup *resolve;
-node_details_t *result_node;
-struct fproc *rfp;
+static int
+lookup(struct vnode *start_node, struct lookup *resolve, node_details_t *result_node, struct fproc *rfp)
 {
 /* Resolve a path name relative to start_node. */
 
@@ -581,12 +574,8 @@ struct fproc *rfp;
 /*===========================================================================*
  *                             lookup_init                                  *
  *===========================================================================*/
-void lookup_init(resolve, path, flags, vmp, vp)
-struct lookup *resolve;
-char *path;
-int flags;
-struct vmnt **vmp;
-struct vnode **vp;
+void
+lookup_init(struct lookup *resolve, char *path, int flags, struct vmnt **vmp, struct vnode **vp)
 {
   assert(vmp != NULL);
   assert(vp != NULL);
@@ -604,10 +593,8 @@ struct vnode **vp;
 /*===========================================================================*
  *                             get_name                                     *
  *===========================================================================*/
-int get_name(dirp, entry, ename)
-struct vnode *dirp;
-struct vnode *entry;
-char ename[NAME_MAX + 1];
+int
+get_name(struct vnode *dirp, struct vnode *entry, char ename[NAME_MAX + 1])
 {
 #define DIR_ENTRIES 8
 #define DIR_ENTRY_SIZE (sizeof(struct dirent) + NAME_MAX)
@@ -660,9 +647,8 @@ char ename[NAME_MAX + 1];
 /*===========================================================================*
  *                             canonical_path                               *
  *===========================================================================*/
-int canonical_path(orig_path, rfp)
-char orig_path[PATH_MAX];
-struct fproc *rfp;
+int
+canonical_path(char orig_path[PATH_MAX], struct fproc *rfp)
 {
 /* Find canonical path of a given path */
   int len = 0;
index 0b50c838a05492ca41fe7fc5d7459a801965193d..bd89fedaa263a0b31db4fdb05d8d4617af53a318 100644 (file)
@@ -147,9 +147,8 @@ static int create_pipe(int fil_des[2], int flags)
 /*===========================================================================*
  *                             map_vnode                                    *
  *===========================================================================*/
-int map_vnode(vp, map_to_fs_e)
-struct vnode *vp;
-endpoint_t map_to_fs_e;
+int
+map_vnode(struct vnode *vp, endpoint_t map_to_fs_e)
 {
   int r;
   struct vmnt *vmp;
index f36b71800c743acddb1392ab918deaee0ed47ef3..d94b98d5ded7df63e09988decf514dfe49f61e1e 100644 (file)
@@ -289,8 +289,10 @@ int forbidden(struct fproc *rfp, struct vnode *vp, mode_t access_desired)
 /*===========================================================================*
  *                             read_only                                    *
  *===========================================================================*/
-int read_only(vp)
-struct vnode *vp;              /* ptr to inode whose file sys is to be cked */
+int
+read_only(
+       struct vnode *vp                /* ptr to inode whose file sys is to be cked */
+)
 {
 /* Check to see if the file system on which the inode 'ip' resides is mounted
  * read only.  If so, return EROFS, else return OK.
index e34874d35e4b136f4ea16836a736805a354f60be..19ae609b2adf52d2a08bda5e6fd16bd68ec05e09 100644 (file)
@@ -696,10 +696,8 @@ int req_newdriver(
 /*===========================================================================*
  *                             req_putnode                                  *
  *===========================================================================*/
-int req_putnode(fs_e, inode_nr, count)
-int fs_e;
-ino_t inode_nr;
-int count;
+int
+req_putnode(int fs_e, ino_t inode_nr, int count)
 {
   message m;
 
@@ -925,12 +923,8 @@ int req_peek(endpoint_t fs_e, ino_t inode_nr, off_t pos, unsigned int bytes)
 /*===========================================================================*
  *                             req_rename                                   *
  *===========================================================================*/
-int req_rename(fs_e, old_dir, old_name, new_dir, new_name)
-endpoint_t fs_e;
-ino_t old_dir;
-char *old_name;
-ino_t new_dir;
-char *new_name;
+int
+req_rename(endpoint_t fs_e, ino_t old_dir, char *old_name, ino_t new_dir, char *new_name)
 {
   int r;
   cp_grant_id_t gid_old, gid_new;
@@ -969,10 +963,8 @@ char *new_name;
 /*===========================================================================*
  *                             req_rmdir                                    *
  *===========================================================================*/
-int req_rmdir(fs_e, inode_nr, lastc)
-endpoint_t fs_e;
-ino_t inode_nr;
-char *lastc;
+int
+req_rmdir(endpoint_t fs_e, ino_t inode_nr, char *lastc)
 {
   int r;
   cp_grant_id_t grant_id;
@@ -1138,8 +1130,8 @@ int req_stat(endpoint_t fs_e, ino_t inode_nr, endpoint_t proc_e,
 /*===========================================================================*
  *                             req_sync                                     *
  *===========================================================================*/
-int req_sync(fs_e)
-endpoint_t fs_e;
+int
+req_sync(endpoint_t fs_e)
 {
   message m;
 
@@ -1154,10 +1146,8 @@ endpoint_t fs_e;
 /*===========================================================================*
  *                             req_unlink                                   *
  *===========================================================================*/
-int req_unlink(fs_e, inode_nr, lastc)
-endpoint_t fs_e;
-ino_t inode_nr;
-char *lastc;
+int
+req_unlink(endpoint_t fs_e, ino_t inode_nr, char *lastc)
 {
   cp_grant_id_t grant_id;
   size_t len;
@@ -1186,8 +1176,8 @@ char *lastc;
 /*===========================================================================*
  *                             req_unmount                                  *
  *===========================================================================*/
-int req_unmount(fs_e)
-endpoint_t fs_e;
+int
+req_unmount(endpoint_t fs_e)
 {
   message m;
 
index 92b852b543993b5027ea152576545f8c24e5e706..c8347d0bb9ca9eec41c945ad37774b6c0da48cc6 100644 (file)
@@ -582,7 +582,7 @@ static int copy_fdsets(struct selectentry *se, int nfds, int direction)
   src_fds = (direction == FROM_PROC) ? se->vir_readfds : &se->ready_readfds;
   dst_fds = (direction == FROM_PROC) ? &se->readfds : se->vir_readfds;
   if (se->vir_readfds) {
-       r = sys_datacopy_wrapper(src_e, (vir_bytes) src_fds, dst_e, 
+       r = sys_datacopy_wrapper(src_e, (vir_bytes) src_fds, dst_e,
                        (vir_bytes) dst_fds, fd_setsize);
        if (r != OK) return(r);
   }
@@ -591,7 +591,7 @@ static int copy_fdsets(struct selectentry *se, int nfds, int direction)
   src_fds = (direction == FROM_PROC) ? se->vir_writefds : &se->ready_writefds;
   dst_fds = (direction == FROM_PROC) ? &se->writefds : se->vir_writefds;
   if (se->vir_writefds) {
-       r = sys_datacopy_wrapper(src_e, (vir_bytes) src_fds, dst_e, 
+       r = sys_datacopy_wrapper(src_e, (vir_bytes) src_fds, dst_e,
                        (vir_bytes) dst_fds, fd_setsize);
        if (r != OK) return(r);
   }
@@ -600,7 +600,7 @@ static int copy_fdsets(struct selectentry *se, int nfds, int direction)
   src_fds = (direction == FROM_PROC) ? se->vir_errorfds : &se->ready_errorfds;
   dst_fds = (direction == FROM_PROC) ? &se->errorfds : se->vir_errorfds;
   if (se->vir_errorfds) {
-       r = sys_datacopy_wrapper(src_e, (vir_bytes) src_fds, dst_e, 
+       r = sys_datacopy_wrapper(src_e, (vir_bytes) src_fds, dst_e,
                        (vir_bytes) dst_fds, fd_setsize);
        if (r != OK) return(r);
   }
@@ -1034,9 +1034,8 @@ static void select_restart_filps(void)
 /*===========================================================================*
  *                             filp_status                                  *
  *===========================================================================*/
-static void filp_status(f, status)
-struct filp *f;
-int status;
+static void
+filp_status(struct filp *f, int status)
 {
 /* Tell processes that need to know about the status of this filp. This
  * function MUST NOT block its calling thread.
@@ -1065,8 +1064,8 @@ int status;
 /*===========================================================================*
  *                             restart_proc                                 *
  *===========================================================================*/
-static void restart_proc(se)
-struct selectentry *se;
+static void
+restart_proc(struct selectentry *se)
 {
 /* Tell process about select results (if any) unless there are still results
  * pending. This function MUST NOT block its calling thread.
index 7e17d204fcaa6d3ad471e4a9bf70db9fbce8a976..a262de167177cb4d082e36eab6fc635eeb5fcb55 100644 (file)
@@ -41,7 +41,8 @@ void check_vmnt_locks_by_me(struct fproc *rfp)
 /*===========================================================================*
  *                             check_vmnt_locks                             *
  *===========================================================================*/
-void check_vmnt_locks()
+void
+check_vmnt_locks(void)
 {
   struct vmnt *vmp;
   int count = 0;
index a45b3d885dfc029041641c1be9fb910b56b4c6ae..73d63d2a55635c1a6c1f2e48a2325f7e70e49c24 100644 (file)
@@ -61,7 +61,8 @@ void check_vnode_locks_by_me(struct fproc *rfp)
 /*===========================================================================*
  *                             check_vnode_locks                            *
  *===========================================================================*/
-void check_vnode_locks()
+void
+check_vnode_locks(void)
 {
   struct vnode *vp;
   int count = 0;
@@ -80,7 +81,8 @@ void check_vnode_locks()
 /*===========================================================================*
  *                             get_free_vnode                               *
  *===========================================================================*/
-struct vnode *get_free_vnode()
+struct vnode *
+get_free_vnode(void)
 {
 /* Find a free vnode slot in the vnode table (it's not actually allocated) */
   struct vnode *vp;