Re: [PATCH] bsd-user: Catch up with sys/param.h requirement for machine/pmap.h
On 12/10/22 20:56, Warner Losh wrote: From: Muhammad Moinur Rahman Some versions of FreeBSD now require sys/param.h for machine/pmap.h on x86. Include them here to meet that requirement. It does no harm on older versions, so there's no need to #ifdef it. Signed-off-by: Muhammad Moinur Rahman Reviewed-by:John Baldwin Signed-off-by: Warner Losh --- bsd-user/host/i386/host-signal.h | 1 + bsd-user/host/x86_64/host-signal.h | 1 + 2 files changed, 2 insertions(+) Reviewed-by: Philippe Mathieu-Daudé
Re: [PATCH] bsd-user: Catch up with sys/param.h requirement for machine/pmap.h
On 10/13/22 11:57, Warner Losh wrote: Note: This is my first patch that someone else wrote that I'm trying to accept and land as the bsd-user maintainer unrelated to the other upstreaming work. Please be let me know if I'm doing something wrong. I'll queue it to this month's bsd-user pull request (which likely will only include this patch). Thanks! On Wed, Oct 12, 2022 at 12:54 PM Warner Losh mailto:i...@bsdimp.com>> wrote: From: Muhammad Moinur Rahman Some versions of FreeBSD now require sys/param.h for machine/pmap.h on x86. Include them here to meet that requirement. It does no harm on older versions, so there's no need to #ifdef it. Signed-off-by: Muhammad Moinur Rahman Reviewed-by: John Baldwin Signed-off-by: Warner Losh This is exactly correct for a maintainer accepting work into a branch for a PR. r~
Re: [PATCH] bsd-user: Catch up with sys/param.h requirement for machine/pmap.h
Note: This is my first patch that someone else wrote that I'm trying to accept and land as the bsd-user maintainer unrelated to the other upstreaming work. Please be let me know if I'm doing something wrong. I'll queue it to this month's bsd-user pull request (which likely will only include this patch). Thanks! On Wed, Oct 12, 2022 at 12:54 PM Warner Losh wrote: > From: Muhammad Moinur Rahman > > Some versions of FreeBSD now require sys/param.h for machine/pmap.h on > x86. Include them here to meet that requirement. It does no harm on > older versions, so there's no need to #ifdef it. > > Signed-off-by: Muhammad Moinur Rahman > Reviewed-by:John Baldwin > Signed-off-by: Warner Losh > --- > bsd-user/host/i386/host-signal.h | 1 + > bsd-user/host/x86_64/host-signal.h | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/bsd-user/host/i386/host-signal.h > b/bsd-user/host/i386/host-signal.h > index 169e61b154c..ffdfaba534a 100644 > --- a/bsd-user/host/i386/host-signal.h > +++ b/bsd-user/host/i386/host-signal.h > @@ -9,6 +9,7 @@ > #ifndef I386_HOST_SIGNAL_H > #define I386_HOST_SIGNAL_H > > +#include > #include > #include > #include > diff --git a/bsd-user/host/x86_64/host-signal.h > b/bsd-user/host/x86_64/host-signal.h > index 47ca19f8814..32ac4e41803 100644 > --- a/bsd-user/host/x86_64/host-signal.h > +++ b/bsd-user/host/x86_64/host-signal.h > @@ -9,6 +9,7 @@ > #ifndef X86_64_HOST_SIGNAL_H > #define X86_64_HOST_SIGNAL_H > > +#include > #include > #include > #include > -- > 2.33.1 > >
[PATCH] bsd-user: Catch up with sys/param.h requirement for machine/pmap.h
From: Muhammad Moinur Rahman Some versions of FreeBSD now require sys/param.h for machine/pmap.h on x86. Include them here to meet that requirement. It does no harm on older versions, so there's no need to #ifdef it. Signed-off-by: Muhammad Moinur Rahman Reviewed-by:John Baldwin Signed-off-by: Warner Losh --- bsd-user/host/i386/host-signal.h | 1 + bsd-user/host/x86_64/host-signal.h | 1 + 2 files changed, 2 insertions(+) diff --git a/bsd-user/host/i386/host-signal.h b/bsd-user/host/i386/host-signal.h index 169e61b154c..ffdfaba534a 100644 --- a/bsd-user/host/i386/host-signal.h +++ b/bsd-user/host/i386/host-signal.h @@ -9,6 +9,7 @@ #ifndef I386_HOST_SIGNAL_H #define I386_HOST_SIGNAL_H +#include #include #include #include diff --git a/bsd-user/host/x86_64/host-signal.h b/bsd-user/host/x86_64/host-signal.h index 47ca19f8814..32ac4e41803 100644 --- a/bsd-user/host/x86_64/host-signal.h +++ b/bsd-user/host/x86_64/host-signal.h @@ -9,6 +9,7 @@ #ifndef X86_64_HOST_SIGNAL_H #define X86_64_HOST_SIGNAL_H +#include #include #include #include -- 2.33.1