Re: [PATCH] bsd-user: Catch up with sys/param.h requirement for machine/pmap.h

[Richard Henderson]

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 <imp@bsdimp.com 
> <mailto:imp@bsdimp.com>> wrote:
>
>     From: Muhammad Moinur Rahman <bofh@FreeBSD.org>
>
>     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 <bofh@FreeBSD.org>
>     Reviewed-by:    John Baldwin <jhb@FreeBSD.org>
>     Signed-off-by:  Warner Losh <imp@bsdimp.com>

This is exactly correct for a maintainer accepting work into a branch for a PR.


r~