Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 2 Mar 2011 02:28:54 +0000
From:      Alexander Best <arundel@freebsd.org>
To:        Roman Divacky <rdivacky@freebsd.org>
Cc:        current@freebsd.org
Subject:   Re: [TESTING]: boot2 changes
Message-ID:  <20110302022854.GA42880@freebsd.org>
In-Reply-To: <20110228203928.GA93494@freebsd.org>
References:  <20110228203928.GA93494@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon Feb 28 11, Roman Divacky wrote:
> hi there,
> 
> I have a patch that shrinks boot2 some:
> 
> 1) it switches kname to be just a pointer instead of an array
>    thus avoiding a couple of memcpy()s
> 
> 2) it changes ioctl to unsigned from uint8_t
> 
> 3) it changes the first keyhit limit to 5 seconds from 3
>    so that constant propagation can take place
> 
> 4) it changes the ticks overflow computation as suggested by bde@
> 
> 5) it removes bi_basemem/bi_extmem/bi_memsizes_valid setting from
>    bootinfo as it is unused
> 
> 
> the patch:
> 
>         http://lev.vlakno.cz/~rdivacky/boot2-almost-there-diet.patch
> 
> It works for me and I'd like you to review/test this patch

did you also take a look at the code in sys/boot/i386/common? many functions in
sys/boot/i386/boot2/boot2.c are also defined there and some seem to contain
improved code.

e.g. was the return type of xputc() changed from int to void, i think that's
quite a resonable change and might reduce the boot2 binary size further.

cheers.
alex

> 
> thank you!
> 
> roman



-- 
a13x



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110302022854.GA42880>