<sys/types.h>: Reduce vm_* types pollution in userland.
authorzrj <rimvydas.jasinskas@gmail.com>
Mon, 11 Nov 2019 17:40:49 +0000 (19:40 +0200)
committerzrj <rimvydas.jasinskas@gmail.com>
Thu, 14 Nov 2019 14:02:41 +0000 (16:02 +0200)
 Userland should not be using these types outside kernel structures or
 <vm/*.h> scope.  Adjust several userland and boot code sources to
 include <machine/types.h> where they use them (mainly for vm_offset_t).
 * sys/fbio.h: Used in several world utilities and libvgl.
 * sys/kinfo.h: Use size_t for kp_vm_map_size (header used in userland).
 * sys/emulation/ndis/pe_var.h: Used in ndiscvt(8).
 * bus/firewire/fwcrom.c: Compiled directly in fwcontrol(8).
 * mptable(1): include <machine/types.h>, should really not be using
   vm_offset_t and looks to be suspiciously broken.
 * sys/boot: For now include <machine/types.h> for vm_offset_t in
   several headers, should really not be using vm types.

 Rest of world utilities get vm types through <sys/user.h>.

 While there, move pthread type inclusion to the very end of the source,
 doing so improves readability of compiler intermediates a lot.

 Strictly visibility changes only.

sys/boot/common/bootstrap.h
sys/boot/efi/loader/loader_efi.h
sys/boot/pc32/btx/lib/btxv86.h
sys/boot/pc32/libi386/libi386.h
sys/bus/firewire/fwcrom.c
sys/emulation/ndis/pe_var.h
sys/sys/fbio.h
sys/sys/kinfo.h
sys/sys/types.h
usr.sbin/mptable/mptable.c

index 45ed5fd..6042915 100644 (file)
@@ -29,6 +29,7 @@
 #include <sys/types.h>
 #include <sys/queue.h>
 #include <sys/linker_set.h>
+#include <machine/types.h>     /* XXX for vm_offset_t */
 
 struct stat;
 
index 12ab7b8..364a5e5 100644 (file)
@@ -32,6 +32,7 @@
 #define        _LOADER_EFI_COPY_H_
 
 #include <stand.h>
+#include <machine/types.h>     /* XXX for vm_offset_t */
 
 int    efi_autoload(void);
 
index 061762d..a299d92 100644 (file)
 
 /*
  * $FreeBSD: src/sys/boot/i386/btx/lib/btxv86.h,v 1.5 1999/08/28 00:40:08 peter Exp $
- * $DragonFly: src/sys/boot/pc32/btx/lib/btxv86.h,v 1.3 2003/11/10 06:08:35 dillon Exp $
  */
 
 #ifndef _BTXV86_H_
 #define _BTXV86_H_
 
 #include <sys/types.h>
+#include <machine/types.h>     /* XXX for vm_offset_t */
 
 #define V86_ADDR   0x10000     /* Segment:offset address */
 #define V86_CALLF  0x20000     /* Emulate far call */
index a15905d..6c71d89 100644 (file)
@@ -26,6 +26,7 @@
  * $FreeBSD: src/sys/boot/i386/libi386/libi386.h,v 1.16 2003/05/01 03:56:29 peter Exp $
  */
 
+#include <machine/types.h>     /* XXX for vm_offset_t */
 
 /*
  * i386 fully-qualified device descriptor.
index fcf9e8d..e4ad766 100644 (file)
@@ -42,6 +42,7 @@
 #include <sys/systm.h>
 #include <sys/kernel.h>
 #else
+#include <machine/types.h>     /* XXX for vm_offset_t */
 #include <netinet/in.h>
 #include <fcntl.h>
 #include <stdio.h>
index 0ac691c..4c13a7a 100644 (file)
@@ -35,6 +35,8 @@
 #ifndef _PE_VAR_H_
 #define        _PE_VAR_H_
 
+#include <machine/types.h>     /* XXX for vm_offset_t */
+
 /*
  *  Image Format
  */
index e1264c3..3d8c9e7 100644 (file)
@@ -44,6 +44,7 @@
 #ifndef _SYS_IOCCOM_H_
 #include <sys/ioccom.h>
 #endif
+#include <machine/types.h>     /* for vm_offset_t */
 
 /*
  * Frame buffer ioctls (from Sprite, trimmed to essentials for X11).
index 98b6b24..aad1b7a 100644 (file)
@@ -192,13 +192,13 @@ struct kinfo_proc {
        int             kp_nice;
        unsigned int    kp_swtime;
 
-       vm_size_t       kp_vm_map_size; /* vmmap virtual size in bytes */
+       size_t          kp_vm_map_size;         /* vmmap virtual size in bytes */
        segsz_t         kp_vm_rssize;           /* resident set size in pages */
        segsz_t         kp_vm_swrss;            /* rss before last swap in pages */
        segsz_t         kp_vm_tsize;            /* text size in pages */
        segsz_t         kp_vm_dsize;            /* data size in pages */
        segsz_t         kp_vm_ssize;            /* stack size in pages */
-        u_int          kp_vm_prssize;          /* proportional rss in pages */
+       u_int           kp_vm_prssize;          /* proportional rss in pages */
 
        int             kp_jailid;
 
index a2095da..8b65d7b 100644 (file)
 #include <stdint.h>
 #endif
 #include <machine/endian.h>
-#ifndef _MACHINE_TYPES_H_
-#include <machine/types.h>
-#endif
 #include <machine/stdint.h>
-#ifndef _SYS__PTHREADTYPES_H_
-#include <sys/_pthreadtypes.h>
-#endif
 
 #if __BSD_VISIBLE
 typedef        unsigned char   u_char;
@@ -253,9 +247,15 @@ typedef    __time_t        time_t;
 typedef        __timer_t       timer_t;
 #endif
 
+#if defined(_KERNEL) || defined(_KERNEL_STRUCTURES)
+#include <machine/types.h>             /* for vm_offet_t */
+#endif
+
 #if __BSD_VISIBLE
 #include <sys/_fd_set.h>
 #include <sys/_timeval.h>
 #endif /* __BSD_VISIBLE */
 
+#include <sys/_pthreadtypes.h>         /* now POSIX thread types */
+
 #endif /* !_SYS_TYPES_H_ */
index 94cdfd1..fdcaaf7 100644 (file)
@@ -23,7 +23,6 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.sbin/mptable/mptable.c,v 1.12.2.3 2000/12/11 01:03:34 obrien Exp $
- * $DragonFly: src/usr.sbin/mptable/mptable.c,v 1.4 2008/04/20 13:44:26 swildner Exp $
  */
 
 /*
@@ -39,6 +38,7 @@
 #define OEM_PROCESSING_READY_NOT
 
 #include <sys/types.h>
+#include <machine/types.h>     /* XXX for vm_offset_t */
 #include <err.h>
 #include <fcntl.h>
 #include <inttypes.h>