Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 28 Oct 2022 11:00:31 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        John Baldwin <jhb@freebsd.org>
Cc:        Warner Losh <imp@freebsd.org>, src-committers <src-committers@freebsd.org>,  "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, dev-commits-src-main@freebsd.org
Subject:   Re: git: 8e9a651acbe5 - main - hyperv: Don't build on 32-bit arm
Message-ID:  <CANCZdfrBPRT5qV1qy3k=p-0jMZgD2VhRz4cfut8t5GOqbYoOEg@mail.gmail.com>
In-Reply-To: <58c8b7f5-3aba-96fa-5bc0-72dda9f0e812@FreeBSD.org>
References:  <202210271922.29RJM7N0045526@gitrepo.freebsd.org> <58c8b7f5-3aba-96fa-5bc0-72dda9f0e812@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000c1b05a05ec1b32c3
Content-Type: text/plain; charset="UTF-8"

On Fri, Oct 28, 2022, 9:59 AM John Baldwin <jhb@freebsd.org> wrote:

> On 10/27/22 12:22 PM, Warner Losh wrote:
> > The branch main has been updated by imp:
> >
> > URL:
> https://cgit.FreeBSD.org/src/commit/?id=8e9a651acbe523f03b22d9c19c0315f686eb14d6
> >
> > commit 8e9a651acbe523f03b22d9c19c0315f686eb14d6
> > Author:     Warner Losh <imp@FreeBSD.org>
> > AuthorDate: 2022-10-27 19:18:52 +0000
> > Commit:     Warner Losh <imp@FreeBSD.org>
> > CommitDate: 2022-10-27 19:18:52 +0000
> >
> >      hyperv: Don't build on 32-bit arm
> >
> >      Sponsored by:           Netflix
> > ---
> >   sys/modules/Makefile | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/sys/modules/Makefile b/sys/modules/Makefile
> > index 5d6f8a352a86..d7aa34ca7611 100644
> > --- a/sys/modules/Makefile
> > +++ b/sys/modules/Makefile
> > @@ -659,6 +659,8 @@ _sdhci_fdt=       sdhci_fdt
> >   .endif
> >   _e6000sw=   e6000sw
> >   _neta=              neta
> > +.endif
> > +.if ${MACHINE_CPUARCH} == "aarch64"
> >   _hyperv=  hyperv
> >   .endif
>
> Hmm, maybe move it up to the existing aarch64-only section above this?
>


Sure. I just did this fast after I committed something and got the ci mail

Warner

> --
> John Baldwin
>
>

--000000000000c1b05a05ec1b32c3
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" =
class=3D"gmail_attr">On Fri, Oct 28, 2022, 9:59 AM John Baldwin &lt;<a href=
=3D"mailto:jhb@freebsd.org">jhb@freebsd.org</a>&gt; wrote:<br></div><blockq=
uote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc =
solid;padding-left:1ex">On 10/27/22 12:22 PM, Warner Losh wrote:<br>
&gt; The branch main has been updated by imp:<br>
&gt; <br>
&gt; URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D8e9a651acbe5=
23f03b22d9c19c0315f686eb14d6" rel=3D"noreferrer noreferrer" target=3D"_blan=
k">https://cgit.FreeBSD.org/src/commit/?id=3D8e9a651acbe523f03b22d9c19c0315=
f686eb14d6</a><br>
&gt; <br>
&gt; commit 8e9a651acbe523f03b22d9c19c0315f686eb14d6<br>
&gt; Author:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt; AuthorDate: 2022-10-27 19:18:52 +0000<br>
&gt; Commit:=C2=A0 =C2=A0 =C2=A0Warner Losh &lt;imp@FreeBSD.org&gt;<br>
&gt; CommitDate: 2022-10-27 19:18:52 +0000<br>
&gt; <br>
&gt;=C2=A0 =C2=A0 =C2=A0 hyperv: Don&#39;t build on 32-bit arm<br>
&gt;=C2=A0 =C2=A0 =C2=A0 <br>
&gt;=C2=A0 =C2=A0 =C2=A0 Sponsored by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0Netflix<br>
&gt; ---<br>
&gt;=C2=A0 =C2=A0sys/modules/Makefile | 2 ++<br>
&gt;=C2=A0 =C2=A01 file changed, 2 insertions(+)<br>
&gt; <br>
&gt; diff --git a/sys/modules/Makefile b/sys/modules/Makefile<br>
&gt; index 5d6f8a352a86..d7aa34ca7611 100644<br>
&gt; --- a/sys/modules/Makefile<br>
&gt; +++ b/sys/modules/Makefile<br>
&gt; @@ -659,6 +659,8 @@ _sdhci_fdt=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0sdhci_fdt<=
br>
&gt;=C2=A0 =C2=A0.endif<br>
&gt;=C2=A0 =C2=A0_e6000sw=3D=C2=A0 =C2=A0e6000sw<br>
&gt;=C2=A0 =C2=A0_neta=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 n=
eta<br>
&gt; +.endif<br>
&gt; +.if ${MACHINE_CPUARCH} =3D=3D &quot;aarch64&quot;<br>
&gt;=C2=A0 =C2=A0_hyperv=3D=C2=A0 hyperv<br>
&gt;=C2=A0 =C2=A0.endif<br>
<br>
Hmm, maybe move it up to the existing aarch64-only section above this?<br><=
/blockquote></div></div><div dir=3D"auto"><br></div><div dir=3D"auto"><br><=
/div><div dir=3D"auto">Sure. I just did this fast after I committed somethi=
ng and got the ci mail</div><div dir=3D"auto"><br></div><div dir=3D"auto">W=
arner=C2=A0</div><div dir=3D"auto"><div class=3D"gmail_quote"><blockquote c=
lass=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;=
padding-left:1ex">
-- <br>
John Baldwin<br>
<br>
</blockquote></div></div></div>

--000000000000c1b05a05ec1b32c3--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrBPRT5qV1qy3k=p-0jMZgD2VhRz4cfut8t5GOqbYoOEg>