From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: owner-linux-mm@kvack.org Subject: BOUNCE linux-mm@kvack.org: Header field too long (>2048) Message-Id: <20120330100002.8F5896B004D@kanga.kvack.org> Date: Fri, 30 Mar 2012 06:00:02 -0400 (EDT) Sender: owner-linux-mm@kvack.org List-ID: To: linux-mm-approval@kvack.org >>From bcrl@kvack.org Fri Mar 30 06:00:02 2012 Return-Path: X-Original-To: int-list-linux-mm@kvack.org Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 71D416B007E; Fri, 30 Mar 2012 06:00:02 -0400 (EDT) X-Original-To: linux-mm@kvack.org Delivered-To: linux-mm@kvack.org Received: from psmtp.com (na3sys010amx139.postini.com [74.125.245.139]) by kanga.kvack.org (Postfix) with SMTP id BCB776B004A for ; Fri, 30 Mar 2012 06:00:01 -0400 (EDT) Received: from metis.ext.pengutronix.de ([92.198.50.35]) (using TLSv1) by na3sys010amx139.postini.com ([74.125.244.10]) with SMTP; Fri, 30 Mar 2012 10:00:01 GMT Received: from dude.hi.pengutronix.de ([2001:6f8:1178:2:21e:67ff:fe11:9c5c]) by metis.ext.pengutronix.de with esmtp (Exim 4.72) (envelope-from ) id 1SDYc7-0004Tg-IU; Fri, 30 Mar 2012 11:59:23 +0200 Received: from ukl by dude.hi.pengutronix.de with local (Exim 4.77) (envelope-from ) id 1SDYbk-0004Z4-0y; Fri, 30 Mar 2012 11:59:00 +0200 Date: Fri, 30 Mar 2012 11:59:00 +0200 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= To: linux-kernel@vger.kernel.org, Andrew Morton Cc: Alexey Dobriyan , Anatolij Gustschin , Andreas Koensgen , Andrew Lunn , Andrew Victor , Arnd Bergmann , Barry Song , Benjamin Herrenschmidt , Bryan Huntsman , cbe-oss-dev@lists.ozlabs.org, Christoph Lameter , Daniel Walker , David Brown , David Howells , "David S. Miller" , David Woodhouse , davinci-linux-open-source@linux.davincidsp.com, Eric Miao , Fenghua Yu , Grant Likely , Guenter Roeck , Haojian Zhuang , Henrique de Moraes Holschuh , ibm-acpi-devel@lists.sourceforge.net, Jean-Christophe Plagniol-Villard , Jean Delvare , Jean-Paul Roubelat , Joerg Reuter , Josh Boyer , Kevin Hilman , Klaus Kudielka , Kukjin Kim , Kumar Gala , Kyungmin Park , Lennert Buytenhek , Linus Walleij , Linus Walleij , linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-hams@vger.kernel.org, linux-ia64@vger.kernel.org, linux-ide@vger.kernel.org, linux-media@vger.kernel.org, linux-mips@linux-mips.org, linux-mm@kvack.org, linux-mtd@lists.infradead.org, linux-omap@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-samsung-soc@vger.kernel.org, lm-sensors@lm-sensors.org, Lucas De Marchi , Matthew Garrett , Matt Porter , Mauro Carvalho Chehab , netdev@vger.kernel.org, Nicolas Ferre , Nicolas Pitre , Paul Mackerras , platform-driver-x86@vger.kernel.org, Ralf Baechle , Randy Dunlap , Russell King , Samuel Ortiz , Sascha Hauer , Sekhar Nori , Shawn Guo , Tejun Heo , Tomasz Stanislawski , Tony Lindgren , Tony Luck , Yoshinori Sato Subject: Re: [PATCH 00/17] mark const init data with __initconst instead of __initdata Message-ID: <20120330095859.GT15647@pengutronix.de> References: <20120329211131.GA31250@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20120329211131.GA31250@pengutronix.de> User-Agent: Mutt/1.5.21 (2010-09-15) X-SA-Exim-Connect-IP: 2001:6f8:1178:2:21e:67ff:fe11:9c5c X-SA-Exim-Mail-From: ukl@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-mm@kvack.org X-pstn-neptune: 1/1/1.00/100 X-pstn-levels: (S:99.90000/99.90000 CV:99.9000 FC:95.5390 LC:95.5390 R:95.9108 P:95.9108 M:97.0282 C:98.6951 ) X-pstn-dkim: 0 skipped:not-enabled X-pstn-settings: 3 (1.0000:1.0000) s cv gt3 gt2 gt1 r p m c X-pstn-addresses: from [db-null] X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.2 On Thu, Mar 29, 2012 at 11:11:31PM +0200, Uwe Kleine-Ki? 1/2 nig wrote: > Hello, > > this series fixes a common error to use __initdata to mark const > variables. Most of the time this works well enough to go unnoticed > (though I wonder why the linker doesn't warn about that). > Just try adding something like > > int something __initdata; > > to one of the patched files and compile to see the error. > > While touching these annotations I also corrected the position where it > was wrong to go between the variable name and the =. > > Note this series is not compile tested. After a question by Shawn Guo I noticed that my command to do the changes was to lax and changed things that must not be changed (at least not with further care). Affected are lines like: static const char *at91_dt_board_compat[] __initconst = { While at91_dt_board_compat[0] is const, at91_dt_board_compat is not. I will send a fixed series later today. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-Ki? 1/2 nig | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org