Re: CVS commit: src/sys/arch/sparc/stand/bootxx

[Havard Eidnes <he%NetBSD.org@localhost>]

> > Module Name:        src
> > Committed By:       he
> > Date:               Thu Mar 19 14:05:24 UTC 2009
> >
> > Modified Files:
> >     src/sys/arch/sparc/stand/bootxx: bootxx.c
> >
> > Log Message:
> > Now that the stand/ code was converted to use memcpy(), the private
> > implementation of bcopy() needs to become memcpy() as well.  The
> > simpler local implementation is needed so that the boot code doesn't
> > overflow the available space.
>
> Hmm, that's because sparc/stand/Makefile.buildboot doesn't have
> -Os (which enables __OPTIMIZE_SIZE__) but -O2.
>
> Maybe it's worth to try -Os and remove the local one.

Replacing -O2 with -Os (and at the same time #if'ing out the
local memcpy()) definately makes an impact:


-O2-compiled boot code:

   text    data     bss     dec     hex filename
  72936     552    3424   76912   12c70 boot.388000.tmp
   text    data     bss     dec     hex filename
  72936     552    3424   76912   12c70 bootjs.net
   text    data     bss     dec     hex filename
   6652     572     576    7800    1e78 bootxx.sym

Image `miniroot.fs.tmp' complete
File system:         miniroot.fs
File system type:    ffs (blocksize 4096, needswap 1)
Primary bootstrap:   /u/build/HEAD/dest/sparc/usr/mdec/bootxx
Secondary bootstrap: /boot
Bootstrap start sector: 1
Bootstrap byte count:   7228
Bootstrap block table:  118 entries of 4096 bytes available, 18 used: 8896 8904 
8912 8920 8928 8936 8944 8952 8960 8968 8976 8984 9000 9008 9016 9024 9032 9040
Writing bootstrap
dependall ===> sparc


-Os-compiled boot code:

   text    data     bss     dec     hex filename
  66264     552    3424   70240   11260 boot.388000.tmp
   text    data     bss     dec     hex filename
  66264     552    3424   70240   11260 bootjs.net
   text    data     bss     dec     hex filename
   6280     572     576    7428    1d04 bootxx.sym


Image `miniroot.fs.tmp' complete
File system:         miniroot.fs
File system type:    ffs (blocksize 4096, needswap 1)
Primary bootstrap:   /u/build/HEAD/dest/sparc/usr/mdec/bootxx
Secondary bootstrap: /boot
Bootstrap start sector: 1
Bootstrap byte count:   6852
Bootstrap block table:  118 entries of 4096 bytes available, 17 used: 8896 8904 
8912 8920 8928 8936 8944 8952 8960 8968 8976 8984 9000 9008 9016 9024 9032
Writing bootstrap
dependall ===> sparc


Regards,

- Håvard