<p dir="ltr"><br>
On 7 Aug 2013 02:26, "Andy Lutomirski" <<a href="mailto:luto@amacapital.net">luto@amacapital.net</a>> wrote:<br>
><br>
> On Tue, Aug 6, 2013 at 5:24 PM, Tom Gundersen <<a href="mailto:teg@jklm.no">teg@jklm.no</a>> wrote:<br>
> ><br>
> > On 6 Aug 2013 18:32, "Bryan Kadzban" <<a href="mailto:bryan@kadzban.is-a-geek.net">bryan@kadzban.is-a-geek.net</a>> wrote:<br>
> >><br>
> >> On Tue, Aug 06, 2013 at 11:17:17AM +0200, Tom Gundersen wrote:<br>
> >> > On Tue, Aug 6, 2013 at 11:11 AM, Tom Gundersen <<a href="mailto:teg@jklm.no">teg@jklm.no</a>> wrote:<br>
> >> > > On Tue, Aug 6, 2013 at 10:20 AM, Maarten Lankhorst<br>
> >> > > <<a href="mailto:m.b.lankhorst@gmail.com">m.b.lankhorst@gmail.com</a>> wrote:<br>
> >> > >> Op 05-08-13 18:29, Andy Lutomirski schreef:<br>
> >> > >>> The systemd commit below can delay firmware loading by multiple<br>
> >> > >>> minutes if CONFIG_FW_LOADER_USER_HELPER=y.  Unfortunately no one<br>
> >> > >>> noticed that the systemd-udev change would break new kernels as well<br>
> >> > >>> as old kernels.<br>
> >> > >>><br>
> >> > >>> Since the kernel apparently can't count on reasonable userspace<br>
> >> > >>> support, turn this thing off by default.<br>
> >> > >>><br>
> >> > >>> commit a3bd8447be4ea2ce230eb8ae0e815c04d85fa15a<br>
> >> > >>> Author: Tom Gundersen <<a href="mailto:teg@jklm.no">teg@jklm.no</a>><br>
> >> > >>> Date:   Mon Mar 18 15:12:18 2013 +0100<br>
> >> > >>><br>
> >> > >>>     udev: make firmware loading optional and disable by default<br>
> >> > >>><br>
> >> > >>>     Distros that whish to support old kernels should set<br>
> >> > >>><br>
> >> > >>> --with-firmware-dirs="/usr/lib/firmware/updates:/usr/lib/firmware"<br>
> >> > >>>     to retain the old behaviour.<br>
> >> > >>><br>
> >> > >> methinks this patch should be reverted then,<br>
> >> > ><br>
> >> > > Well, all the code is still there, so it can be enabled if anyone<br>
> >> > > wants it.<br>
> >> > ><br>
> >> > >> or a stub should be added to udev to always fail firmware loading so<br>
> >> > >> timeouts don't occur.<br>
> >> > ><br>
> >> > > I think the only use (if any) of a userspace firmware loader would be<br>
> >> > > for anyone who wants a custom one (i.e., not udev), so we shouldn't<br>
> >> > > just fail the loading from udev unconditionally.<br>
> >> > ><br>
> >> > > How about we just improve the udev documentation a bit, similar to<br>
> >> > > Andy's kernel patch?<br>
> >> ><br>
> >> > Sorry, I should first have checked. We already document this in the<br>
> >> > README:<br>
> >> ><br>
> >> > >        Userspace firmware loading is deprecated, will go away, and<br>
> >> > >        sometimes causes problems:<br>
> >> > >          CONFIG_FW_LOADER_USER_HELPER=n<br>
> >><br>
> >> ...And this patch is making the kernel default to the correct behavior,<br>
> >> instead of the now-broken-by-udev behavior.<br>
> >><br>
> >> I'm not sure I see the issue with it?  :-)<br>
> ><br>
> > Oh yeah this patch is totally the right thing to do, I was just arguing that<br>
> > there is nothing to be done on the udev side.<br>
> ><br>
> >> (Add me to the list of people that think udev is broken too, fwiw.  But<br>
> >> let's at least not leave *both* sides in a broken-by-default state.)<br>
> ><br>
> > Well I don't think it is too much to ask that the kernel and udev should be<br>
> > configured in a consistent way. Especially as thing still work even if you<br>
> > get it wrong, albeit with a delay.<br>
><br>
> Except that the current defaults are inconsistent</p>
<p dir="ltr">Which your patch fixes. I don't see the problem here...</p>
<p dir="ltr">> and there is no<br>
> explanation anywhere in the logs when this is screwed up.<br>
</p>