Date: Fri, 31 May 2024 11:48:36 -0400 From: Warner Losh <imp@bsdimp.com> To: Cy Schubert <Cy.Schubert@cschubert.com> Cc: Warner Losh <imp@freebsd.org>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 676041c41ba5 - main - WPA: Allow CLOCK_BOOTTIME and CLOCK_MONOTONIC to #define the same Message-ID: <CANCZdfpyhd=BAcF_YFCCTumYNpGUMs0bQandZnoTYjFWcM7hqg@mail.gmail.com> In-Reply-To: <20240531152819.33B6597@slippy.cwsent.com> References: <202405311447.44VEl06w078151@gitrepo.freebsd.org> <20240531152819.33B6597@slippy.cwsent.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--00000000000056c22d0619c1eb4c Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Fri, May 31, 2024 at 11:28=E2=80=AFAM Cy Schubert <Cy.Schubert@cschubert= .com> wrote: > In message <202405311447.44VEl06w078151@gitrepo.freebsd.org>, Warner Losh > write > s: > > The branch main has been updated by imp: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=3D676041c41ba587285bb934aa2fca290= e > > a7208038 > > > > commit 676041c41ba587285bb934aa2fca290ea7208038 > > Author: Warner Losh <imp@FreeBSD.org> > > AuthorDate: 2024-05-31 14:44:55 +0000 > > Commit: Warner Losh <imp@FreeBSD.org> > > CommitDate: 2024-05-31 14:44:55 +0000 > > > > WPA: Allow CLOCK_BOOTTIME and CLOCK_MONOTONIC to #define the same > > > > Historically, these have been differnet values, and only one was > defined > > or they were defined as different values. Now that they are about t= o > be > > the same value, add #ifdef to cope. > > > > Sponsored by: Netflix > > Reviewed by: olce, val_packett.cool, adrian > > Differential Revision: https://reviews.freebsd.org/D45418 > > --- > > contrib/wpa/src/utils/os_unix.c | 11 ++++++++--- > > 1 file changed, 8 insertions(+), 3 deletions(-) > > > > diff --git a/contrib/wpa/src/utils/os_unix.c > b/contrib/wpa/src/utils/os_unix. > > c > > index 315c973f3228..a856179fb3b1 100644 > > --- a/contrib/wpa/src/utils/os_unix.c > > +++ b/contrib/wpa/src/utils/os_unix.c > > @@ -72,7 +72,6 @@ int os_get_time(struct os_time *t) > > return res; > > } > > > > - > > int os_get_reltime(struct os_reltime *t) > > { > > #ifndef __MACH__ > > @@ -97,15 +96,21 @@ int os_get_reltime(struct os_reltime *t) > > return 0; > > } > > switch (clock_id) { > > -#ifdef CLOCK_BOOTTIME > > +#if defined(CLOCK_BOOTTIME) > > Why this change? This could become a potential merge conflict next time > WPA > is MFVed. > Oh, that was a mistake. > > case CLOCK_BOOTTIME: > > clock_id =3D CLOCK_MONOTONIC; > > break; > > #endif > > -#ifdef CLOCK_MONOTONIC > > +#if defined(CLOCK_MONOTONIC) > > Ditto. > Ditto. I iterated on these changes and thought I'd restored them to the original. I'll do a followup diff minimization diff. > > > +/* > > + * FreeBSD has both BOOTTIME and MONOTONIC defined to the same value, > since > > they > > + * mean the same thing. FreeBSD 14.1 and ealier don't, so need this > case. > > + */ > > +#if !(defined(CLOCK_BOOTTIME) && CLOCK_BOOTTIME =3D=3D CLOCK_MONOTONIC= ) > > This I probably want to apply to the ports. > Yes. I had you on the review, but I did move kinda quickly on this, and so those ports are likely broken by the change in FreeBSD. My apologies. Warner > > case CLOCK_MONOTONIC: > > clock_id =3D CLOCK_REALTIME; > > break; > > +#endif > > #endif > > case CLOCK_REALTIME: > > return -1; > > > > > -- > Cheers, > Cy Schubert <Cy.Schubert@cschubert.com> > FreeBSD UNIX: <cy@FreeBSD.org> Web: https://FreeBSD.org > NTP: <cy@nwtime.org> Web: https://nwtime.org > > e^(i*pi)+1=3D0 > > > --00000000000056c22d0619c1eb4c Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote">= <div dir=3D"ltr" class=3D"gmail_attr">On Fri, May 31, 2024 at 11:28=E2=80= =AFAM Cy Schubert <<a href=3D"mailto:Cy.Schubert@cschubert.com">Cy.Schub= ert@cschubert.com</a>> wrote:<br></div><blockquote class=3D"gmail_quote"= style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);p= adding-left:1ex">In message <<a href=3D"mailto:202405311447.44VEl06w0781= 51@gitrepo.freebsd.org" target=3D"_blank">202405311447.44VEl06w078151@gitre= po.freebsd.org</a>>, Warner Losh <br> write<br> s:<br> > The branch main has been updated by imp:<br> ><br> > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D676041c41ba5= 87285bb934aa2fca290e" rel=3D"noreferrer" target=3D"_blank">https://cgit.Fre= eBSD.org/src/commit/?id=3D676041c41ba587285bb934aa2fca290e</a><br> > a7208038<br> ><br> > commit 676041c41ba587285bb934aa2fca290ea7208038<br> > Author:=C2=A0 =C2=A0 =C2=A0Warner Losh <imp@FreeBSD.org><br> > AuthorDate: 2024-05-31 14:44:55 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Warner Losh <imp@FreeBSD.org><br> > CommitDate: 2024-05-31 14:44:55 +0000<br> ><br> >=C2=A0 =C2=A0 =C2=A0WPA: Allow CLOCK_BOOTTIME and CLOCK_MONOTONIC to #d= efine the same<br> >=C2=A0 =C2=A0 =C2=A0<br> >=C2=A0 =C2=A0 =C2=A0Historically, these have been differnet values, and= only one was defined<br> >=C2=A0 =C2=A0 =C2=A0or they were defined as different values. Now that = they are about to be<br> >=C2=A0 =C2=A0 =C2=A0the same value, add #ifdef to cope.<br> >=C2=A0 =C2=A0 =C2=A0<br> >=C2=A0 =C2=A0 =C2=A0Sponsored by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0Netflix<br> >=C2=A0 =C2=A0 =C2=A0Reviewed by:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 olce, val_packett.cool, adrian<br> >=C2=A0 =C2=A0 =C2=A0Differential Revision:=C2=A0 <a href=3D"https://rev= iews.freebsd.org/D45418" rel=3D"noreferrer" target=3D"_blank">https://revie= ws.freebsd.org/D45418</a><br> > ---<br> >=C2=A0 contrib/wpa/src/utils/os_unix.c | 11 ++++++++---<br> >=C2=A0 1 file changed, 8 insertions(+), 3 deletions(-)<br> ><br> > diff --git a/contrib/wpa/src/utils/os_unix.c b/contrib/wpa/src/utils/o= s_unix.<br> > c<br> > index 315c973f3228..a856179fb3b1 100644<br> > --- a/contrib/wpa/src/utils/os_unix.c<br> > +++ b/contrib/wpa/src/utils/os_unix.c<br> > @@ -72,7 +72,6 @@ int os_get_time(struct os_time *t)<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0return res;<br> >=C2=A0 }<br> >=C2=A0 <br> > -<br> >=C2=A0 int os_get_reltime(struct os_reltime *t)<br> >=C2=A0 {<br> >=C2=A0 #ifndef __MACH__<br> > @@ -97,15 +96,21 @@ int os_get_reltime(struct os_reltime *t)<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0return 0;<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0}<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0switch (clock_id= ) {<br> > -#ifdef CLOCK_BOOTTIME<br> > +#if defined(CLOCK_BOOTTIME)<br> <br> Why this change? This could become a potential merge conflict next time WPA= <br> is MFVed.<br></blockquote><div><br></div><div>Oh, that was a mistake.=C2=A0= </div><div>=C2=A0</div><blockquote class=3D"gmail_quote" style=3D"margin:0p= x 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0case CLOCK_BOOTT= IME:<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0clock_id =3D CLOCK_MONOTONIC;<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0break;<br> >=C2=A0 #endif<br> > -#ifdef CLOCK_MONOTONIC<br> > +#if defined(CLOCK_MONOTONIC)<br> <br> Ditto.<br></blockquote><div><br></div><div>Ditto. I iterated on these chang= es and thought I'd restored them to the original. I'll do a followu= p diff minimization diff.=C2=A0</div><blockquote class=3D"gmail_quote" styl= e=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);paddin= g-left:1ex"> <br> > +/*<br> > + * FreeBSD has both BOOTTIME and MONOTONIC defined to the same value,= since <br> > they<br> > + * mean the same thing. FreeBSD 14.1 and ealier don't, so need th= is case.<br> > + */<br> > +#if !(defined(CLOCK_BOOTTIME) && CLOCK_BOOTTIME =3D=3D CLOCK_= MONOTONIC)<br> <br> This I probably want to apply to the ports.<br></blockquote><div><br></div>= <div>Yes. I had you on the review, but I did move kinda quickly on this, an= d so those ports</div><div>are likely broken by the change in FreeBSD. My a= pologies.</div><div><br></div><div>Warner</div><div>=C2=A0</div><blockquote= class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px so= lid rgb(204,204,204);padding-left:1ex"> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0case CLOCK_MONOT= ONIC:<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0clock_id =3D CLOCK_REALTIME;<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0break;<br> > +#endif<br> >=C2=A0 #endif<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0case CLOCK_REALT= IME:<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0return -1;<br> ><br> <br> <br> -- <br> Cheers,<br> Cy Schubert <<a href=3D"mailto:Cy.Schubert@cschubert.com" target=3D"_bla= nk">Cy.Schubert@cschubert.com</a>><br> FreeBSD UNIX:=C2=A0 <cy@FreeBSD.org>=C2=A0 =C2=A0Web:=C2=A0 <a href= =3D"https://FreeBSD.org" rel=3D"noreferrer" target=3D"_blank">https://FreeB= SD.org</a><br> NTP:=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0<<a href=3D"mailto:cy@nwtim= e.org" target=3D"_blank">cy@nwtime.org</a>>=C2=A0 =C2=A0 Web:=C2=A0 <a h= ref=3D"https://nwtime.org" rel=3D"noreferrer" target=3D"_blank">https://nwt= ime.org</a><br> <br> =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 e^(i*pi)+1=3D0<br> <br> <br> </blockquote></div></div> --00000000000056c22d0619c1eb4c--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfpyhd=BAcF_YFCCTumYNpGUMs0bQandZnoTYjFWcM7hqg>