From owner-cvs-src@FreeBSD.ORG Wed Jun 1 14:48:08 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.ORG Delivered-To: cvs-src@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2568116A41C; Wed, 1 Jun 2005 14:48:08 +0000 (GMT) (envelope-from sobomax@portaone.com) Received: from www.portaone.com (web.portaone.com [195.70.151.35]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9233E43D49; Wed, 1 Jun 2005 14:48:07 +0000 (GMT) (envelope-from sobomax@portaone.com) Received: from [192.168.0.95] (2-zyx-piter-f0285.r5.ll.kw.ua [195.140.246.50]) (authenticated bits=0) by www.portaone.com (8.12.11/8.12.11) with ESMTP id j51Em4uH075685 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 1 Jun 2005 16:48:05 +0200 (CEST) (envelope-from sobomax@portaone.com) Message-ID: <429DCAA3.8020303@portaone.com> Date: Wed, 01 Jun 2005 17:48:03 +0300 From: Maxim Sobolev Organization: Porta Software Ltd User-Agent: Mozilla Thunderbird 1.0.2 (Windows/20050317) X-Accept-Language: en-us, en MIME-Version: 1.0 To: obrien@FreeBSD.ORG References: <200505312116.j4VLGpYw072934@repoman.freebsd.org> <429CD6E8.1070009@portaone.com> <20050601142848.GA7964@dragon.NUXI.org> In-Reply-To: <20050601142848.GA7964@dragon.NUXI.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Virus-Scanned: ClamAV 0.83/905/Wed Jun 1 09:28:38 2005 on www.portaone.com X-Virus-Status: Clean Cc: cvs-src@FreeBSD.ORG, src-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/boot/i386/loader Makefile X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Maxim.Sobolev@portaone.com List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 01 Jun 2005 14:48:08 -0000 David O'Brien wrote: > On Wed, Jun 01, 2005 at 12:28:08AM +0300, Maxim Sobolev wrote: > >>David E. O'Brien wrote: >> >>>obrien 2005-05-31 21:16:51 UTC >>> >>> FreeBSD src repository >>> >>> Modified files: >>> sys/boot/i386/loader Makefile >>> Log: >>> Put bzip2 support on equal footing with gzip support. >>> Enable bzip2 support by default, set LOADER_NO_BZIP2_SUPPORT to disable >>> it. >> >>This only increases the size of loader without any good reason. Please >>stop making nonsense commits. > > > And having gzip support on by default only increases the loader without > any good reason. We provide a feature rich environment. The loader > isn't at the brink of its size limit like boot0 is. I put bzip2 on par No, it is, since currently it's still limited to 640KB of memory both for data and for text on i386 and possibly on amd64. For that reason you can't decompress bzip2-compressed filesystems compressed with block size higher than 100k, which is the smalles block size possible with bzip2. Still it provides noticeably better compression than libz. Decompression requires about 100k + ( 2.5 x block size ), that is 350k. Given that loader's text is currently 200-250k and the fact that it needs some heap for other functionality even 20-30k can really matter. Actually I am not sure if you can do bzip2 decompression w/o turning forth and gzip support off on i386 now. -Maxim > with gzip support. If this support bloats the loader too much, we should > disable the gzip support then, not the bzip2 support - as you know bzip2 > compresses better. >