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>