Re: [PATCH 3/4] vmlinux.ld.S: correctly indent .data section

[Franck Bui-Huu <fbuihuu@gmail.com>]

Andrew Sharp wrote:
> On Thu, 18 Oct 2007 23:12:32 +0200 Franck Bui-Huu <fbuihuu@gmail.com>
> wrote:
> 
>> Signed-off-by: Franck Bui-Huu <fbuihuu@gmail.com>
>> ---
>>  arch/mips/kernel/vmlinux.lds.S |   32
>> +++++++++++++++++--------------- 1 files changed, 17 insertions(+),
>> 15 deletions(-)
>>
>> diff --git a/arch/mips/kernel/vmlinux.lds.S
>> b/arch/mips/kernel/vmlinux.lds.S index e0a4dc0..2f6c225 100644
>> --- a/arch/mips/kernel/vmlinux.lds.S
>> +++ b/arch/mips/kernel/vmlinux.lds.S
>> @@ -55,21 +55,23 @@ SECTIONS
>>  
>>      /* writeable */
>>      .data : {       /* Data */
>> -      . = . + DATAOFFSET;           /* for
>> CONFIG_MAPPED_KERNEL */
>> -      /*
>> -       * This ALIGN is needed as a workaround for a bug a gcc
>> bug upto 4.1 which
>> -       * limits the maximum alignment to at most 32kB and
>> results in the following
>> -       * warning:
>> -       *
>> -       *  CC      arch/mips/kernel/init_task.o
>> -       * arch/mips/kernel/init_task.c:30: warning: alignment of
>> â??init_thread_unionâ??
>> -       * is greater than maximum object file alignment.  Using
>> 32768
>> -       */
>> -      . = ALIGN(_PAGE_SIZE);
>> -      *(.data.init_task)
>> -
>> -      DATA_DATA
>> -      CONSTRUCTORS
>> +            . = . + DATAOFFSET;             /* for
>> CONFIG_MAPPED_KERNEL */
>> +            /*
>> +             * This ALIGN is needed as a workaround for a bug a
>> +             * gcc bug upto 4.1 which limits the maximum
> 
> at least fix the text while you're here: 'a bug a gcc bug'
> 
>> +             * to at most 32kB and results in the following
>> +             * warning:
>> +             *
>> +             *  CC      arch/mips/kernel/init_task.o
>> +             * arch/mips/kernel/init_task.c:30: warning:
>> alignment
>> +             * of â??init_thread_unionâ?? is greater than maximum
> 
> is that utf-8 or something?  probably should remove that.
> 
> sorry for the picayunes, but hey as long as you're here changing the
> indenting ~:^)
> 
> 

Okay I'll fix them.

Thanks
                Franck