[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: CWARNFLAGS



 電通大の齊藤です。

 > とりあえずこっちでパッチを作りますので、これを確認していただけますか?
 > 自信がないところもありますので。

とりあえず powerpc/ な部分だけ差分を作ってみました。
これ以外に、各アーキテクチャの machine/cpu.h に child_return() の
プロトタイプ宣言を入れてください。

403GC に関連していないファイルに対しても変更を加えてありますが、
これについては、コンパイルを試していませんのでご注意願います。

diff -urN powerpc/NOTE KNFpowerpc/NOTE
--- powerpc/NOTE	Thu Jan  1 09:00:00 1970
+++ KNFpowerpc/NOTE	Tue Dec  1 20:00:52 1998
@@ -0,0 +1,167 @@
+copyinstr.c
+
+	プロトタイプ宣言を include するために sys/systm.h を追加。
+
+	copyinstr(9) の第 1 引数は const が必要。
+
+	return に () をつける。
+
+
+copyoutstr.c
+
+	プロトタイプ宣言を include するために sys/systm.h を追加。
+
+	copyoutstr(9) の第 1 引数は const が必要。
+
+	return に () をつける。
+
+	u_char *kp に const をつける。
+
+
+copystr.c
+
+	プロトタイプ宣言を include するために sys/systm.h を追加。
+
+	copystr(9) の第 1 引数は const が必要。
+
+	return に () をつける。
+
+	u_char *kfp に const をつける。
+
+
+db_interface.c
+
+	return に () をつける。
+
+
+fubyte.c
+
+	プロトタイプ宣言を include するために sys/param.h, sys/systm.h を追加。
+
+	return に () をつける。
+
+
+fuswintr.c
+
+	プロトタイプ宣言を include するために sys/param.h, sys/systm.h を追加。
+
+	fuswintr(9) の引数は char * でなくて const void *。
+
+	return に () をつける。
+
+	ローカル変数なしの場合は compound statement の最初は空行。
+
+
+in_cksum.c
+
+	プロトタイプ宣言を include するために netinet/in.h を追加。
+
+	return に () をつける。
+
+
+mem.c
+
+	プロトタイプ宣言を include するために sys/proc.h, sys/fcntl.h,
+	sys/msgbuf を追加。
+
+	return に () をつける。
+
+	ローカル変数なしの場合は compound statement の最初は空行。
+
+	mm* 系のプロトタイプ宣言の追加。
+
+
+process_machdep.c
+
+	return に () をつける。
+
+
+subyte.c
+
+	プロトタイプ宣言を include するために sys/param.h, sys/systm.h を追加。
+
+	return に () をつける。
+
+	subyte の引数の型の修正。
+
+	subyte の第 2 引数は int なので、copyout の第 3 引数を sizeof(c) から
+	sizeof(char) に変更。
+
+suswintr.c
+
+	プロトタイプ宣言を include するために sys/param.h, sys/systm.h を追加。
+
+	return に () をつける。
+
+	ローカル変数なしの場合は compound statement の最初は空行。
+
+	suswintr の引数の型の修正。
+
+
+suword.c
+
+	プロトタイプ宣言を include するために sys/param.h, sys/systm.h を追加。
+
+	return に () をつける。
+
+	suword の引数の型の修正。
+
+	suword の第 2 引数は long なので、copyout の第 3 引数を sizeof(l) から
+	sizeof(long) に変更。
+
+
+sys_machdep.c
+
+	プロトタイプ宣言を include するために sys/mount.h, sys/syscallargs.h
+	を追加。
+
+vm_machdep.c
+
+	プロトタイプ宣言を include するために sys/cpu.h	を追加。
+
+	child_return() のプロトタイプ宣言を sys/cpu.h に入れ、関数内での
+	プロトタイプ宣言を削除。
+
+	if (xxx = foo()) の修正。修正方法は alpha に合わせた
+	(ちょっと直感的でないという話もある)。
+
+db_disasm.c
+
+	return に () をつける。
+
+
+pmap.c
+
+	ローカル変数なしの場合は compound statement の最初は空行。
+
+	return に () をつける。
+
+
+trap.c
+
+	ローカル変数なしの場合は compound statement の最初は空行。
+
+	return に () をつける。
+
+
+soft_spl.c
+
+	ローカル変数なしの場合は compound statement の最初は空行。
+
+	return に () をつける。
+
+
+openfirm.c
+
+	return に () をつける。
+
+
+ofw_machdep.c
+
+	return に () をつける。
+
+
+ipkdb_glue.c
+
+	return に () をつける。
+
diff -urN powerpc/powerpc/copyinstr.c KNFpowerpc/powerpc/copyinstr.c
--- powerpc/powerpc/copyinstr.c	Tue Oct 15 01:20:23 1996
+++ KNFpowerpc/powerpc/copyinstr.c	Tue Dec  1 18:21:24 1998
@@ -32,13 +32,14 @@
  */
 #include <sys/param.h>
 #include <sys/errno.h>
+#include <sys/systm.h>
 
 /*
  * Emulate copyinstr.
  */
 int
 copyinstr(udaddr, kaddr, len, done)
-	void *udaddr;
+	const void *udaddr;
 	void *kaddr;
 	size_t len;
 	size_t *done;
@@ -50,11 +51,11 @@
 	for (l = 0; len-- > 0; l++) {
 		if ((c = fubyte(udaddr++)) < 0) {
 			*done = l;
-			return EACCES;
+			return (EACCES);
 		}
 		if (!(*kp++ = c)) {
 			*done = l + 1;
-			return 0;
+			return (0);
 		}
 	}
 	*done = l;
diff -urN powerpc/powerpc/copyoutstr.c KNFpowerpc/powerpc/copyoutstr.c
--- powerpc/powerpc/copyoutstr.c	Tue Oct 15 01:20:23 1996
+++ KNFpowerpc/powerpc/copyoutstr.c	Tue Dec  1 18:26:46 1998
@@ -32,30 +32,31 @@
  */
 #include <sys/param.h>
 #include <sys/errno.h>
+#include <sys/systm.h>
 
 /*
  * Emulate copyoutstr.
  */
 int
 copyoutstr(kaddr, udaddr, len, done)
-	void *kaddr;
+	const void *kaddr;
 	void *udaddr;
 	size_t len;
 	size_t *done;
 {
-	u_char *kp = kaddr;
+	const u_char *kp = kaddr;
 	int l;
 	
 	for (l = 0; len-- > 0; l++) {
 		if (subyte(udaddr++, *kp) < 0) {
 			*done = l;
-			return EACCES;
+			return (EACCES);
 		}
 		if (!*kp++) {
 			*done = l + 1;
-			return 0;
+			return (0);
 		}
 	}
 	*done = l;
-	return ENAMETOOLONG;
+	return (ENAMETOOLONG);
 }
diff -urN powerpc/powerpc/copystr.c KNFpowerpc/powerpc/copystr.c
--- powerpc/powerpc/copystr.c	Tue Oct 15 01:20:24 1996
+++ KNFpowerpc/powerpc/copystr.c	Tue Dec  1 18:33:56 1998
@@ -32,27 +32,28 @@
  */
 #include <sys/param.h>
 #include <sys/errno.h>
+#include <sys/systm.h>
 
 /*
  * Emulate copyinstr.
  */
 int
 copystr(kfaddr, kdaddr, len, done)
-	void *kfaddr;
+	const void *kfaddr;
 	void *kdaddr;
 	size_t len;
 	size_t *done;
 {
-	u_char *kfp = kfaddr;
+	const u_char *kfp = kfaddr;
 	u_char *kdp = kdaddr;
 	size_t l;
 	
 	for (l = 0; len-- > 0; l++) {
 		if (!(*kdp++ = *kfp++)) {
 			*done = l + 1;
-			return 0;
+			return (0);
 		}
 	}
 	*done = l;
-	return ENAMETOOLONG;
+	return (ENAMETOOLONG);
 }
diff -urN powerpc/powerpc/db_disasm.c KNFpowerpc/powerpc/db_disasm.c
--- powerpc/powerpc/db_disasm.c	Wed Jan 28 21:22:30 1998
+++ KNFpowerpc/powerpc/db_disasm.c	Tue Dec  1 19:45:38 1998
@@ -855,5 +855,5 @@
 	class = opcode >> 26;
 	(opcodes_base[class])(opcode);
 
-	return loc + 4;
+	return (loc + 4);
 }
