Reduce use of sys/param.h, or annotate the reason why it is needed
authorderaadt <deraadt@openbsd.org>
Wed, 1 Dec 2021 16:53:28 +0000 (16:53 +0000)
committerderaadt <deraadt@openbsd.org>
Wed, 1 Dec 2021 16:53:28 +0000 (16:53 +0000)
(pretty much MAXCOMLEN for struct process or struct core), and remove
sys/vnode.h where not needed

13 files changed:
lib/libkvm/kvm_alpha.c
lib/libkvm/kvm_amd64.c
lib/libkvm/kvm_arm.c
lib/libkvm/kvm_arm64.c
lib/libkvm/kvm_hppa.c
lib/libkvm/kvm_i386.c
lib/libkvm/kvm_m88k.c
lib/libkvm/kvm_mips64.c
lib/libkvm/kvm_powerpc.c
lib/libkvm/kvm_powerpc64.c
lib/libkvm/kvm_riscv64.c
lib/libkvm/kvm_sh.c
lib/libkvm/kvm_sparc64.c

index 7f3169e..22803b3 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_alpha.c,v 1.16 2015/12/19 18:40:30 mmcc Exp $     */
+/*     $OpenBSD: kvm_alpha.c,v 1.17 2021/12/01 16:53:28 deraadt Exp $  */
 /*     $NetBSD: kvm_alpha.c,v 1.5 1996/10/01 21:12:05 cgd Exp $        */
 
 /*
@@ -30,7 +30,8 @@
 
 #define __KVM_ALPHA_PRIVATE         /* see <machine/pte.h> */
 
-#include <sys/param.h>
+#include <sys/types.h>
+#include <sys/signal.h>
 #include <sys/proc.h>
 #include <sys/stat.h>
 #include <sys/kcore.h>
@@ -194,7 +195,7 @@ _kvm_pa2off(kvm_t *kd, paddr_t pa)
        int i;
 
        cpu_kh = kd->cpu_data;
