diff options
author | rsc <rsc> | 2006-07-17 01:25:22 +0000 |
---|---|---|
committer | rsc <rsc> | 2006-07-17 01:25:22 +0000 |
commit | c54c79267fd4c3404000b3c545c428209e4d8a3d (patch) | |
tree | 575779df60f79558c183f08646f77fdf39f6bad8 | |
parent | 564f787e916392af7b1960d079abf110234305d4 (diff) | |
download | xv6-labs-c54c79267fd4c3404000b3c545c428209e4d8a3d.tar.gz xv6-labs-c54c79267fd4c3404000b3c545c428209e4d8a3d.tar.bz2 xv6-labs-c54c79267fd4c3404000b3c545c428209e4d8a3d.zip |
nitpicks
-rw-r--r-- | console.c | 10 | ||||
-rw-r--r-- | defs.h | 4 | ||||
-rw-r--r-- | fd.c | 4 | ||||
-rw-r--r-- | kalloc.c | 2 | ||||
-rw-r--r-- | lapic.c | 4 | ||||
-rw-r--r-- | mp.c | 4 | ||||
-rw-r--r-- | proc.c | 2 | ||||
-rw-r--r-- | trap.c | 8 | ||||
-rw-r--r-- | user1.c | 2 | ||||
-rw-r--r-- | userfs.c | 2 | ||||
-rw-r--r-- | usertests.c | 19 |
11 files changed, 29 insertions, 32 deletions
@@ -4,7 +4,7 @@ #include "spinlock.h" struct spinlock console_lock; -int paniced = 0; +int panicked = 0; int use_console_lock = 0; /* @@ -31,9 +31,9 @@ real_cons_putc(int c) unsigned short *crt = (unsigned short *) 0xB8000; // base of CGA memory int ind; - if(paniced){ + if(panicked){ cli(); - while(1) + for(;;) ; } @@ -161,7 +161,7 @@ panic(char *s) cprintf("panic: "); cprintf(s, 0); cprintf("\n", 0); - paniced = 1; // freeze other CPU - while(1) + panicked = 1; // freeze other CPU + for(;;) ; } @@ -79,8 +79,8 @@ int pipe_write(struct pipe *p, char *addr, int n); int pipe_read(struct pipe *p, char *addr, int n); // fd.c -int fd_ualloc(); -struct fd * fd_alloc(); +int fd_ualloc(void); +struct fd * fd_alloc(void); void fd_close(struct fd *); int fd_read(struct fd *fd, char *addr, int n); int fd_write(struct fd *fd, char *addr, int n); @@ -15,7 +15,7 @@ struct fd fds[NFD]; * allocate a file descriptor number for curproc. */ int -fd_ualloc() +fd_ualloc(void) { int fd; struct proc *p = curproc[cpu()]; @@ -29,7 +29,7 @@ fd_ualloc() * allocate a file descriptor structure */ struct fd * -fd_alloc() +fd_alloc(void) { int i; @@ -23,7 +23,7 @@ struct run { }; struct run *freelist; -void ktest(); +void ktest(void); /* * initialize free list of physical pages. this code @@ -108,7 +108,7 @@ lapic_write(int r, int data) void -lapic_timerinit() +lapic_timerinit(void) { cprintf("%d: init timer\n", cpu()); lapic_write(LAPIC_TDCR, LAPIC_X1); @@ -118,7 +118,7 @@ lapic_timerinit() } void -lapic_timerintr() +lapic_timerintr(void) { cprintf("%d: timer interrupt!\n", cpu()); lapic_write (LAPIC_EOI, 0); @@ -117,7 +117,7 @@ mp_detect(void) } void -mp_init() +mp_init(void) { int r; uint8_t *p, *e; @@ -193,7 +193,7 @@ mp_bcpu(void) extern void mpmain(void); void -mp_startthem() +mp_startthem(void) { extern uint8_t _binary_bootother_start[], _binary_bootother_size[]; extern int main(); @@ -311,7 +311,7 @@ proc_kill(int pid) // Exited processes remain in the zombie state // until their parent calls wait() to find out they exited. void -proc_exit() +proc_exit(void) { struct proc *p; struct proc *cp = curproc[cpu()]; @@ -10,11 +10,11 @@ struct Gatedesc idt[256]; extern unsigned vectors[]; /* vectors.S, array of 256 entry point addresses */ -extern void trapenter(); -extern void trapenter1(); +extern void trapenter(void); +extern void trapenter1(void); void -tvinit() +tvinit(void) { int i; @@ -25,7 +25,7 @@ tvinit() } void -idtinit() +idtinit(void) { lidt(idt, sizeof idt); } @@ -3,7 +3,7 @@ char buf[32]; int -main() +main(void) { int pid, fds[2], n; @@ -5,7 +5,7 @@ char buf[1024]; int -main() +main(void) { puts("userfs running\n"); block(); diff --git a/usertests.c b/usertests.c index 78b9477..370054b 100644 --- a/usertests.c +++ b/usertests.c @@ -5,7 +5,7 @@ char buf[2048]; // simple fork and pipe read/write void -pipe1() +pipe1(void) { int fds[2], pid; int seq = 0, i, n, cc, total; @@ -27,10 +27,7 @@ pipe1() close(fds[1]); total = 0; cc = 1; - while(1){ - n = read(fds[0], buf, cc); - if(n < 1) - break; + while((n = read(fds[0], buf, cc)) > 0){ for(i = 0; i < n; i++){ if((buf[i] & 0xff) != (seq++ & 0xff)){ panic("pipe1 oops 2\n"); @@ -52,19 +49,19 @@ pipe1() // meant to be run w/ at most two CPUs void -preempt() +preempt(void) { int pid1, pid2, pid3; int pfds[2]; pid1 = fork(); if(pid1 == 0) - while(1) + for(;;) ; pid2 = fork(); if(pid2 == 0) - while(1) + for(;;) ; pipe(pfds); @@ -74,7 +71,7 @@ preempt() if(write(pfds[1], "x", 1) != 1) panic("preempt write error"); close(pfds[1]); - while(1) + for(;;) ; } @@ -95,7 +92,7 @@ preempt() // try to find any races between exit and wait void -exitwait() +exitwait(void) { int i, pid; @@ -118,7 +115,7 @@ exitwait() } int -main() +main(void) { puts("usertests starting\n"); |