Re: [PATCH] parisc: Kconfig: use 'PA8X00' as the default choise instead of 'PA7000'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/27/2013 04:08 AM, Helge Deller wrote:
> Hi Chen,
> 
> On 07/25/2013 02:37 AM, Chen Gang F T wrote:
>> On 07/24/2013 10:00 PM, James Bottomley wrote:
>>>
>>> On Wed, 2013-07-24 at 16:12 +0800, Chen Gang wrote:
>>>> For server, 64-bit kernel is widely used and '64BIT' have been already
>>>> as the default option of a server.
>>>>
>>>> So better to use 64-bit kernel as default choise instead of 32-bit
>>>> kernel.
>>>
>>> I think this is irrelevant since we have defconfig for this.
> 
> I do have the same opinion.
> It's not necessary.
> 
> Furthermore, for a real 64 bit kernel you need to enable
> CONFIG_64BIT not CONFIG_PA8X00.
> 

But for "allmodconfig", it can not enable 'CONFIG_64BIT' automatically.

>> Hmm... but I still suggest when use 'allmodconfig' let 64-bit as the
>> default choice (normally 64-bit is the default choice for servers).
> 
> Yeah, but we do support workstations as well.
> 

OH, thanks, I really don't know about it (today I know :-) ), in my
original memory, parisc is mainly for server and well known by many members.

As far as I know, we often set mainly used item as default choice, if
have several mainly used item, need set general or compatible item as
default choice.

If 'PA8X00' is our mainly used item (only 'PA8X00' support 64-bit), I
still suggest to set it as default choice. Or when 'allmodconfig' works
for a server, finds 'CONFIG_64BIT' unset, it really wonders "Why ?"

>>>> Or when use cross-compiler 'hppa64-linux-gnu-gcc' with allmodconfig, it
>>>> will report BUILD_BUG_ON:
>>>>
>>>>   arch/parisc/mm/init.c: In function ‘mem_init’:
>>>>   arch/parisc/mm/init.c:581:2: error: call to ‘__compiletime_assert_581’ declared with attribute error: BUILD_BUG_ON failed: PTE_ENTRY_SIZE != sizeof(pte_t)
>>>
>>> This shows you're using a buggy compiler.  PTE_OFFSET_SIZE is generated
>>> as part of the build process, so you need a correct compiler.
>>>
>>
>> Hmm... It means the compiler assumes to build 64-bit code (hppa64), but
>> our configuration is for 32-bit, they are mismatch.
>>
>> Maybe we can say hppa64-linux-gnu-gcc can not build the 32-bit parisc
>> code (although it is not precise), but at least, it is not a real bug.
> 
> It's a fact:
> hppa64-linux-gnu-gcc can *not* build the 32-bit parisc.
> Only hppa-linux-gnu-gcc can build 32bit code. 
> 
> 

Thank you for your confirmation.

I should build hppa-linux-gnu-gcc cross-compiler firstly.

Hmm... I try to finish it within this week (but not quite sure), but at
least I should finish it within next month (2013-08-31).

Welcome any additional suggestions or completions (especially the
related information for parisc cross-compiler).

>> This contents wants to say: when use 'allmodconfig' for a server, we do
>> not build a 64-bit server.
>>
>>
>>>> Signed-off-by: Chen Gang <gang.chen@xxxxxxxxxxx>
> 
> Thanks for your patch, but:
> NAK.
> 

OK, I can understand.

Thank you too.

> Helge
> 
>>>> ---
>>>>  arch/parisc/Kconfig |    2 +-
>>>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>>>
>>>> diff --git a/arch/parisc/Kconfig b/arch/parisc/Kconfig
>>>> index aa399a5..37bdecf 100644
>>>> --- a/arch/parisc/Kconfig
>>>> +++ b/arch/parisc/Kconfig
>>>> @@ -111,7 +111,7 @@ menu "Processor type and features"
>>>>  
>>>>  choice
>>>>  	prompt "Processor type"
>>>> -	default PA7000
>>>> +	default PA8X00
>>>
>>> This doesn't require 64 bit: PA8xxx processors can operate happily in 32
>>> bit mode it does make the question appear, since PA7xxx are 32 bit only.
>>>
>>
>> Yeah, 'PA8X00' can compitable with 32-bit, but it likes 64-bit, the
>> following config files are for 'PA8X00':
>>
>>   a500_defconfig: for 64-bit
>>   c3000_defconfig: for 32-bit
>>   c8000_defconfig: for 64-bit
>>   generic-64bit_defconfig: for 64-bit
>>
>> At least, we can say PA8xxx is a 64-bit processor for parisc, is it
>> correct ?
>>
>> And for 'allmodconfig', if let 'PA8X00' as default choice, it will build
>> 64-bit kernel.
>>
>>
>> Thanks.
>>
> 


-- 
Chen Gang

-- 
Chen Gang
--
To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux