Re: [PATCH 0/3] debloat aic7xxx and aic79xx drivers

Denys Vlasenko
2007-08-31T21:21:13Z

On Friday 31 August 2007 17:27, Valdis.Kletnieks@vt.edu wrote:
> On Fri, 31 Aug 2007 16:13:59 BST, Denys Vlasenko said:
> >
> > text data bss dec hex filename
> > 261433 50018 1172 312623 4c52f linux-2.6.23-rc1.org.t/drivers/scsi/aic7xxx/built-in.o
> > 199654 50018 1172 250844 3d3dc linux-2.6.23-rc1.aic.t/drivers/scsi/aic7xxx/built-in.o
> > 184014 21314 1172 206500 326a4 linux-2.6.23-rc1.aic1.t/drivers/scsi/aic7xxx/built-in.o
> > 202378 2850 1172 206400 32640 linux-2.6.23-rc1.aic2.t/drivers/scsi/aic7xxx/built-in.o
> >
> > 1-debloat.patch deinlines a lot of functions
> > 2-addstatic.patch adds statics, #ifdefs out huge amount of unused code, adds consts
> > 3-addconst.patch adds more consts
>
> Yowza. Looking at aic1->aic2, it looks like 20K became 'const', and only
> 3K *wasn't* 'const'?

Exactly. There are firmware images/patches and a lot of structures
with char* members pointing to text/messages.

Btw, aic94xx driver needs the same treatment.
--
vda