Date: Fri, 3 Aug 2018 02:32:23 -0600 From: Warner Losh <imp@bsdimp.com> To: Toomas Soome <tsoome@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r337231 - head/stand/efi/loader Message-ID: <CANCZdfrM%2B4vvpAi1bH5t_qWWVue2YnkQ09r8bqTvg%2BdHxmPUjw@mail.gmail.com> In-Reply-To: <201808030759.w737xTwO075764@repo.freebsd.org> References: <201808030759.w737xTwO075764@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Any reason why efipart_inithandles() should even return an error in this case? Seems to cause us nothing but trouble so we have to filter some, but not all errors which strikes me as a bad design. We should only return errors for real errors, like not having enough memory for the arrays we need. Warner On Fri, Aug 3, 2018 at 1:59 AM, Toomas Soome <tsoome@freebsd.org> wrote: > Author: tsoome > Date: Fri Aug 3 07:59:29 2018 > New Revision: 337231 > URL: https://svnweb.freebsd.org/changeset/base/337231 > > Log: > loader.efi: clean up misleading noise from missing block devices > > If there are no block devices, there is no need to printout > error (ENOENT). > > In case of netboot, our image path has no block device, no need to make > noise about it. > > Modified: > head/stand/efi/loader/main.c > > Modified: head/stand/efi/loader/main.c > ============================================================ > ================== > --- head/stand/efi/loader/main.c Fri Aug 3 02:51:37 2018 > (r337230) > +++ head/stand/efi/loader/main.c Fri Aug 3 07:59:29 2018 > (r337231) > @@ -545,8 +545,6 @@ find_currdev(EFI_LOADED_IMAGE *img, bool do_bootmgr, b > return (0); > } > } > - } else { > - printf("Can't find device by handle\n"); > } > > /* > @@ -862,9 +860,9 @@ main(int argc, CHAR16 *argv[]) > * march through the device switch probing for things. > */ > i = efipart_inithandles(); > - if (i != 0) { > + if (i != 0 && i != ENOENT) { > printf("efipart_inithandles failed with ERRNO %d, expect " > - "failures", i); > + "failures\n", i); > } > > for (i = 0; devsw[i] != NULL; i++) > >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrM%2B4vvpAi1bH5t_qWWVue2YnkQ09r8bqTvg%2BdHxmPUjw>