diff -urN powerpc/powerpc/db_interface.c KNFpowerpc/powerpc/db_interface.c
--- powerpc/powerpc/db_interface.c	Tue Sep  1 20:09:39 1998
+++ KNFpowerpc/powerpc/db_interface.c	Tue Dec  1 18:35:52 1998
@@ -36,7 +36,7 @@
 
 		bcopy(DDB_REGS->r, frame->fixreg, 32 * sizeof(u_int32_t));
 
-		return 1;
+		return (1);
 	}
-	return 0;
+	return (0);
 }
diff -urN powerpc/powerpc/fubyte.c KNFpowerpc/powerpc/fubyte.c
--- powerpc/powerpc/fubyte.c	Tue Oct 15 01:20:25 1996
+++ KNFpowerpc/powerpc/fubyte.c	Tue Dec  1 18:43:27 1998
@@ -31,16 +31,19 @@
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include <sys/param.h>
+#include <sys/systm.h>
+
 /*
  * Emulate fubyte.
  */
 int
 fubyte(addr)
-char *addr;
+	const void *addr;
 {
 	unsigned char c;
-	
-	if (copyin(addr,&c,sizeof(c)))
-		return -1;
-	return c;
+
+	if (copyin(addr, &c, sizeof(c)))
+		return (-1);
+	return (c);
 }
diff -urN powerpc/powerpc/fuswintr.c KNFpowerpc/powerpc/fuswintr.c
--- powerpc/powerpc/fuswintr.c	Tue Oct 15 01:20:25 1996
+++ KNFpowerpc/powerpc/fuswintr.c	Tue Dec  1 19:12:22 1998
@@ -31,6 +31,9 @@
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include <sys/param.h>
+#include <sys/systm.h>
+
 /*
  * Emulate fuswintr
  *
@@ -38,7 +41,8 @@
  */
 int
 fuswintr(addr)
