svn commit: r346441 - in head/sys/modules: em fusefs iavf
Rodney W. Grimes
freebsd at gndrsh.dnsmgr.net
Tue Sep 3 14:08:07 UTC 2019
> On Sat, Apr 20, 2019, 7:47 AM Alan Somers <asomers at freebsd.org> wrote:
>
> > On Sat, Apr 20, 2019 at 7:23 AM Justin Hibbits <chmeeedalf at gmail.com>
> > wrote:
> > >
> > >
> > >
> > > On Sat, Apr 20, 2019, 08:21 Alan Somers <asomers at freebsd.org> wrote:
> > >>
> > >> On Sat, Apr 20, 2019 at 6:58 AM Justin Hibbits <chmeeedalf at gmail.com>
> > wrote:
> > >> >
> > >> >
> > >> >
> > >> > On Sat, Apr 20, 2019, 07:51 Alan Somers <asomers at freebsd.org> wrote:
> > >> >>
> > >> >> Author: asomers
> > >> >> Date: Sat Apr 20 12:51:05 2019
> > >> >> New Revision: 346441
> > >> >> URL: https://svnweb.freebsd.org/changeset/base/346441
> > >> >>
> > >> >> Log:
> > >> >> Use symlinks for kernel modules rather than hardlinks
> > >> >>
> > >> >> When aliasing a kernel module to a different name (ie if_igb for
> > if_em),
> > >> >> it's better to use symlinks than hard links. kldxref will omit
> > entries for
> > >> >> the links, ensuring that the loaded module has the correct name.
> > >> >>
> > >> >>
> > >> >
> > >> >
> > >> > Thanks! This should fix installkernel on my POWER9.
> > >> >
> > >> > - Justin
> > >>
> > >> What's the problem with your POWER9? Is that one of those msdosfs
> > >> /boot systems? If so, I don't think this will fix it. msdosfs
> > >> doesn't support either symlinks or hardlinks. Or is there some other
> > >> problem?
> > >> -Alan
> > >
> > >
> > > Yes it is. Well that's a bummer then. I thought we faked symlinks on
> > msdosfs, but on second thought not sure how well would do that.
> > >
> > > - Justin
> >
> > We should probably just remove the offending links on ppc, then. The
> > only harm is that after upgrading, ppc users would have to replace
> > fuse_load="YES" with fusefs_load="YES".
> >
>
> We should only do the Intel links on those platforms that have legacy users
> that need the old names. We should also only support the old names for one
> release or so.
IIRC this was done so that the dropping of one of the drivers in ^head
could be merged back to ^stable. It probably should of been killed in
^head after that fact, but if I recall the values correctly it was
@head=12 and merged to @stable=11. We would need to dig those facts
out, but yes, please, this link should be killed and cleaned up quickly as
it creates almost as many problems as it solved.
> Warner
--
Rod Grimes rgrimes at freebsd.org
More information about the svn-src-all
mailing list