lkml.org 
[lkml]   [2011]   [Sep]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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.


\
 
 \ /
  Last update: 2011-09-15 23:45    [W:0.077 / U:0.376 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site