-	char *addr;
+	const void *addr;
 {
-	return -1;
+
+	return (-1);
 }
diff -urN powerpc/powerpc/in_cksum.c KNFpowerpc/powerpc/in_cksum.c
--- powerpc/powerpc/in_cksum.c	Tue Oct 15 01:20:26 1996
+++ KNFpowerpc/powerpc/in_cksum.c	Tue Dec  1 19:14:19 1998
@@ -33,6 +33,8 @@
 #include <sys/param.h>
 #include <sys/mbuf.h>
 
+#include <netinet/in.h>
+
 /*
  * First cut for in_cksum.
  * This code is in C and should be optimized for PPC later.
@@ -80,5 +82,5 @@
 		ROL;
 	}
 	ADDCARRY;
-	return sum ^ 0xffff;
+	return (sum ^ 0xffff);
 }
diff -urN powerpc/powerpc/ipkdb_glue.c KNFpowerpc/powerpc/ipkdb_glue.c
--- powerpc/powerpc/ipkdb_glue.c	Thu Apr 17 20:20:02 1997
+++ KNFpowerpc/powerpc/ipkdb_glue.c	Tue Dec  1 20:00:54 1998
@@ -57,8 +57,9 @@
 int
 ipkdb_poll()
 {
+
 	/* for now: */
-	return 0;
+	return (0);
 }
 
 int
@@ -102,7 +103,7 @@
 #ifdef	IPKDBUSERHACK
 		asm ("mtsr %0,%1; isync" :: "n"(USER_SR), "r"(savesr));
 #endif
-		return 1;
+		return (1);
 	}
-	return 0;
+	return (0);
 }
diff -urN powerpc/powerpc/mem.c KNFpowerpc/powerpc/mem.c
--- powerpc/powerpc/mem.c	Fri Nov 20 21:24:25 1998
+++ KNFpowerpc/powerpc/mem.c	Tue Dec  1 19:19:58 1998
@@ -50,16 +50,26 @@
 #include <sys/systm.h>
 #include <sys/uio.h>
 #include <sys/malloc.h>
+#include <sys/proc.h>
+#include <sys/fcntl.h>
+#include <sys/msgbuf.h>
 
 #include <vm/vm.h>
 
+int	mmopen __P((dev_t, int, int));
+int	mmclose __P((dev_t, int, int));
+int	mmrw __P((dev_t, struct uio *, int));
+int	mmmmap __P((dev_t, int, int));
+
+
 /*ARGSUSED*/
 int
 mmopen(dev, flag, mode)
 	dev_t dev;
 	int flag, mode;
 {
-	return 0;
+
+	return (0);
 }
 
 /*ARGSUSED*/
@@ -69,7 +79,7 @@
 	int flag, mode;
 {
 
-	return 0;
+	return (0);
 }
 
 /*ARGSUSED*/
@@ -79,7 +89,7 @@
 	struct uio *uio;
 	int flags;
 {
-	vaddr_t o, v;
+	vaddr_t v;
 	u_int c;
 	struct iovec *iov;
 	int error = 0;
@@ -101,7 +111,7 @@
 			v = uio->uio_offset;
 			c = uio->uio_resid;
 			/* This doesn't allow device mapping!	XXX */
-			pmap_real_memory(&v, &c);
+			pmap_real_memory(&v, (psize_t *)&c);
 			error = uiomove((caddr_t)v, c, uio);
 			continue;
 
@@ -116,7 +126,7 @@
 		case 2:
 			if (uio->uio_rw == UIO_WRITE)
 				uio->uio_resid = 0;
-			return 0;
+			return (0);
 
 /* minor device 12 (/dev/zero) is source of nulls on read, rathole on write */
 		case 12:
@@ -133,7 +143,7 @@
 			continue;
 
 		default:
-			return ENXIO;
+			return (ENXIO);
 		}
 		if (error)
 			break;
@@ -142,7 +152,7 @@
 		uio->uio_offset += c;
 		uio->uio_resid -= c;
 	}
-	return error;
+	return (error);
 }
 
 int
@@ -150,5 +160,6 @@
         dev_t dev;
         int off, prot;
 {
-	return -1;
+
+	return (-1);
 }
diff -urN powerpc/powerpc/ofw_machdep.c KNFpowerpc/powerpc/ofw_machdep.c
--- powerpc/powerpc/ofw_machdep.c	Fri May 29 20:12:49 1998
+++ KNFpowerpc/powerpc/ofw_machdep.c	Tue Dec  1 19:59:49 1998
@@ -238,20 +238,20 @@
 			if (name[l] == '\0') {
 				/* Default partition, (or none at all) */
 				dk_setroot(od, -1);
-				return 0;
+				return (0);
 			}
 			if (name[l + 1] == '\0') {
 				switch (name[l]) {
 				case '*':
 					/* Default partition */
 					dk_setroot(od, -1);
-					return 0;
+					return (0);
 				default:
 					if (name[l] >= 'a'
 					    && name[l] < 'a' + MAXPARTITIONS) {
 						/* specified partition */
 						dk_setroot(od, name[l] - 'a');
-						return 0;
+						return (0);
 					}
 					break;
 				}
@@ -289,9 +289,9 @@
 				    od->ofb_unit != unit)
 					continue;
 				dk_setroot(od, part);
