parlib: Trim some header includes
authorBarret Rhoden <brho@cs.berkeley.edu>
Tue, 18 Apr 2017 20:08:57 +0000 (16:08 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Wed, 3 May 2017 16:13:02 +0000 (12:13 -0400)
Including stdio.h and to a lesser extent assert.h is a problem if we ever
include them transitively in glibc.  This happens if we include something
like uthread.h, which we'll likely need to fix glibc's libc-lock.h

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
27 files changed:
tests/childfdmap.c
tests/lock_test.c
tests/mhello.c
tests/microb_test.c
tests/ping.c
tests/pthread_barrier_test.c
tools/apps/ipconfig/ipv6.c
user/iplib/epoll.c
user/ndblib/include/ndblib/ndb.h
user/parlib/alarm.c
user/parlib/asynccall.c
user/parlib/debug.c
user/parlib/event.c
user/parlib/include/parlib/common.h
user/parlib/include/parlib/parlib.h
user/parlib/include/parlib/ros_debug.h
user/parlib/include/parlib/vcore.h
user/parlib/parlib.c
user/parlib/signal.c
user/parlib/syscall.c
user/parlib/thread0_sched.c
user/parlib/ucq.c
user/parlib/uthread.c
user/parlib/vcore.c
user/pthread/pthread.c
user/vmm/include/vmm/vmm.h
user/vmm/virtio_lguest_console.c

index 494fafb..98092af 100644 (file)
@@ -1,6 +1,6 @@
 #define _LARGEFILE64_SOURCE /* needed to use lseek64 */
 
-#include <stdio.h> 
+#include <parlib/stdio.h>
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <fcntl.h>
index da5e787..6072153 100644 (file)
@@ -27,6 +27,7 @@
 #ifdef __ros__
 
 #include <parlib/parlib.h>
+#include <parlib/stdio.h>
 #include <parlib/vcore.h>
 #include <parlib/timing.h>
 #include <parlib/spinlock.h>
index 5416dbb..2fd53fb 100644 (file)
@@ -13,6 +13,7 @@
 #include <parlib/assert.h>
 #include <parlib/event.h>
 #include <parlib/uthread.h>
+#include <unistd.h>
 
 __thread int temp;
 void *core0_tls = 0;
index b2dac1d..7bb1eea 100644 (file)
@@ -45,6 +45,7 @@
 #include <parlib/parlib.h>
 #include <parlib/vcore.h>
 #include <parlib/timing.h>
+#include <parlib/stdio.h>
 
 static uint32_t __get_pcoreid(void)
 {
index e56c64c..c3386a5 100644 (file)
@@ -18,6 +18,7 @@
 #include <parlib/timing.h>
 #include <parlib/tsc-compat.h>
 #include <parlib/printf-ext.h>
+#include <parlib/stdio.h>
 #include <parlib/alarm.h>
 #include <ndblib/ndb.h>
 
index 76abc04..dd2d303 100644 (file)
@@ -1,4 +1,4 @@
-#include <stdio.h>
+#include <parlib/stdio.h>
 #include <pthread.h>
 #include <stdlib.h>
 #include <parlib/parlib.h>
index 94cfb51..fe60a26 100644 (file)
@@ -27,6 +27,7 @@
 #include <string.h>
 #include <time.h>
 #include <unistd.h>
+#include <stdarg.h>
 
 #include "icmp.h"
 #include "ipconfig.h"
index b191f61..edc1a43 100644 (file)
@@ -34,6 +34,7 @@
 #include <parlib/uthread.h>
 #include <parlib/timing.h>
 #include <parlib/slab.h>
+#include <parlib/assert.h>
 #include <sys/user_fd.h>
 #include <sys/close_cb.h>
 #include <stdio.h>
index ae44376..4c9ba5d 100644 (file)
@@ -9,6 +9,7 @@
 #pragma once
 
 #include <fcall.h>
+#include <stdio.h>
 
 __BEGIN_DECLS
 
index 22f08e8..00d7eea 100644 (file)
@@ -27,6 +27,7 @@
 #include <parlib/alarm.h>
 #include <stdio.h>
 #include <parlib/assert.h>
+#include <parlib/stdio.h>
 #include <stdlib.h>
 #include <unistd.h>
 #include <sys/types.h>
index da73ba7..f9c18bd 100644 (file)
@@ -1,6 +1,8 @@
 #include <stdlib.h>
 
 #include <parlib/common.h>
+#include <parlib/assert.h>
+#include <parlib/stdio.h>
 #include <ros/syscall.h>
 #include <ros/ring_syscall.h>
 #include <ros/sysevent.h>
index e81acb5..1419a34 100644 (file)
@@ -1,4 +1,6 @@
 #include <parlib/common.h>
+#include <parlib/assert.h>
+#include <parlib/stdio.h>
 #include <parlib/parlib.h>
 #include <stdio.h>
 #include <unistd.h>
index daff94a..cbd6964 100644 (file)
@@ -15,6 +15,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <parlib/assert.h>
+#include <parlib/stdio.h>
 #include <errno.h>
 #include <parlib/parlib.h>
 #include <parlib/event.h>
index b812130..ede9617 100644 (file)
@@ -7,7 +7,5 @@
 #pragma once
 
 #include <ros/common.h>
-#include <parlib/assert.h>
-#include <parlib/stdio.h>
 
 #define IS_PWR2(x) ((x) && !((x) & (x - 1)))
index 32e7402..e1589e4 100644 (file)
@@ -17,7 +17,6 @@
 #include <signal.h>
 #include <stdint.h>
 #include <errno.h>
-#include <parlib/ros_debug.h>
 #include <ros/fdtap.h>
 
 __BEGIN_DECLS
index 8962fab..412ce9d 100644 (file)
@@ -6,6 +6,7 @@
 #include <unistd.h>
 #include <sys/types.h>
 #include <unistd.h>
+#include <stdio.h>
 
 __BEGIN_DECLS
 
@@ -30,4 +31,9 @@ void __print_func_exit(const char *func, const char *file);
 /* user/parlib/hexdump.c */
 void hexdump(FILE *f, void *v, int length);
 
+void fprintf_hw_tf(FILE *f, struct hw_trapframe *hw_tf);
+void fprintf_sw_tf(FILE *f, struct sw_trapframe *sw_tf);
+void fprintf_vm_tf(FILE *f, struct vm_trapframe *vm_tf);
+void print_user_context(struct user_context *ctx);
+
 __END_DECLS
index 9998692..679bed7 100644 (file)
@@ -6,7 +6,6 @@
 #include <string.h>
 #include <parlib/timing.h>
 #include <parlib/common.h>
-#include <stdio.h>
 
 __BEGIN_DECLS
 
@@ -68,10 +67,6 @@ void cpu_relax_vc(uint32_t vcoreid);
 uint32_t get_vcoreid(void);
 bool check_vcoreid(const char *str, uint32_t vcoreid);
 void __attribute__((noreturn)) vcore_yield_or_restart(void);
-void fprintf_hw_tf(FILE *f, struct hw_trapframe *hw_tf);
-void fprintf_sw_tf(FILE *f, struct sw_trapframe *sw_tf);
-void fprintf_vm_tf(FILE *f, struct vm_trapframe *vm_tf);
-void print_user_context(struct user_context *ctx);
 
 /* This works so long as we don't dlopen parlib (which we never do) */
 #define get_tlsvar_linaddr(_vcoreid, _var)                                     \
index 6fcb027..ce39cd7 100644 (file)
@@ -3,6 +3,7 @@
  * See LICENSE for details. */
 
 #include <parlib/parlib.h>
+#include <parlib/ros_debug.h>
 #include <stdlib.h>
 
 /* Control variables */
index e44516e..5daff3c 100644 (file)
@@ -32,6 +32,7 @@
 #include <parlib/signal.h>
 #include <parlib/uthread.h>
 #include <parlib/event.h>
+#include <parlib/ros_debug.h>
 #include <errno.h>
 #include <parlib/assert.h>
 #include <ros/procinfo.h>
index 8b18ef4..aaf3a1c 100644 (file)
@@ -3,6 +3,8 @@
 #include <parlib/parlib.h>
 #include <parlib/vcore.h>
 #include <parlib/serialize.h>
+#include <parlib/assert.h>
+#include <parlib/stdio.h>
 
 int sys_proc_destroy(int pid, int exitcode)
 {
@@ -86,7 +88,7 @@ int sys_self_notify(uint32_t vcoreid, unsigned int ev_type,
 int sys_send_event(struct event_queue *ev_q, struct event_msg *ev_msg,
                    uint32_t vcoreid)
 {
-       return syscall(SYS_send_event, ev_q, ev_msg, vcoreid);
+       return ros_syscall(SYS_send_event, ev_q, ev_msg, vcoreid, 0, 0, 0);
 }
 
 int sys_halt_core(unsigned long usec)
index d973c41..aed2dce 100644 (file)
@@ -14,6 +14,7 @@
 #include <parlib/uthread.h>
 #include <parlib/event.h>
 #include <parlib/arch/trap.h>
+#include <parlib/ros_debug.h>
 #include <stdlib.h>
 
 static void thread0_sched_entry(void);
index 46834b7..dd4328d 100644 (file)
@@ -12,6 +12,7 @@
 #include <parlib/spinlock.h>
 #include <sys/mman.h>
 #include <parlib/assert.h>
+#include <parlib/stdio.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <parlib/vcore.h>
index 123a5f8..eaf8d42 100644 (file)
@@ -10,7 +10,9 @@
 #include <parlib/event.h>
 #include <stdlib.h>
 #include <parlib/assert.h>
+#include <parlib/stdio.h>
 #include <parlib/arch/trap.h>
+#include <parlib/ros_debug.h>
 
 /* SCPs have a default 2LS that only manages thread 0.  Any other 2LS, such as
  * pthreads, should override sched_ops in its init code. */
index fb4b5c9..d2d9ef4 100644 (file)
@@ -15,6 +15,8 @@
 #include <ros/arch/membar.h>
 #include <parlib/printf-ext.h>
 #include <parlib/poke.h>
+#include <parlib/assert.h>
+#include <parlib/stdio.h>
 
 __thread int __vcoreid = 0;
 __thread bool __vcore_context = FALSE;
index 77a90b7..a79eb97 100644 (file)
@@ -17,6 +17,8 @@
 #include <parlib/ucq.h>
 #include <parlib/signal.h>
 #include <parlib/arch/trap.h>
+#include <parlib/ros_debug.h>
+#include <parlib/stdio.h>
 
 struct pthread_queue ready_queue = TAILQ_HEAD_INITIALIZER(ready_queue);
 struct pthread_queue active_queue = TAILQ_HEAD_INITIALIZER(active_queue);
index 566b4e4..9d09b59 100644 (file)
@@ -9,6 +9,7 @@
 #include <ros/vmm.h>
 #include <vmm/sched.h>
 #include <vmm/linux_bootparam.h>
+#include <parlib/stdio.h>
 
 // We need to reserve an area of the low 4G for thinks like tables, APIC, and
 // so on. So far, 256 MiB has been more than enough, so ...
index c261c2c..8178a45 100644 (file)
@@ -31,6 +31,7 @@
 #include <sys/uio.h>
 #include <vmm/virtio.h>
 #include <vmm/virtio_mmio.h>
+#include <parlib/stdio.h>
 
 void *cons_receiveq_fn(void *_vq) // host -> guest
 {