Date: Thu, 30 Nov 2017 00:03:38 -0700 From: Scott Long <scottl@samsco.org> To: Bruce Evans <brde@optusnet.com.au> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r326377 - in head/sys: amd64/conf arm/conf i386/conf mips/conf modules/aic7xxx/ahc modules/aic7xxx/ahd powerpc/conf sparc64/conf Message-ID: <B0B12180-C71B-4343-8B7D-FEA7673FEAB4@samsco.org> In-Reply-To: <20171130115317.X1349@besplex.bde.org> References: <201711292341.vATNfoSS054606@repo.freebsd.org> <20171130115317.X1349@besplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Nov 29, 2017, at 6:19 PM, Bruce Evans <brde@optusnet.com.au> wrote: >=20 > On Wed, 29 Nov 2017, Scott Long wrote: >=20 >> Log: >> It's time to retire AHC_REG_PRETTY_PRINT and AHD_REG_PRETTY_PRINT = from >> the standard kernels. They are still available as custom compile >> options. >=20 > MI NOTES still gives MI sizes for these options. It seems unlikely > that these sizes are still correct even for 1 arch where they were = measured > on. >=20 Today, I measured a savings of ~130KiB when both options were removed. = I also did the original size measurements back in 2000-whatever. I=E2=80=99= m guessing that the difference is due to improvements in compiler technology. >> Modified: head/sys/amd64/conf/GENERIC >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/amd64/conf/GENERIC Wed Nov 29 23:28:40 2017 = (r326376) >> +++ head/sys/amd64/conf/GENERIC Wed Nov 29 23:41:49 2017 = (r326377) >> @@ -123,11 +123,7 @@ device siis # = SiliconImage SiI3124/SiI3132/SiI3531 >>=20 >> # SCSI Controllers >> device ahc # AHA2940 and onboard = AIC7xxx devices >> -options AHC_REG_PRETTY_PRINT # Print register bitfields in = debug >> - # output. Adds ~128k to driver. >> device ahd # AHA39320/29320 and = onboard AIC79xx devices >> -options AHD_REG_PRETTY_PRINT # Print register bitfields in = debug >> - # output. Adds ~215k to driver. >=20 > k is a bogus unit for memory sizes. NOTES has the same bug. >=20 >> Modified: head/sys/i386/conf/GENERIC >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/i386/conf/GENERIC Wed Nov 29 23:28:40 2017 = (r326376) >> +++ head/sys/i386/conf/GENERIC Wed Nov 29 23:41:49 2017 = (r326377) >> @@ -120,11 +120,6 @@ device siis # = SiliconImage SiI3124/SiI3132/SiI3531 >>=20 >> # SCSI Controllers >> device ahc # AHA2940 and onboard = AIC7xxx devices >> -options AHC_REG_PRETTY_PRINT # Print register bitfields in = debug >> - # output. Adds ~128k to driver. >> -device ahd # AHA39320/29320 and = onboard AIC79xx devices >> -options AHD_REG_PRETTY_PRINT # Print register bitfields in = debug >> - # output. Adds ~215k to driver. >> device esp # AMD Am53C974 (Tekram = DC-390(T)) >> device hptiop # Highpoint RocketRaid = 3xxx series >> device isp # Qlogic family >>=20 >=20 > This also removes ahd. This bug is only new on i386/GENERIC. Oops! > Other arches > except amd64 never had ahd. arm/conf/CRB was most broken -- it had 2 = PRETTY > options but neither ahc or ahd. Perhaps these old drivers don't = belong in > any GENERIC. There=E2=80=99s a good argument to be made for doing a house-cleaning of = GENERIC between now and when FreeBSD 12.0 is released. Scott
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?B0B12180-C71B-4343-8B7D-FEA7673FEAB4>