-				return 0;
+				return (0);
 			}
 		}
 	}
-	return ENODEV;
+	return (ENODEV);
 }
diff -urN powerpc/powerpc/openfirm.c KNFpowerpc/powerpc/openfirm.c
--- powerpc/powerpc/openfirm.c	Mon Nov 16 08:31:34 1998
+++ KNFpowerpc/powerpc/openfirm.c	Tue Dec  1 19:59:12 1998
@@ -61,8 +61,8 @@
 	ofw_stack();
 	args.phandle = phandle;
 	if (openfirmware(&args) == -1)
-		return 0;
-	return args.sibling;
+		return (0);
+	return (args.sibling);
 }
 
 int
@@ -84,8 +84,8 @@
 	ofw_stack();
 	args.phandle = phandle;
 	if (openfirmware(&args) == -1)
-		return 0;
-	return args.child;
+		return (0);
+	return (args.child);
 }
 
 int
@@ -107,8 +107,8 @@
 	ofw_stack();
 	args.phandle = phandle;
 	if (openfirmware(&args) == -1)
-		return 0;
-	return args.parent;
+		return (0);
+	return (args.parent);
 }
 
 int
@@ -130,8 +130,8 @@
 	ofw_stack();
 	args.ihandle = ihandle;
 	if (openfirmware(&args) == -1)
-		return -1;
-	return args.phandle;
+		return (-1);
+	return (args.phandle);
 }
 
 int
@@ -156,8 +156,8 @@
 	args.phandle = handle;
 	args.prop = prop;
 	if (openfirmware(&args) == -1)
-		return -1;
-	return args.proplen;
+		return (-1);
+	return (args.proplen);
 }
 
 int
@@ -184,18 +184,18 @@
 
 	ofw_stack();
 	if (buflen > NBPG)
-		return -1;
+		return (-1);
 	args.phandle = handle;
 	args.prop = prop;
 	args.buf = OF_buf;
 	args.buflen = buflen;
 	if (openfirmware(&args) == -1)
-		return -1;
+		return (-1);
 	if (args.size > buflen)
 		args.size = buflen;
 	if (args.size > 0)
 		ofbcopy(OF_buf, buf, args.size);
-	return args.size;
+	return (args.size);
 }
 
 int
@@ -217,8 +217,8 @@
 	ofw_stack();
 	args.device = name;
 	if (openfirmware(&args) == -1)
-		return -1;
-	return args.phandle;
+		return (-1);
+	return (args.phandle);
 }
 
 int
@@ -242,17 +242,17 @@
 	};
 
 	if (buflen > NBPG)
-		return -1;
+		return (-1);
 	args.ihandle = ihandle;
 	args.buf = OF_buf;
 	args.buflen = buflen;
 	if (openfirmware(&args) < 0)
-		return -1;
+		return (-1);
 	if (args.length > buflen)
 		args.length = buflen;
 	if (args.length > 0)
 		ofbcopy(OF_buf, buf, args.length);
-	return args.length;
+	return (args.length);
 }
 
 int
@@ -277,17 +277,17 @@
 	
 	ofw_stack();
 	if (buflen > NBPG)
-		return -1;
+		return (-1);
 	args.phandle = phandle;
 	args.buf = OF_buf;
 	args.buflen = buflen;
 	if (openfirmware(&args) < 0)
-		return -1;
+		return (-1);
 	if (args.length > buflen)
 		args.length = buflen;
 	if (args.length > 0)
 		ofbcopy(OF_buf, buf, args.length);
-	return args.length;
+	return (args.length);
 }
 
 int
@@ -318,7 +318,7 @@
 	int *ip, n;
 	
 	if (nargs > 6)
-		return -1;
+		return (-1);
 	args.nargs = nargs + 2;
 	args.nreturns = nreturns + 1;
 	args.method = method;
@@ -328,13 +328,13 @@
 		*--ip = va_arg(ap, int);
 	ofw_stack();
 	if (openfirmware(&args) == -1)
-		return -1;
+		return (-1);
 	if (args.args_n_results[nargs])
-		return args.args_n_results[nargs];
+		return (args.args_n_results[nargs]);
 	for (ip = args.args_n_results + nargs + (n = args.nreturns); --n > 0;)
 		*va_arg(ap, int *) = *--ip;
 	va_end(ap);
-	return 0;
+	return (0);
 }
 
 int
@@ -364,7 +364,7 @@
 	int *ip, n;
 	
 	if (nargs > 6)
-		return -1;
+		return (-1);
 	args.nargs = nargs + 2;
 	args.method = method;
 	args.ihandle = ihandle;
@@ -374,10 +374,10 @@
 	va_end(ap);
 	ofw_stack();
 	if (openfirmware(&args) == -1)
-		return -1;
+		return (-1);
 	if (args.args_n_results[nargs])
-		return -1;
-	return args.args_n_results[nargs + 1];
+		return (-1);
+	return (args.args_n_results[nargs + 1]);
 }
 
 int
@@ -399,12 +399,12 @@
 	
 	ofw_stack();
 	if ((l = strlen(dname)) >= NBPG)
