Messages in this thread | | | Date | Thu, 15 Sep 2011 16:43:08 -0500 | From | Jonathan Nieder <> | Subject | Re: [PATCH 07/27] Initconst section fixes for POWERPC |
| |
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.
| |