Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1763408AbZANPNs (ORCPT ); Wed, 14 Jan 2009 10:13:48 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1763260AbZANPM5 (ORCPT ); Wed, 14 Jan 2009 10:12:57 -0500 Received: from krynn.se.axis.com ([193.13.178.10]:50959 "EHLO krynn.se.axis.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1763185AbZANPM4 (ORCPT ); Wed, 14 Jan 2009 10:12:56 -0500 Date: Wed, 14 Jan 2009 16:12:21 +0100 From: Jesper Nilsson To: David Miller Cc: "roucaries.bastien@gmail.com" , "linux-kernel@vger.kernel.org" , "tj@kernel.org" , "gregkh@suse.de" , "stern@rowland.harvard.edu" , "jens.axboe@oracle.com" , "hinko.kocevar@cetrtapot.si" Subject: Re: lib/klist.c: bit 0 in pointer can't be used as flag Message-ID: <20090114151221.GK19262@axis.com> References: <20090114101908.GF19262@axis.com> <20090114.022140.169499153.davem@davemloft.net> <20090114103624.GH19262@axis.com> <20090114.024528.80563421.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20090114.024528.80563421.davem@davemloft.net> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2041 Lines: 48 On Wed, Jan 14, 2009 at 11:45:28AM +0100, David Miller wrote: > From: Jesper Nilsson > Date: Wed, 14 Jan 2009 11:36:24 +0100 > > > On Wed, Jan 14, 2009 at 11:21:40AM +0100, David Miller wrote: > > > From: Jesper Nilsson > > > Date: Wed, 14 Jan 2009 11:19:08 +0100 > > > > The following patch works for CRIS, and is less intrusive than > > > > my earlier patch. If this the way to go I can resubmit a proper patch. > > > > > > Out of curiosity, is there a way to get gcc to output code such > > > that data objects are aligned more naturally? Some option or > > > similar? > > > > There's flags for alignment of objects, but no flags for changing > > structure layout or size, which is probably what we run into here. > > Really? Yes, after some more research, I found that the bug is indeed triggered by the struct klist being aligned to odd bytes inside other structs. Since CRIS uses packed structs, having a char * inside the kobjects shifts the alignment for all data after any struct that contains a kobject (and indeed inside the kobject itself) In this case it was the class pointer of the struct device that had a klist in the private pointer (struct class_private), after a struct kset (which contains a kobject). > I thought the problem is that the base object can have any odd > alignment in the kernel image. And this is why the problem isn't run > into for objects allocated using dynamic allocation. Actually, it seems that the default is for objects to be aligned at 16bits for CRIS, which may also explain why this is not seen more often. Objects allocated dynamically may still trigger the bug, due to the struct not being padded. /^JN - Jesper Nilsson -- Jesper Nilsson -- jesper.nilsson@axis.com -- 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/