-       ramsegs = (phys_ram_seg_t *)((char *)cpu_kh + ALIGN(sizeof *cpu_kh));
+       ramsegs = (phys_ram_seg_t *)((char *)cpu_kh + _ALIGN(sizeof *cpu_kh));
 
        off = 0;
        for (i = 0; i < cpu_kh->nmemsegs; i++) {
index 9a33bf1..bdffdbc 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_amd64.c,v 1.14 2021/09/10 00:02:43 deraadt Exp $  */
+/*     $OpenBSD: kvm_amd64.c,v 1.15 2021/12/01 16:53:28 deraadt Exp $  */
 /*     $NetBSD: kvm_x86_64.c,v 1.3 2002/06/05 22:01:55 fvdl Exp $      */
 
 /*-
@@ -38,7 +38,6 @@
  * x86-64 machine dependent routines for kvm.
  */
 
-#include <sys/param.h> /* MAXCOMLEN ALIGN */
 #include <sys/types.h>
 #include <sys/signal.h>
 #include <sys/proc.h>
@@ -194,7 +193,7 @@ _kvm_pa2off(kvm_t *kd, paddr_t pa)
        int i;
 
        cpu_kh = kd->cpu_data;
-       ramsegs = (void *)((char *)(void *)cpu_kh + ALIGN(sizeof *cpu_kh));
+       ramsegs = (void *)((char *)(void *)cpu_kh + _ALIGN(sizeof *cpu_kh));
 
        off = 0;
        for (i = 0; i < cpu_kh->nmemsegs; i++) {
index d49aaff..faa860a 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_arm.c,v 1.10 2018/05/15 10:04:44 otto Exp $       */
+/*     $OpenBSD: kvm_arm.c,v 1.11 2021/12/01 16:53:28 deraadt Exp $    */
 /*
  * Copyright (c) 2006 Miodrag Vallat.
  *
  * ARM machine dependent routines for kvm.
  */
 
-#include <sys/param.h>
+#include <sys/param.h> /* MAXCOMLEN */
+#include <sys/types.h>
 #include <sys/core.h>
 #include <sys/kcore.h>
-#include <sys/vnode.h>
 
 #include <unistd.h>
 #include <stdlib.h>
index d10c160..dab933b 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_arm64.c,v 1.1 2017/01/11 14:32:36 patrick Exp $   */
+/*     $OpenBSD: kvm_arm64.c,v 1.2 2021/12/01 16:53:28 deraadt Exp $   */
 /*
  * Copyright (c) 2006 Miodrag Vallat.
  *
  * ARM64 machine dependent routines for kvm.
  */
 
-#include <sys/param.h>
+#include <sys/param.h> /* MAXCOMLEN for sys/core.h */
+#include <sys/types.h>
 #include <sys/core.h>
 #include <sys/kcore.h>
-#include <sys/vnode.h>
 
 #include <unistd.h>
 #include <stdlib.h>
index a82379e..3dd1d22 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_hppa.c,v 1.10 2015/12/19 18:40:30 mmcc Exp $      */
+/*     $OpenBSD: kvm_hppa.c,v 1.11 2021/12/01 16:53:28 deraadt Exp $   */
 
 /*
  * Copyright (c) 2002, Miodrag Vallat.
@@ -25,7 +25,8 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#include <sys/param.h>
+#include <sys/types.h>
+#include <sys/signal.h>
 #include <sys/proc.h>
 #include <sys/stat.h>
 
index 5c68c43..6114af4 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_i386.c,v 1.27 2015/12/19 18:40:30 mmcc Exp $ */
+/*     $OpenBSD: kvm_i386.c,v 1.28 2021/12/01 16:53:28 deraadt Exp $ */
 /*     $NetBSD: kvm_i386.c,v 1.9 1996/03/18 22:33:38 thorpej Exp $     */
 
 /*-
@@ -39,7 +39,8 @@
  * vm code will one day obsolete this module.
  */
 
-#include <sys/param.h>
+#include <sys/types.h>
+#include <sys/signal.h>
 #include <sys/proc.h>
 #include <sys/stat.h>
 #include <stdlib.h>
@@ -48,6 +49,7 @@
 #include <kvm.h>
 
 #include <uvm/uvm_extern.h>
+#include <machine/param.h>
 #include <machine/vmparam.h>
 #include <machine/pmap.h>
 
index 0d5df39..ea47115 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_m88k.c,v 1.6 2015/12/19 18:40:30 mmcc Exp $       */
+/*     $OpenBSD: kvm_m88k.c,v 1.7 2021/12/01 16:53:28 deraadt Exp $    */
 /*     $NetBSD: kvm_alpha.c,v 1.2 1995/09/29 03:57:48 cgd Exp $        */
 
 /*
@@ -28,7 +28,9 @@
  * rights to redistribute these changes.
  */
 
-#include <sys/param.h>
+#include <sys/param.h> /* MAXCOMLEN */
+#include <sys/types.h>
+#include <sys/signal.h>
 #include <sys/proc.h>
 #include <sys/stat.h>
 #include <unistd.h>
index 6ab0677..ea619bb 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_mips64.c,v 1.16 2018/05/03 16:42:07 zhuk Exp $ */
+/*     $OpenBSD: kvm_mips64.c,v 1.17 2021/12/01 16:53:28 deraadt Exp $ */
 /*     $NetBSD: kvm_mips.c,v 1.3 1996/03/18 22:33:44 thorpej Exp $     */
 
 /*-
@@ -39,7 +39,8 @@
  * vm code will one day obsolete this module.
  */
 
-#include <sys/param.h>
+#include <sys/types.h>
+#include <sys/signal.h>
 #include <sys/proc.h>
 #include <sys/stat.h>
 #include <sys/sysctl.h>
index 7bfc421..8693c56 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_powerpc.c,v 1.10 2015/12/19 18:40:30 mmcc Exp $   */
+/*     $OpenBSD: kvm_powerpc.c,v 1.11 2021/12/01 16:53:28 deraadt Exp $        */
 
 /*-
  * Copyright (C) 1996 Wolfgang Solfrank.
@@ -35,8 +35,9 @@
  * PowerPC machine dependent routines for kvm.
  */
 
-#include <sys/param.h>
-#include <sys/exec.h>
+#include <sys/param.h> /* MAXCOMLEN */
+#include <sys/types.h>
+#include <sys/signal.h>
 #include <sys/proc.h>
 #include <sys/stat.h>
 #include <sys/core.h>
index 1be4f92..80239b8 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_powerpc64.c,v 1.1 2020/06/28 09:46:58 kettenis Exp $      */
+/*     $OpenBSD: kvm_powerpc64.c,v 1.2 2021/12/01 16:53:28 deraadt Exp $       */
 
 /*
  * Copyright (c) 2007 Miodrag Vallat.
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
+#include <sys/param.h> /* MAXCOMLEN */
+#include <sys/types.h>
 #include <sys/core.h>
 #include <sys/kcore.h>
-#include <sys/vnode.h>
 
 #include <unistd.h>
 #include <stdlib.h>
index 10d3d6b..68a5d64 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_riscv64.c,v 1.1 2021/04/29 20:00:18 drahn Exp $   */
+/*     $OpenBSD: kvm_riscv64.c,v 1.2 2021/12/01 16:53:28 deraadt Exp $ */
 /*
  * Copyright (c) 2006 Miodrag Vallat.
  *
  * RISCV machine dependent routines for kvm.
  */
 
-#include <sys/param.h>
+#include <sys/param.h> /* MAXCOMLEN */
+#include <sys/types.h>
 #include <sys/core.h>
 #include <sys/kcore.h>
-#include <sys/vnode.h>
 
 #include <unistd.h>
 #include <stdlib.h>
index e4a72c7..4017e85 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_sh.c,v 1.7 2020/06/28 09:45:12 kettenis Exp $     */
+/*     $OpenBSD: kvm_sh.c,v 1.8 2021/12/01 16:53:28 deraadt Exp $      */
 
 /*
  * Copyright (c) 2007 Miodrag Vallat.
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <sys/param.h>
+#include <sys/param.h> /* MAXCOMLEN */
+#include <sys/types.h>
 #include <sys/core.h>
 #include <sys/kcore.h>
-#include <sys/vnode.h>
 
 #include <unistd.h>
 #include <stdlib.h>
index 3cf5c7e..d73b80f 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: kvm_sparc64.c,v 1.11 2021/10/06 00:42:47 deraadt Exp $        */
+/*     $OpenBSD: kvm_sparc64.c,v 1.12 2021/12/01 16:53:28 deraadt Exp $        */
 /*     $NetBSD: kvm_sparc64.c,v 1.7 2001/08/05 03:33:15 matt Exp $     */
 
 /*-
@@ -39,7 +39,7 @@
  * vm code will one day obsolete this module.
  */
 
-#include <sys/param.h> /* MAXCOMLEN PAGE_SIZE */
+#include <sys/param.h> /* MAXCOMLEN */
 #include <sys/types.h>
 #include <sys/signal.h>
 #include <sys/exec.h>
@@ -53,6 +53,7 @@
 
 #include <uvm/uvm_extern.h>
 
+#include <machine/param.h>
 #include <machine/vmparam.h>
 #include <machine/pmap.h>
 #include <machine/kcore.h>