Date: Fri, 10 Mar 2017 14:12:41 -0700 From: Warner Losh <imp@bsdimp.com> To: John Baldwin <jhb@freebsd.org> Cc: Navdeep Parhar <nparhar@gmail.com>, Ryan Stone <rysto32@gmail.com>, Sean Bruno <sbruno@freebsd.org>, =?UTF-8?Q?Roger_Pau_Monn=C3=A9?= <roger.pau@citrix.com>, src-committers <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@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: <CANCZdfq-H5U%2B%2By6PcRBxH1tTagRmp6CB%2BHMber_FUf0Mu=mang@mail.gmail.com> In-Reply-To: <2098064.nampQkWPZt@ralph.baldwin.cx> References: <201701100323.v0A3NMAI045868@repo.freebsd.org> <CAFMmRNyLuKZbMK3fVm=1G7Ky2KQsFOP4BNqkEhyRodxFZbBPpA@mail.gmail.com> <CAPFoGT9qDs%2Bu5erPNc-v825X7y%2B5icG0SNZj0OSrhoLy8ReJ9Q@mail.gmail.com> <2098064.nampQkWPZt@ralph.baldwin.cx>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Mar 10, 2017 at 1:21 PM, John Baldwin <jhb@freebsd.org> wrote: > On Friday, March 10, 2017 11:12:45 AM Navdeep Parhar wrote: >> On Fri, Mar 10, 2017 at 10:41 AM, Ryan Stone <rysto32@gmail.com> wrote: >> > There's also the issue that running "ifconfig igb0 blah" during startup will >> > cause if_igb to be automatically loaded by ifconfig. >> > >> > I guess we could add a dummy if_igb.ko that just has a dependency on >> > if_em.ko >> >> I do similar stuff in cxgbe (if_cxl.ko and if_cc.ko exist solely to >> drag in if_cxgbe.ko as a dependency). In hindsight I wish I'd just >> added a list of ifnet names -> kld map to ifconfig instead. It would >> have been an ugly hack but much simpler. > > For now I think having an if_igb.c that is like if_cc.c and if_cxl.c > is probably the simplest way to go. It's certainly the easiest way forward. + if_igb.ko module, right? Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfq-H5U%2B%2By6PcRBxH1tTagRmp6CB%2BHMber_FUf0Mu=mang>