From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from 93-97-173-237.zone5.bethere.co.uk ([93.97.173.237] helo=tim.rpsys.net) by linuxtogo.org with esmtp (Exim 4.72) (envelope-from ) id 1SDjqX-0007FE-9r for openembedded-core@lists.openembedded.org; Fri, 30 Mar 2012 23:59:01 +0200 Received: from localhost (localhost [127.0.0.1]) by tim.rpsys.net (8.13.6/8.13.8) with ESMTP id q2ULnvwE018462 for ; Fri, 30 Mar 2012 22:49:57 +0100 Received: from tim.rpsys.net ([127.0.0.1]) by localhost (tim.rpsys.net [127.0.0.1]) (amavisd-new, port 10024) with LMTP id 18287-02 for ; Fri, 30 Mar 2012 22:49:53 +0100 (BST) Received: from [192.168.3.10] ([192.168.3.10]) (authenticated bits=0) by tim.rpsys.net (8.13.6/8.13.8) with ESMTP id q2ULnlEW018456 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO) for ; Fri, 30 Mar 2012 22:49:48 +0100 Message-ID: <1333144186.18082.134.camel@ted> From: Richard Purdie To: Patches and discussions about the oe-core layer Date: Fri, 30 Mar 2012 22:49:46 +0100 In-Reply-To: <3ABC0140-1A7C-45A2-A601-3D8BDCB839D6@dominion.thruhere.net> References: <1333058859.18082.44.camel@ted> <3ABC0140-1A7C-45A2-A601-3D8BDCB839D6@dominion.thruhere.net> X-Mailer: Evolution 3.2.2- Mime-Version: 1.0 X-Virus-Scanned: amavisd-new at rpsys.net Subject: Re: [PATCH 0/1] genext2fs: support large files and filesystems without using large amounts of memor X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list Reply-To: Patches and discussions about the oe-core layer List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 30 Mar 2012 21:59:01 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Fri, 2012-03-30 at 13:43 -0700, Koen Kooi wrote: > Op 29 mrt. 2012, om 15:07 heeft Richard Purdie het volgende geschreven: > > > On Thu, 2012-03-29 at 00:51 +0800, Dexuan Cui wrote: > >> Hi RP, Saul, Paul, Darren, Mark, Josh and joaohf and all, please comment. > >> > >> Let's figure out if this big patch is accepatable or not... > >> > >> With this patch, I can successfully create a 8.5GB .ext3 file with genext2fs. > >> The speed is slow -- I spent about 1.5 hours. > > > > If these patches solved all the problems and made things work > > wonderfully I'd probably say we'd take them. Unfortunately I don't > > consider taking 1.5 hours to build am 8GB filesystem "wonderful", its > > rather worrying and I don't think its performing any where need fast > > enough for our needs :(. > > > > Patching genext2fs at this point in the cycle is a rather risky > > undertaking too and I'm getting very concerned about this. > > The biggest issue I have with genext2fs is that it currently allocates > RAM for the complete image, which in my case will cause it to swap and > for other users cause it to crash, since image > (ram + swap). Fixing > the memory issue before the release would be greatly appreciated. > > What about putting it on the shortlist for backports after the > release? That should give us all more time to test it and eventually > get into the hands of stable branch users. Of course I'd still prefer > it to go in before the release and backport fixes :) I should point out one of the patches in this mega patch set is one for the memory issue. I'd appreciate some others giving this patch set a go, see if they can find any regressions for their usual use cases. Based on the feedback I've had so far I'm leaning towards merging it but I'm not making a final decision yet, I'd like some test reports... Cheers, Richard