Date: Tue, 10 Dec 2024 15:50:02 -0700 From: Warner Losh <imp@bsdimp.com> To: Ed Maste <emaste@freebsd.org> Cc: "Bjoern A. Zeeb" <bz@freebsd.org>, Jessica Clarke <jrtc27@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>" <dev-commits-src-main@freebsd.org> Subject: Re: git: 87e140a5c6f8 - main - iwlwifi: avoid (hard) hang on loading module Message-ID: <CANCZdfpC-HX2XFonp%2BbYKJP5why7kYy6r%2Bb1HDRa7ja6bBdktQ@mail.gmail.com> In-Reply-To: <CAPyFy2CHFH5SF3=YNZLA2iUOA2z9Kugn5Xx4xagDrt4jnz2a5g@mail.gmail.com> References: <202412091447.4B9ElFBH050286@gitrepo.freebsd.org> <6F9C2C43-A3DD-4CF5-AC27-899FE139E9A6@freebsd.org> <0883q825-802q-2n9q-3s4o-3nq13qpqno71@SerrOFQ.bet> <CAPyFy2CHFH5SF3=YNZLA2iUOA2z9Kugn5Xx4xagDrt4jnz2a5g@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000e6cca30628f24d1b Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, Dec 10, 2024, 3:09=E2=80=AFPM Ed Maste <emaste@freebsd.org> wrote: > On Mon, 9 Dec 2024 at 20:55, Bjoern A. Zeeb <bz@freebsd.org> wrote: > > > > On Mon, 9 Dec 2024, Jessica Clarke wrote: > > > > > On 9 Dec 2024, at 14:47, Bjoern A. Zeeb <bz@FreeBSD.org> wrote: > > >> > > >> The branch main has been updated by bz: > > >> > > >> URL: > https://cgit.FreeBSD.org/src/commit/?id=3D87e140a5c6f89eea7ea6320d1ae3456= 6492abfc0 > > >> + if (bootverbose) > > >> + printf("%s: module '%s' not yet available; will be" > > >> + "initialized in a moment\n", __func__, op->name); > > > > > > Missing a space in the string. > > > > Thanks; I'll add that. > > Or just allow the line to go beyond 80 columns, allowing someone to > grep for "will be initialized in a moment" to find the source of this > string. > It's either in style.9 as a length exception or is in a review somewhere... it make it easier to grep the error messages... Warner > --000000000000e6cca30628f24d1b 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 Tue, Dec 10, 2024, 3:09=E2=80=AFPM Ed Maste <<a = href=3D"mailto:emaste@freebsd.org">emaste@freebsd.org</a>> wrote:<br></d= iv><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left= :1px #ccc solid;padding-left:1ex">On Mon, 9 Dec 2024 at 20:55, Bjoern A. Ze= eb <<a href=3D"mailto:bz@freebsd.org" target=3D"_blank" rel=3D"noreferre= r">bz@freebsd.org</a>> wrote:<br> ><br> > On Mon, 9 Dec 2024, Jessica Clarke wrote:<br> ><br> > > On 9 Dec 2024, at 14:47, Bjoern A. Zeeb <bz@FreeBSD.org> wr= ote:<br> > >><br> > >> The branch main has been updated by bz:<br> > >><br> > >> URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D87e= 140a5c6f89eea7ea6320d1ae34566492abfc0" rel=3D"noreferrer noreferrer" target= =3D"_blank">https://cgit.FreeBSD.org/src/commit/?id=3D87e140a5c6f89eea7ea63= 20d1ae34566492abfc0</a><br> > >> + if (bootverbose)<br> > >> +=C2=A0 =C2=A0 =C2=A0 =C2=A0printf("%s: module '%s&#= 39; not yet available; will be"<br> > >> +=C2=A0 =C2=A0"initialized in a moment\n", __func__= , op->name);<br> > ><br> > > Missing a space in the string.<br> ><br> > Thanks; I'll add that.<br> <br> Or just allow the line to go beyond 80 columns, allowing someone to<br> grep for "will be initialized in a moment" to find the source of = this<br> string.<br></blockquote></div></div><div dir=3D"auto"><br></div><div dir=3D= "auto">It's either in style.9 as a length exception or is in a review s= omewhere...=C2=A0 it make it easier to grep the error messages...</div><div= dir=3D"auto"><br></div><div dir=3D"auto">Warner=C2=A0</div><div dir=3D"aut= o"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=3D"ma= rgin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> </blockquote></div></div></div> --000000000000e6cca30628f24d1b--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfpC-HX2XFonp%2BbYKJP5why7kYy6r%2Bb1HDRa7ja6bBdktQ>