-		return -1;
+		return (-1);
 	ofbcopy(dname, OF_buf, l + 1);
 	args.dname = OF_buf;
 	if (openfirmware(&args) == -1)
-		return -1;
-	return args.handle;
+		return (-1);
+	return (args.handle);
 }
 
 void
@@ -458,18 +458,18 @@
 		l = min(NBPG, len);
 		args.len = l;
 		if (openfirmware(&args) == -1)
-			return -1;
+			return (-1);
 		if (args.actual > 0) {
 			ofbcopy(OF_buf, addr, args.actual);
 			act += args.actual;
 		}
 		if (args.actual < l)
 			if (act)
-				return act;
+				return (act);
 			else
-				return args.actual;
+				return (args.actual);
 	}
-	return act;
+	return (act);
 }
 
 int
@@ -501,11 +501,11 @@
 		ofbcopy(addr, OF_buf, l);
 		args.len = l;
 		if (openfirmware(&args) == -1)
-			return -1;
+			return (-1);
 		l = args.actual;
 		act += l;
 	}
-	return act;
+	return (act);
 }
 
 int
@@ -532,8 +532,8 @@
 	args.poshi = (int)(pos >> 32);
 	args.poslo = (int)pos;
 	if (openfirmware(&args) == -1)
-		return -1;
-	return args.status;
+		return (-1);
+	return (args.status);
 }
 
 void
@@ -615,8 +615,8 @@
 	ofw_stack();
 	args.newfunc = newfunc;
 	if (openfirmware(&args) == -1)
-		return 0;
-	return args.oldfunc;
+		return (0);
+	return (args.oldfunc);
 }
 
 /*
diff -urN powerpc/powerpc/pmap.c KNFpowerpc/powerpc/pmap.c
--- powerpc/powerpc/pmap.c	Wed Oct 14 20:10:48 1998
+++ KNFpowerpc/powerpc/pmap.c	Tue Dec  1 19:52:17 1998
@@ -162,7 +162,8 @@
 	sr_t *sr;
 	vaddr_t addr;
 {
-	return sr[(u_int)addr >> ADDR_SR_SHFT];
+
+	return (sr[(u_int)addr >> ADDR_SR_SHFT]);
 }
 
 static inline int
@@ -173,7 +174,7 @@
 	int hash;
 	
 	hash = (sr & SR_VSID) ^ (((u_int)addr & ADDR_PIDX) >> ADDR_PIDX_SHFT);
-	return hash & ptab_mask;
+	return (hash & ptab_mask);
 }
 
 static inline int
@@ -183,6 +184,7 @@
 	vaddr_t va;
 	int which;
 {
+
 	return ptp->pte_hi
 		== (((sr & SR_VSID) << PTE_VSID_SHFT)
 		    | (((u_int)va >> ADDR_API_SHFT) & PTE_API)
@@ -198,8 +200,8 @@
 
 	bank = vm_physseg_find(atop(pa), &pg);
 	if (bank == -1)
-		return NULL;
-	return &vm_physmem[bank].pmseg.pvent[pg];
+		return (NULL);
+	return (&vm_physmem[bank].pmseg.pvent[pg]);
 }
 
 static __inline char *
@@ -210,8 +212,8 @@
 
 	bank = vm_physseg_find(atop(pa), &pg);
 	if (bank == -1)
-		return NULL;
-	return &vm_physmem[bank].pmseg.attrs[pg];
+		return (NULL);
+	return (&vm_physmem[bank].pmseg.attrs[pg]);
 }
 #endif
 
@@ -238,7 +240,7 @@
 			ptp->pte_hi &= ~PTE_HID;
 			asm volatile ("sync");
 			ptp->pte_hi |= PTE_VALID;
-			return 1;
+			return (1);
 		}
 	idx ^= ptab_mask;
 	for (ptp = ptable + idx * 8, i = 8; --i >= 0; ptp++)
@@ -247,9 +249,9 @@
 			ptp->pte_hi |= PTE_HID;
 			asm volatile ("sync");
 			ptp->pte_hi |= PTE_VALID;
-			return 1;
+			return (1);
 		}
-	return 0;
+	return (0);
 }
 
 /*
@@ -279,7 +281,7 @@
 			if (pte_insert(idx, &po->po_pte)) {
 				LIST_REMOVE(po, po_list);
 				pofree(po, 0);
-				return 1;
+				return (1);
 			}
 			/*
 			 * Have to substitute some entry. Use the primary hash for this.
@@ -307,9 +309,9 @@
 				LIST_INSERT_HEAD(potable + (idx ^ ptab_mask),
 						 po, po_list);
 			}
-			return 1;
+			return (1);
 		}
-	return 0;
+	return (0);
 }
 
 /*
@@ -609,10 +611,10 @@
 	for (pre = 0, mp = avail; mp->size; mp++) {
 		if (pa >= mp->start
 		    && pa < mp->start + mp->size)
-			return btoc(pre + (pa - mp->start));
+			return (btoc(pre + (pa - mp->start)));
 		pre += mp->size;
 	}
-	return -1;
+	return (-1);
 }
 #endif
 
@@ -623,6 +625,7 @@
 pmap_virtual_space(start, end)
 	vaddr_t *start, *end;
 {
+
 	/*
 	 * Reserve one segment for kernel virtual memory
 	 */
