Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 10 Mar 2017 10:12:22 -0800 (PST)
From:      "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net>
To:        Sean Bruno <sbruno@freebsd.org>
Cc:        =?UTF-8?Q?Roger_Pau_Monn=C3=A9?= <roger.pau@citrix.com>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: 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
Message-ID:  <201703101812.v2AICMt5013417@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <f6022c26-ea27-1964-426a-dd61b66abe13@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
-- 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@freebsd.org



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201703101812.v2AICMt5013417>