From: Richard Sailer <richard@weltraumpflege.org>
Date: Mon, 3 Oct 2016 22:27:02 +0000 (+0200)
Subject: VFS: Convert K&R C -> ANSI C
X-Git-Url: http://zhaoyanbai.com/repos/%22http:/www.isc.org/icons/Bv9ARM.html?a=commitdiff_plain;h=a0814afb2e128f52ed10c9a5d4c91bf6abc22290;p=minix.git

VFS: Convert K&R C -> ANSI C

Aditionally this removes all trailing whitespaces
using: sed -i 's/[[:space:]]*$//' *.c

Change-Id: I88451fdb6f6e79e61f8aae5bd5a7f2e3538f9944
---

diff --git a/minix/servers/vfs/exec.c b/minix/servers/vfs/exec.c
index 9f67a0e1d..24704f08a 100644
--- a/minix/servers/vfs/exec.c
+++ b/minix/servers/vfs/exec.c
@@ -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");
diff --git a/minix/servers/vfs/filedes.c b/minix/servers/vfs/filedes.c
index 13bd6c8b6..1233dd9dd 100644
--- a/minix/servers/vfs/filedes.c
+++ b/minix/servers/vfs/filedes.c
@@ -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 */
 
diff --git a/minix/servers/vfs/link.c b/minix/servers/vfs/link.c
index ea6124c39..129a80595 100644
--- a/minix/servers/vfs/link.c
+++ b/minix/servers/vfs/link.c
@@ -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;
diff --git a/minix/servers/vfs/lock.c b/minix/servers/vfs/lock.c
index 8cf341934..20d410265 100644
--- a/minix/servers/vfs/lock.c
+++ b/minix/servers/vfs/lock.c
@@ -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
diff --git a/minix/servers/vfs/main.c b/minix/servers/vfs/main.c
index b690ffd33..9be33245e 100644
--- a/minix/servers/vfs/main.c
+++ b/minix/servers/vfs/main.c
@@ -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.
diff --git a/minix/servers/vfs/misc.c b/minix/servers/vfs/misc.c
index d7e30bb94..54ed7ad0f 100644
--- a/minix/servers/vfs/misc.c
+++ b/minix/servers/vfs/misc.c
@@ -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				     *
diff --git a/minix/servers/vfs/open.c b/minix/servers/vfs/open.c
index 4ace3b985..b5e3f8513 100644
--- a/minix/servers/vfs/open.c
+++ b/minix/servers/vfs/open.c
@@ -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;
diff --git a/minix/servers/vfs/path.c b/minix/servers/vfs/path.c
index ffbc67d3d..21fab6f1a 100644
--- a/minix/servers/vfs/path.c
+++ b/minix/servers/vfs/path.c
@@ -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;
diff --git a/minix/servers/vfs/pipe.c b/minix/servers/vfs/pipe.c
index 0b50c838a..bd89fedaa 100644
--- a/minix/servers/vfs/pipe.c
+++ b/minix/servers/vfs/pipe.c
@@ -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;
diff --git a/minix/servers/vfs/protect.c b/minix/servers/vfs/protect.c
index f36b71800..d94b98d5d 100644
--- a/minix/servers/vfs/protect.c
+++ b/minix/servers/vfs/protect.c
@@ -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.
diff --git a/minix/servers/vfs/request.c b/minix/servers/vfs/request.c
index e34874d35..19ae609b2 100644
--- a/minix/servers/vfs/request.c
+++ b/minix/servers/vfs/request.c
@@ -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;
 
diff --git a/minix/servers/vfs/select.c b/minix/servers/vfs/select.c
index 92b852b54..c8347d0bb 100644
--- a/minix/servers/vfs/select.c
+++ b/minix/servers/vfs/select.c
@@ -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.
diff --git a/minix/servers/vfs/vmnt.c b/minix/servers/vfs/vmnt.c
index 7e17d204f..a262de167 100644
--- a/minix/servers/vfs/vmnt.c
+++ b/minix/servers/vfs/vmnt.c
@@ -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;
diff --git a/minix/servers/vfs/vnode.c b/minix/servers/vfs/vnode.c
index a45b3d885..73d63d2a5 100644
--- a/minix/servers/vfs/vnode.c
+++ b/minix/servers/vfs/vnode.c
@@ -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;