@@ -638,7 +641,8 @@
 u_int
 pmap_free_pages()
 {
-	return npgs;
+
+	return (npgs);
 }
 
 /*
@@ -655,12 +659,12 @@
 	if (lastidx < 0
 	    || nextavail >= avail[lastidx].start + avail[lastidx].size) {
 		if (avail[++lastidx].size == 0)
-			return FALSE;
+			return (FALSE);
 		nextavail = avail[lastidx].start;
 	}
 	*paddr = nextavail;
 	nextavail += NBPG;
-	return TRUE;
+	return (TRUE);
 }
 #endif
 
@@ -676,7 +680,7 @@
 	pm = (struct pmap *)malloc(sizeof *pm, M_VMPMAP, M_WAITOK);
 	bzero((caddr_t)pm, sizeof *pm);
 	pmap_pinit(pm);
-	return pm;
+	return (pm);
 }
 
 /*
@@ -711,6 +715,7 @@
 pmap_reference(pm)
 	struct pmap *pm;
 {
+
 	pm->pm_refs++;
 }
 
@@ -722,6 +727,7 @@
 pmap_destroy(pm)
 	struct pmap *pm;
 {
+
 	if (--pm->pm_refs == 0) {
 		pmap_release(pm);
 		free((caddr_t)pm, M_VMPMAP);
@@ -804,6 +810,7 @@
 pmap_zero_page(pa)
 	paddr_t pa;
 {
+
 	bzero((caddr_t)pa, NBPG);
 }
 
@@ -814,6 +821,7 @@
 pmap_copy_page(src, dst)
 	paddr_t src, dst;
 {
+
 	bcopy((caddr_t)src, (caddr_t)dst, NBPG);
 }
 
@@ -848,7 +856,7 @@
 		pv = pvp->pvp_pgi.pgi_freelist;
 		pvp->pvp_pgi.pgi_freelist = pv->pv_next;
 	}
-	return pv;
+	return (pv);
 }
 
 static void
@@ -921,7 +929,7 @@
 		po = pop->pop_pgi.pgi_freelist.lh_first;
 		LIST_REMOVE(po, po_list);
 	}
-	return po;
+	return (po);
 }
 
 static void
@@ -967,7 +975,7 @@
 	int s, first;
 	
 	if (!pmap_initialized)
-		return 0;
+		return (0);
 
 	s = splimp();
 
@@ -991,7 +999,7 @@
 		pv->pv_next = npv;
 	}
 	splx(s);
-	return first;
+	return (first);
 }
 
 static void
@@ -1190,14 +1198,14 @@
 	idx = pteidx(sr = ptesr(pm->pm_sr, va), va);
 	for (ptp = ptable + idx * 8, i = 8; --i >= 0; ptp++)
 		if (ptematch(ptp, sr, va, PTE_VALID))
-			return ptp;
+			return (ptp);
 	for (ptp = ptable + (idx ^ ptab_mask) * 8, i = 8; --i >= 0; ptp++)
 		if (ptematch(ptp, sr, va, PTE_VALID | PTE_HID))
-			return ptp;
+			return (ptp);
 	for (po = potable[idx].lh_first; po; po = po->po_list.le_next)
 		if (ptematch(&po->po_pte, sr, va, 0))
-			return &po->po_pte;
-	return 0;
+			return (&po->po_pte);
+	return (0);
 }
 
 /*
@@ -1214,11 +1222,11 @@
 	
 	if (!(ptp = pte_find(pm, va))) {
 		splx(s);
-		return 0;
+		return (0);
 	}
 	o = (ptp->pte_lo & PTE_RPGN) | (va & ADDR_POFF);
 	splx(s);
-	return o;
+	return (o);
 }
 
 /*
@@ -1335,14 +1343,14 @@
 	 */
 	attr = pa_to_attr(pa);
 	if (attr == NULL)
-		return 0;
+		return (0);
 	bits |= (*attr << ATTRSHFT) & bit;
 	if (bits == bit)
-		return bits;
+		return (bits);
 
 	pv = pa_to_pv(pa);
 	if (pv->pv_idx < 0)
-		return 0;
+		return (0);
 	
 	s = splimp();
 	for (; pv; pv = pv->pv_next) {
@@ -1352,7 +1360,7 @@
 				bits |= ptp->pte_lo & bit;
 				if (bits == bit) {
 					splx(s);
-					return bits;
+					return (bits);
 				}
 			}
 		for (ptp = ptable + (pv->pv_idx ^ ptab_mask) * 8, i = 8; --i >= 0; ptp++)
@@ -1361,7 +1369,7 @@
 				bits |= ptp->pte_lo & bit;
 				if (bits == bit) {
 					splx(s);
-					return bits;
+					return (bits);
 				}
 			}
 		for (po = potable[pv->pv_idx].lh_first; po; po = po->po_list.le_next)
@@ -1369,12 +1377,12 @@
 				bits |= po->po_pte.pte_lo & bit;
 				if (bits == bit) {
 					splx(s);
-					return bits;
+					return (bits);
 				}
 			}
 	}
 	splx(s);
