svn commit: r311849 - in head: . sys/amd64/conf sys/arm64/conf sys/conf sys/dev/e1000 sys/i386/conf sys/mips/conf sys/modules sys/modules/em sys/modules/igb sys/powerpc/conf

[Rodney W. Grimes]

-- Start of PGP signed section.
[ Charset windows-1252 unsupported, converting... ]
> 
> 
> On 02/15/17 03:06, Roger Pau Monn? wrote:
> > On Tue, Jan 10, 2017 at 03:23:22AM +0000, Sean Bruno wrote:
> >> Author: sbruno
> >> Date: Tue Jan 10 03:23:22 2017
> >> New Revision: 311849
> >> URL: https://svnweb.freebsd.org/changeset/base/311849
> >>
> >> Log:
> >>   Migrate e1000 to the IFLIB framework:
> >>   - em(4) igb(4) and lem(4)
> >>   - deprecate the igb device from kernel configurations
> >>   - create a symbolic link in /boot/kernel from if_em.ko to if_igb.ko

What about if you make that a hard link?
Would that fix the CP issue too?

> > 
> > This linking causes mfsBSD to choke when building an image from HEAD. It tries
> > to issue the following command:
> > 
> > ${_v}${CP} -rp ${_BOOTDIR}/kernel ${WRKDIR}/disk/boot
> > 
> > Which fails when finding the symbol link. I can send a patch to change that to
> > -Rp, which would work fine, but wouldn't it be better to either completely
> > remove if_igb.ko, or simply copy if_em.ko to if_igb.ko?
> > 
> > I'm wondering if for example anyone strips down it's /boot/kernel/ manually, by
> > removing unused modules, and what would happen if if_em.ko is removed but not
> > if_igb.ko.
> > 
> > Roger.
> > 
> > 
> 
> Well, this was my naive attempt to make upgrades for users to be
> non-eventful in the event they have "if_igb_load=YES" in their
> loader.conf instead of having it built into their kernel.
> 
> If the -Rp works, I'll add that instead.
> 
> sean
> 
-- End of PGP section, PGP failed!

-- 
Rod Grimes                                                 rgrimes at freebsd.org