Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 11 Oct 2022 18:57:31 GMT
From:      Warner Losh <imp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: aa62d30d46f7 - main - sys/_pv_entry.h: Include sys/param.h
Message-ID:  <202210111857.29BIvV5s074231@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by imp:

URL: https://cgit.FreeBSD.org/src/commit/?id=aa62d30d46f78c2e3fadad28a180bf84c6f51ae1

commit aa62d30d46f78c2e3fadad28a180bf84c6f51ae1
Author:     Warner Losh <imp@FreeBSD.org>
AuthorDate: 2022-10-11 18:51:55 +0000
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: 2022-10-11 18:51:55 +0000

    sys/_pv_entry.h: Include sys/param.h
    
    sys/param.h is required for this file because it uses howmany() which is
    defined there. For the kernel, this works today because of namespace
    polllution. However, user land programs, like qemu, can include
    machine/pmap.h without having included sys/param.h (since it wasn't
    required before).
    
    Sponsored by:           Netflix
    Reviewed by:            tsoome, jhb
    Differential Revision:  https://reviews.freebsd.org/D36927
---
 sys/sys/_pv_entry.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/sys/sys/_pv_entry.h b/sys/sys/_pv_entry.h
index 3cd0d9c687f6..ec3ea24a21a5 100644
--- a/sys/sys/_pv_entry.h
+++ b/sys/sys/_pv_entry.h
@@ -36,6 +36,8 @@
 #ifndef __SYS__PV_ENTRY_H__
 #define	__SYS__PV_ENTRY_H__
 
+#include <sys/param.h>
+
 struct pmap;
 
 /*



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202210111857.29BIvV5s074231>