Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753078Ab0K3WGW (ORCPT ); Tue, 30 Nov 2010 17:06:22 -0500 Received: from caramon.arm.linux.org.uk ([78.32.30.218]:53931 "EHLO caramon.arm.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752273Ab0K3WGV (ORCPT ); Tue, 30 Nov 2010 17:06:21 -0500 Date: Tue, 30 Nov 2010 22:05:50 +0000 From: Russell King - ARM Linux To: Greg KH Cc: Arnd Bergmann , Linus Walleij , Jimmy RUBIN , Dan JOHANSSON , Marcus LORENTZON , Linux Kernel Mailing List , dri-devel@lists.freedesktop.org, "linux-arm-kernel@lists.infradead.org" , "linux-media@vger.kernel.org" Subject: Re: [PATCH 09/10] MCDE: Add build files and bus Message-ID: <20101130220550.GD8521@n2100.arm.linux.org.uk> References: <201011261224.59490.arnd@arndb.de> <201011301621.48140.arnd@arndb.de> <20101130184049.GC8521@n2100.arm.linux.org.uk> <20101130184834.GA16055@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101130184834.GA16055@kroah.com> User-Agent: Mutt/1.5.19 (2009-01-05) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1657 Lines: 33 On Tue, Nov 30, 2010 at 10:48:34AM -0800, Greg KH wrote: > On Tue, Nov 30, 2010 at 06:40:49PM +0000, Russell King - ARM Linux wrote: > > There's lots of static devices, not only platform devices, in the ARM > > tree. It's going to be a hell of a lot of work to fix this all up > > properly. > > I agree, it's been abused for many years this way :( I don't agree that it is abuse - it was something explicitly allowed by the original device model design by Patrick, with the condition that such a device was never unregistered. That's exactly the way we treat these devices. What I'm slightly concerned about is that this is going to needlessly bloat the kernel - we're going to have to find some other way to store this information, and create devices from that - which means additional code to do the creation, and data structures for it to create these from. There will be additional wastage from kmalloc as kmalloc doesn't allocate just the size you ask for, but normally a power of two which will contain the size. That could potentially mean that as the device structure is 216 bytes, kmalloc will use the 256 byte allocation size, which means a wastage of 40 bytes per device structure. On top of that goes the size of resources with the allocation slop on top for that, and then there's another allocation for the platform data. Has anyone considered these implications before making this choice? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/