Date: Sun, 11 Dec 2022 07:57:12 -0700 From: Warner Losh <imp@bsdimp.com> To: Cy Schubert <Cy.Schubert@cschubert.com> Cc: Piotr Kubaj <pkubaj@freebsd.org>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 83bf6ab56829 - main - uname: switch machine to HW_MACHINE_ARCH Message-ID: <CANCZdfrh_07uc_EHAPEihA=75H2akBxYNRLGb_4HX1kiGL6hTA@mail.gmail.com> In-Reply-To: <20221211143724.4D42C415@slippy.cwsent.com> References: <202212111208.2BBC835O095410@gitrepo.freebsd.org> <20221211143724.4D42C415@slippy.cwsent.com>
index | next in thread | previous in thread | raw e-mail
[-- Attachment #1 --] On Sun, Dec 11, 2022 at 7:37 AM Cy Schubert <Cy.Schubert@cschubert.com> wrote: > In message <202212111208.2BBC835O095410@gitrepo.freebsd.org>, Piotr Kubaj > write > s: > > The branch main has been updated by pkubaj (ports committer): > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=83bf6ab568293e325f437342cdb87a62 > > 6353e27c > > > > commit 83bf6ab568293e325f437342cdb87a626353e27c > > Author: Piotr Kubaj <pkubaj@FreeBSD.org> > > AuthorDate: 2022-12-11 03:01:44 +0000 > > Commit: Piotr Kubaj <pkubaj@FreeBSD.org> > > CommitDate: 2022-12-11 12:05:39 +0000 > > > > uname: switch machine to HW_MACHINE_ARCH > > > > On powerpc64, powerpc64le and riscv64 some software wrongly assumes > that > > it runs on powerpc or riscv (32-bit). > > > > Differential revision: https://reviews.freebsd.org/D35962 > > Approved by: alfredo, imp > > --- > > lib/libc/gen/__xuname.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/lib/libc/gen/__xuname.c b/lib/libc/gen/__xuname.c > > index fba5eaeebc88..bd328d4f880f 100644 > > --- a/lib/libc/gen/__xuname.c > > +++ b/lib/libc/gen/__xuname.c > > @@ -127,11 +127,11 @@ __xuname(int namesize, void *namebuf) > > } > > q += namesize; > > > > - if ((p = getenv("UNAME_m"))) > > + if ((p = getenv("UNAME_p"))) > > strlcpy(q, p, namesize); > > else { > > mib[0] = CTL_HW; > > - mib[1] = HW_MACHINE; > > + mib[1] = HW_MACHINE_ARCH; > > len = namesize; > > oerrno = errno; > > if (sysctl(mib, 2, q, &len, NULL, 0) == -1) { > > > > This probably needs to go into RELNOTES at some point. Though small it's > significant enough to document. > It's also only for uname(3), not uname(1). Warner [-- Attachment #2 --] <div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Dec 11, 2022 at 7:37 AM Cy Schubert <<a href="mailto:Cy.Schubert@cschubert.com">Cy.Schubert@cschubert.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">In message <<a href="mailto:202212111208.2BBC835O095410@gitrepo.freebsd.org" target="_blank">202212111208.2BBC835O095410@gitrepo.freebsd.org</a>>, Piotr Kubaj <br> write<br> s:<br> > The branch main has been updated by pkubaj (ports committer):<br> ><br> > URL: <a href="https://cgit.FreeBSD.org/src/commit/?id=83bf6ab568293e325f437342cdb87a62" rel="noreferrer" target="_blank">https://cgit.FreeBSD.org/src/commit/?id=83bf6ab568293e325f437342cdb87a62</a><br>; > 6353e27c<br> ><br> > commit 83bf6ab568293e325f437342cdb87a626353e27c<br> > Author: Piotr Kubaj <pkubaj@FreeBSD.org><br> > AuthorDate: 2022-12-11 03:01:44 +0000<br> > Commit: Piotr Kubaj <pkubaj@FreeBSD.org><br> > CommitDate: 2022-12-11 12:05:39 +0000<br> ><br> > uname: switch machine to HW_MACHINE_ARCH<br> > <br> > On powerpc64, powerpc64le and riscv64 some software wrongly assumes that<br> > it runs on powerpc or riscv (32-bit).<br> > <br> > Differential revision: <a href="https://reviews.freebsd.org/D35962" rel="noreferrer" target="_blank">https://reviews.freebsd.org/D35962</a><br>; > Approved by: alfredo, imp<br> > ---<br> > lib/libc/gen/__xuname.c | 4 ++--<br> > 1 file changed, 2 insertions(+), 2 deletions(-)<br> ><br> > diff --git a/lib/libc/gen/__xuname.c b/lib/libc/gen/__xuname.c<br> > index fba5eaeebc88..bd328d4f880f 100644<br> > --- a/lib/libc/gen/__xuname.c<br> > +++ b/lib/libc/gen/__xuname.c<br> > @@ -127,11 +127,11 @@ __xuname(int namesize, void *namebuf)<br> > }<br> > q += namesize;<br> > <br> > - if ((p = getenv("UNAME_m")))<br> > + if ((p = getenv("UNAME_p")))<br> > strlcpy(q, p, namesize);<br> > else {<br> > mib[0] = CTL_HW;<br> > - mib[1] = HW_MACHINE;<br> > + mib[1] = HW_MACHINE_ARCH;<br> > len = namesize;<br> > oerrno = errno;<br> > if (sysctl(mib, 2, q, &len, NULL, 0) == -1) {<br> ><br> <br> This probably needs to go into RELNOTES at some point. Though small it's <br> significant enough to document.<br></blockquote><div><br></div><div>It's also only for uname(3), not uname(1).</div><div><br></div><div>Warner </div></div></div>help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrh_07uc_EHAPEihA=75H2akBxYNRLGb_4HX1kiGL6hTA>