-	return bits;
+	return (bits);
 }
 
 /*
diff -urN powerpc/powerpc/process_machdep.c KNFpowerpc/powerpc/process_machdep.c
--- powerpc/powerpc/process_machdep.c	Tue Oct 15 01:20:41 1996
+++ KNFpowerpc/powerpc/process_machdep.c	Tue Dec  1 19:24:38 1998
@@ -44,7 +44,7 @@
 	struct trapframe *tf = trapframe(p);
 	
 	tf->srr0 = (int)addr;
-	return 0;
+	return (0);
 }
 
 int
@@ -58,5 +58,5 @@
 		tf->srr1 |= PSL_SE;
 	else
 		tf->srr1 &= ~PSL_SE;
-	return 0;
+	return (0);
 }
diff -urN powerpc/powerpc/soft_spl.c KNFpowerpc/powerpc/soft_spl.c
--- powerpc/powerpc/soft_spl.c	Thu Apr 17 20:20:04 1997
+++ KNFpowerpc/powerpc/soft_spl.c	Tue Dec  1 19:54:50 1998
@@ -88,7 +88,7 @@
 	old = cpl;
 	cpl |= bits;
 
-	return old;
+	return (old);
 }
 
 static int
@@ -144,67 +144,77 @@
 		}
 		
 		asm volatile ("mtmsr %0" :: "r"(emsr));
-		return old;
+		return (old);
 	}
 }
 
 static int
 soft_splhigh()
 {
-	return splraise(-1);
+
+	return (splraise(-1));
 }
 
 static int
 soft_spl0()
 {
-	return soft_splx(0);
+
+	return (soft_splx(0));
 }
 
 static int
 soft_splbio()
 {
+
 	return splraise(SPLBIO | SPLSOFTCLOCK | SPLSOFTNET);
 }
 
 static int
 soft_splnet()
 {
+
 	return splraise(SPLNET | SPLSOFTCLOCK | SPLSOFTNET);
 }
 
 static int
 soft_spltty()
 {
+
 	return splraise(SPLTTY | SPLSOFTCLOCK | SPLSOFTNET);
 }
 
 static int
 soft_splimp()
 {
+
 	return splraise(SPLIMP | SPLSOFTCLOCK | SPLSOFTNET);
 }
 
 static int
 soft_splclock()
 {
+
 	return splraise(SPLCLOCK | SPLSOFTCLOCK | SPLSOFTNET);
 }
 
 static int
 soft_splsoftclock()
 {
+
 	return splraise(SPLSOFTCLOCK);
 }
 
 static int
 soft_splsoftnet()
 {
+
 	return splraise(SPLSOFTNET);
 }
 
 static void
 soft_setsoftclock()
 {
+
 	softclockpending = 1;
 	if (!(cpl & SPLSOFTCLOCK))
 		soft_splx(cpl);
@@ -213,6 +223,7 @@
 static void
 soft_setsoftnet()
 {
+
 	softnetpending = 1;
 	if (!(cpl & SPLSOFTNET))
 		soft_splx(cpl);
@@ -224,6 +235,7 @@
 	void (*handler) __P((void *));
 	void *arg;
 {
+
 	panic("soft_irq_establish");
 }
 
diff -urN powerpc/powerpc/subyte.c KNFpowerpc/powerpc/subyte.c
--- powerpc/powerpc/subyte.c	Tue Oct 15 01:20:41 1996
+++ KNFpowerpc/powerpc/subyte.c	Tue Dec  1 19:29:26 1998
@@ -31,15 +31,18 @@
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include <sys/param.h>
+#include <sys/systm.h>
+
 /*
  * Emulate subyte.
  */
 int
-subyte(addr,c)
-char *addr;
-unsigned char c;
+subyte(addr, c)
+	void *addr;
+	int c;
 {
-	if (copyout(&c,addr,sizeof(c)))
-		return -1;
-	return 0;
+	if (copyout(&c, addr, sizeof(char)))
+		return (-1);
+	return (0);
 }
diff -urN powerpc/powerpc/suswintr.c KNFpowerpc/powerpc/suswintr.c
--- powerpc/powerpc/suswintr.c	Tue Oct 15 01:20:41 1996
+++ KNFpowerpc/powerpc/suswintr.c	Tue Dec  1 19:32:19 1998
@@ -31,15 +31,19 @@
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include <sys/param.h>
+#include <sys/systm.h>
+
 /*
  * Emulate suswintr
  *
  * Simply return fault for all cases
  */
 int
-suswintr(addr,s)
-	char *addr;
-	unsigned short s;
+suswintr(addr, s)
+	void *addr;
+	short s;
 {
-	return -1;
+
+	return (-1);
 }
diff -urN powerpc/powerpc/suword.c KNFpowerpc/powerpc/suword.c
--- powerpc/powerpc/suword.c	Tue Oct 15 01:20:41 1996
+++ KNFpowerpc/powerpc/suword.c	Tue Dec  1 19:34:58 1998
@@ -31,15 +31,19 @@
  * ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
+#include <sys/param.h>
+#include <sys/systm.h>
+
 /*
  * Emulate suword
  */
 int
