Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v3 1/6] MIPS: Loongson64: Do not write the read only field
 LPA of CP0_CONFIG3
To:     Huacai Chen <chenhc@lemote.com>
References: <1604387525-23400-1-git-send-email-yangtiezhu@loongson.cn>
 <1604387525-23400-2-git-send-email-yangtiezhu@loongson.cn>
 <CAAhV-H4WfaCLuCzvCJx-UriqgPAz2b0H6LGwMhyhRaxvuSAMwQ@mail.gmail.com>
Cc:     Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        Jiaxun Yang <jiaxun.yang@flygoat.com>,
        "open list:MIPS" <linux-mips@vger.kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Xuefeng Li <lixuefeng@loongson.cn>
From:   Tiezhu Yang <yangtiezhu@loongson.cn>
Message-ID: <e999986a-8236-752a-8b17-353bb87fc521@loongson.cn>
Date:   Wed, 4 Nov 2020 11:50:53 +0800
User-Agent: Mozilla/5.0 (X11; Linux mips64; rv:45.0) Gecko/20100101
 Thunderbird/45.4.0
MIME-Version: 1.0
In-Reply-To: <CAAhV-H4WfaCLuCzvCJx-UriqgPAz2b0H6LGwMhyhRaxvuSAMwQ@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 11/04/2020 10:00 AM, Huacai Chen wrote:
> Hi, Tiezhu,
>
> On Tue, Nov 3, 2020 at 3:13 PM Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
>> The field LPA of CP0_CONFIG3 register is read only for Loongson64, so the
>> write operations are meaningless, remove them.
>>
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>> ---
>>
>> v2: No changes
>> v3: No changes
>>
>>   arch/mips/include/asm/mach-loongson64/kernel-entry-init.h | 8 --------
>>   arch/mips/loongson64/numa.c                               | 3 ---
>>   2 files changed, 11 deletions(-)
>>
>> diff --git a/arch/mips/include/asm/mach-loongson64/kernel-entry-init.h b/arch/mips/include/asm/mach-loongson64/kernel-entry-init.h
>> index 87a5bfb..e4d77f4 100644
>> --- a/arch/mips/include/asm/mach-loongson64/kernel-entry-init.h
>> +++ b/arch/mips/include/asm/mach-loongson64/kernel-entry-init.h
>> @@ -19,10 +19,6 @@
>>          .macro  kernel_entry_setup
>>          .set    push
>>          .set    mips64
>> -       /* Set LPA on LOONGSON3 config3 */
>> -       mfc0    t0, CP0_CONFIG3
>> -       or      t0, (0x1 << 7)
>> -       mtc0    t0, CP0_CONFIG3
> Sorry for the late response, I have the same worry as Jiaxun. As you
> know, Loongson's user manuals are not always correct, but the original
> code comes from Loongson are usually better. So, my opinion is "Don't
> change it if it doesn't break anything".

Hi Huacai,

Thanks for your reply, I have confirmed by Loongson user manuals and
hardware designers, CP0_CONFIG3 register is read only.

Without this patch, the related kernel code is meaningless, with
this patch, it can reflect the reality.

Thanks,
Tiezhu

>
> Huacai
>
>>          /* Set ELPA on LOONGSON3 pagegrain */
>>          mfc0    t0, CP0_PAGEGRAIN
>>          or      t0, (0x1 << 29)
>> @@ -54,10 +50,6 @@
>>          .macro  smp_slave_setup
>>          .set    push
>>          .set    mips64
>> -       /* Set LPA on LOONGSON3 config3 */
>> -       mfc0    t0, CP0_CONFIG3
>> -       or      t0, (0x1 << 7)
>> -       mtc0    t0, CP0_CONFIG3
>>          /* Set ELPA on LOONGSON3 pagegrain */
>>          mfc0    t0, CP0_PAGEGRAIN
>>          or      t0, (0x1 << 29)
>> diff --git a/arch/mips/loongson64/numa.c b/arch/mips/loongson64/numa.c
>> index cf9459f..c7e3cced 100644
>> --- a/arch/mips/loongson64/numa.c
>> +++ b/arch/mips/loongson64/numa.c
>> @@ -40,9 +40,6 @@ static void enable_lpa(void)
>>          unsigned long value;
>>
>>          value = __read_32bit_c0_register($16, 3);
>> -       value |= 0x00000080;
>> -       __write_32bit_c0_register($16, 3, value);
>> -       value = __read_32bit_c0_register($16, 3);
>>          pr_info("CP0_Config3: CP0 16.3 (0x%lx)\n", value);
>>
>>          value = __read_32bit_c0_register($5, 1);
>> --
>> 2.1.0
>>