From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [Qemu-devel] [RFC PATCH 1/7] devicetree: Add 8k instead of double dtb size when reserving extra memory Date: Fri, 9 Apr 2010 08:55:56 -0600 Message-ID: References: <20100407040129.20274.44284.stgit@angua> <20100407041002.20274.51604.stgit@angua> <201004091300.57740.paul@codesourcery.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <201004091300.57740.paul-qD8j1LwMmJjtCj0u4l0SBw@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Paul Brook Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, qemu-devel-qX2TKyscuCcdnm+yROfE0A@public.gmane.org, jeremy.kerr-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org List-Id: devicetree@vger.kernel.org On Fri, Apr 9, 2010 at 6:00 AM, Paul Brook wrote: >> If a small 'seed' dtb file is loaded into qemu and then heavily modified >> (say for runtime population of all the device nodes), then 2x the dtb >> size turns out not to be very much. >> >> This patch changes the device tree loading code to add a fixed 8k of >> additional space to the dtb buffer. > > In that case you almost certainly want double+8k. Or have the calleee tell you > how much they're expecting to grow the tree. Good point. I think I like the latter option. g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd.