Date: Sat, 30 Nov 2019 11:03:16 -0700 From: Warner Losh <imp@bsdimp.com> To: Enji Cooper <yaneurabeya@gmail.com> Cc: Scott Long <scottl@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r355164 - in head: . share/man/man4 sys/amd64/conf sys/conf sys/dev/trm sys/i386/conf sys/modules sys/modules/trm Message-ID: <CANCZdfrLMH_RNdd-hQrzGymZ_gjUc6YuqtfJON4muruutNyNxA@mail.gmail.com> In-Reply-To: <41D40355-A16E-48E1-9C11-0C9E42C83A42@gmail.com> References: <201911280232.xAS2WHtb029602@repo.freebsd.org> <41D40355-A16E-48E1-9C11-0C9E42C83A42@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Nov 30, 2019 at 10:47 AM Enji Cooper <yaneurabeya@gmail.com> wrote: > > > On Nov 27, 2019, at 6:32 PM, Scott Long <scottl@FreeBSD.org> wrote: > > > > Author: scottl > > Date: Thu Nov 28 02:32:17 2019 > > New Revision: 355164 > > URL: https://svnweb.freebsd.org/changeset/base/355164 > > > > Log: > > Remove the trm(4) driver > > > > Differential Revision: https://reviews.freebsd.org/D22575 > > Hi Scott, > I believe this driver was removed because it was impacts the CAM > GIANT lock removal effort =E2=80=94 is that true (I=E2=80=99m asking beca= use the =E2=80=9Cwhy=E2=80=9D > behind the removal is unclear)? > Hi Enji, We're trying hard to get rid of all Giant-locked drivers in the tree, either by updating or removal. Since sym(4) provides a super-set of trm(4) and we have recent-ish reports of sym(4) working, it makes sense to trim this driver from the tree. The specific cards it supports aren't all that popular, the couple-extra features that trm(4) gave over sym(4) aren't really that relevant today, and it's been years since trm has had good testing and maintenance. Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrLMH_RNdd-hQrzGymZ_gjUc6YuqtfJON4muruutNyNxA>