-suword(addr,l)
-char *addr;
-unsigned long l;
+suword(addr, l)
+	void *addr;
+	long l;
 {
-	if (copyout(&l,addr,sizeof(l)))
-		return -1;
-	return 0;
+
+	if (copyout(&l, addr, sizeof(long)))
+		return (-1);
+	return (0);
 }
diff -urN powerpc/powerpc/sys_machdep.c KNFpowerpc/powerpc/sys_machdep.c
--- powerpc/powerpc/sys_machdep.c	Fri Feb 27 21:16:40 1998
+++ KNFpowerpc/powerpc/sys_machdep.c	Tue Dec  1 19:37:42 1998
@@ -32,6 +32,8 @@
  */
 
 #include <sys/param.h>
+#include <sys/mount.h>
+#include <sys/syscallargs.h>
 
 int
 sys_sysarch(p, v, retval)
diff -urN powerpc/powerpc/trap.c KNFpowerpc/powerpc/trap.c
--- powerpc/powerpc/trap.c	Fri Nov 27 21:12:28 1998
+++ KNFpowerpc/powerpc/trap.c	Tue Dec  1 19:53:12 1998
@@ -386,6 +386,7 @@
 setusr(content)
 	int content;
 {
+
 	asm volatile ("isync; mtsr %0,%1; isync"
 		      :: "n"(USER_SR), "r"(content));
 }
@@ -401,7 +402,7 @@
 	faultbuf env;
 
 	if (setfault(env))
-		return EFAULT;
+		return (EFAULT);
 	while (len > 0) {
 		p = USER_ADDR + ((u_int)udaddr & ~SEGMENT_MASK);
 		l = (USER_ADDR + SEGMENT_LENGTH) - p;
@@ -414,7 +415,7 @@
 		len -= l;
 	}
 	curpcb->pcb_onfault = 0;
-	return 0;
+	return (0);
 }
 
 int
@@ -428,7 +429,7 @@
 	faultbuf env;
 
 	if (setfault(env))
-		return EFAULT;
+		return (EFAULT);
 	while (len > 0) {
 		p = USER_ADDR + ((u_int)udaddr & ~SEGMENT_MASK);
 		l = (USER_ADDR + SEGMENT_LENGTH) - p;
@@ -441,7 +442,7 @@
 		len -= l;
 	}
 	curpcb->pcb_onfault = 0;
-	return 0;
+	return (0);
 }
 
 #if defined(UVM)
@@ -466,12 +467,12 @@
 	oldfault = curpcb->pcb_onfault;
 	if (setfault(env)) {
 		curpcb->pcb_onfault = oldfault;
-		return EFAULT;
+		return (EFAULT);
 	}
 
 	bcopy(src, dst, len);
 
 	curpcb->pcb_onfault = oldfault;
-	return 0;
+	return (0);
 }
 #endif
diff -urN powerpc/powerpc/vm_machdep.c KNFpowerpc/powerpc/vm_machdep.c
--- powerpc/powerpc/vm_machdep.c	Wed Nov 11 21:16:49 1998
+++ KNFpowerpc/powerpc/vm_machdep.c	Tue Dec  1 19:44:54 1998
@@ -48,6 +48,7 @@
 #include <uvm/uvm_extern.h>
 #endif
 
+#include <machine/cpu.h>
 #include <machine/pcb.h>
 
 /*
@@ -62,7 +63,6 @@
 	struct switchframe *sf;
 	caddr_t stktop1, stktop2;
 	extern void fork_trampoline __P((void));
-	extern void child_return __P((void *));
 	struct pcb *pcb = &p2->p_addr->u_pcb;
 
 #ifdef DIAGNOSTIC
@@ -206,17 +206,19 @@
 	cseg.c_addr = 0;
 	cseg.c_size = chdr->c_cpusize;
 
-	if (error = vn_rdwr(UIO_WRITE, vp, (caddr_t)&cseg, chdr->c_seghdrsize,
+	error = vn_rdwr(UIO_WRITE, vp, (caddr_t)&cseg, chdr->c_seghdrsize,
 			    (off_t)chdr->c_hdrsize, UIO_SYSSPACE,
-			    IO_NODELOCKED|IO_UNIT, cred, NULL, p))
-		return error;
-	if (error = vn_rdwr(UIO_WRITE, vp, (caddr_t)&md_core, sizeof md_core,
+			    IO_NODELOCKED|IO_UNIT, cred, NULL, p);
+	if (error)
+		return (error);
+
+	error = vn_rdwr(UIO_WRITE, vp, (caddr_t)&md_core, sizeof md_core,
 			    (off_t)(chdr->c_hdrsize + chdr->c_seghdrsize), UIO_SYSSPACE,
-			    IO_NODELOCKED|IO_UNIT, cred, NULL, p))
-		return error;
+			    IO_NODELOCKED|IO_UNIT, cred, NULL, p);
+	if (!error)
+		chdr->c_nseg++;
 
-	chdr->c_nseg++;
-	return 0;
+	return (error);
 }
 
 /*


----------------------------------------------------------
		SAITOH Masanobu (msaitoh@spa.is.uec.ac.jp,
		                 msaitoh@netbsd.org)
		University of Electro-Communications
		ktrace(1) 普及促進委員会会員