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>