From: Jonathan Nieder <jrnieder@gmail.com>
To: Andi Kleen <ak@linux.intel.com>
Cc: Josh Boyer <jwboyer@gmail.com>, Andi Kleen <andi@firstfloor.org>,
linux-kernel@vger.kernel.org, benh@kernel.crashing.org
Subject: Re: [PATCH 07/27] Initconst section fixes for POWERPC
Date: Thu, 15 Sep 2011 16:43:08 -0500 [thread overview]
Message-ID: <20110915214308.GA779@elie> (raw)
In-Reply-To: <4E726489.6080606@linux.intel.com>
On Thu, Sep 15, 2011 at 01:48:09PM -0700, Andi Kleen wrote:
> On 9/15/2011 1:45 PM, Josh Boyer wrote:
>> On Thu, Sep 15, 2011 at 4:09 PM, Andi Kleen<andi@firstfloor.org> wrote:
>>> From: Andi Kleen<ak@linux.intel.com>
>>
>> What's the problem again?
>
> It's wrong. For more details see 0/*
Doesn't that mean that people using "git log", perhaps trying to debug
fallout from one of the patches (even if it's as small as making
context not match on some later patch), would have no idea what's
going on? The cover letter says:
> This is a global tree sweep to fix up initconst section problems.
>
> Any variable const in initdata must be marked __initconst, not initdata.
> const __read_mostly is not allowed.
>
> Otherwise the global sections can get a read only marking and in some
> circumstances the compiler/and or the linker will detect conflicts.
>
> This patchkit aims to fix it up everywhere.
Which seems more sensible. Since this is going through Linus's tree,
it might (or might not) make sense to combine the patches and describe
the effect once.
next prev parent reply other threads:[~2011-09-15 21:43 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-15 20:09 Tree sweep to fix up initconst Andi Kleen
2011-09-15 20:09 ` [PATCH 01/27] Initconst section fixes for ARM Andi Kleen
2011-09-15 20:09 ` [PATCH 02/27] Initconst section fixes for FRV Andi Kleen
2011-09-20 15:18 ` David Howells
2011-09-15 20:09 ` [PATCH 03/27] Initconst section fixes for H8300 Andi Kleen
2011-09-15 20:09 ` [PATCH 04/27] Initconst section fixes for IA64 Andi Kleen
2011-09-15 20:09 ` [PATCH 05/27] Initconst section fixes for M68K Andi Kleen
2011-10-31 18:05 ` Geert Uytterhoeven
2011-09-15 20:09 ` [PATCH 06/27] Initconst section fixes for MIPS Andi Kleen
2011-09-22 16:59 ` Ralf Baechle
2011-09-15 20:09 ` [PATCH 07/27] Initconst section fixes for POWERPC Andi Kleen
2011-09-15 20:45 ` Josh Boyer
2011-09-15 20:48 ` Andi Kleen
2011-09-15 20:56 ` Josh Boyer
2011-09-15 20:57 ` Andi Kleen
2011-09-15 23:04 ` Josh Boyer
2011-09-15 23:10 ` Andi Kleen
2011-09-15 23:26 ` Thomas Gleixner
2011-09-15 21:43 ` Jonathan Nieder [this message]
2011-09-15 20:09 ` [PATCH 08/27] Initconst section fixes for SH Andi Kleen
2011-09-15 20:09 ` [PATCH 09/27] Initconst section fixes for X86 Andi Kleen
2011-09-15 20:09 ` [PATCH 10/27] Initconst section fixes for CPUFREQ Andi Kleen
2011-09-15 21:43 ` Dave Jones
2011-09-15 20:09 ` [PATCH 11/27] Initconst section fixes for EDAC Andi Kleen
2011-09-15 20:09 ` [PATCH 12/27] Initconst section fixes for hwmon Andi Kleen
2011-09-15 20:09 ` [PATCH 13/27] Initconst section fixes for IDE Andi Kleen
2011-09-15 21:12 ` David Miller
2011-09-15 20:09 ` [PATCH 14/27] Initconst section fixes for macintosh Andi Kleen
2011-09-15 20:09 ` [PATCH 15/27] Initconst section fixes for media Andi Kleen
2011-09-15 20:09 ` [PATCH 16/27] Initconst section fixes for mfd Andi Kleen
2011-09-26 8:48 ` Samuel Ortiz
2011-09-15 20:09 ` [PATCH 17/27] Initconst section fixes for mmc Andi Kleen
2011-09-15 20:09 ` [PATCH 18/27] Initconst section fixes for mtd Andi Kleen
2011-09-15 20:09 ` [PATCH 19/27] Initconst section fixes for drivers/net Andi Kleen
2011-09-15 21:12 ` David Miller
2011-09-15 20:09 ` [PATCH 20/27] Initconst section fixes for SCSI Andi Kleen
2011-09-15 20:18 ` James Bottomley
2011-09-15 20:30 ` Andi Kleen
2011-09-15 20:09 ` [PATCH 21/27] Initconst section fixes for SPI Andi Kleen
2011-09-15 20:57 ` Grant Likely
2011-09-15 20:59 ` Andi Kleen
2011-09-15 20:09 ` [PATCH 22/27] Initconst section fixes for TTY Andi Kleen
2011-09-15 20:09 ` [PATCH 23/27] Initconst section fixes for VIDEO Andi Kleen
2011-09-15 20:09 ` [PATCH 24/27] Initconst section fixes for watchdog Andi Kleen
2011-09-15 20:09 ` [PATCH 25/27] Initconst section fixes for percpu Andi Kleen
2011-09-17 0:41 ` Tejun Heo
2011-09-15 20:09 ` [PATCH 26/27] Initconst section fixes for net Andi Kleen
2011-09-15 21:12 ` David Miller
2011-09-15 20:09 ` [PATCH 27/27] Initconst section fixes for lib Andi Kleen
2011-09-15 23:33 ` Tree sweep to fix up initconst Thomas Gleixner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110915214308.GA779@elie \
--to=jrnieder@gmail.com \
--cc=ak@linux.intel.com \
--cc=andi@firstfloor.org \
--cc=benh@kernel.crashing.org \
--cc=jwboyer@gmail.com \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox