[PATCH] MIPS: Loongson: Make default kernel log buffer size as 128KB for Loongson3

[PATCH] MIPS: Loongson: Make default kernel log buffer size as 128KB for Loongson3

[Tiezhu Yang]

When I update kernel with loongson3_defconfig based on the Loongson 3A3000
platform, then using dmesg command to show kernel ring buffer, the initial
kernel messages have disappeared due to the log buffer is too small, it is
better to change the default kernel log buffer size from 16KB to 128KB.

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 arch/mips/configs/loongson3_defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/configs/loongson3_defconfig b/arch/mips/configs/loongson3_defconfig
index 90ee008..3aa2201 100644
--- a/arch/mips/configs/loongson3_defconfig
+++ b/arch/mips/configs/loongson3_defconfig
@@ -12,7 +12,7 @@ CONFIG_TASKSTATS=y
 CONFIG_TASK_DELAY_ACCT=y
 CONFIG_TASK_XACCT=y
 CONFIG_TASK_IO_ACCOUNTING=y
-CONFIG_LOG_BUF_SHIFT=14
+CONFIG_LOG_BUF_SHIFT=17
 CONFIG_MEMCG=y
 CONFIG_MEMCG_SWAP=y
 CONFIG_BLK_CGROUP=y
-- 
2.1.0

Re: [PATCH] MIPS: Loongson: Make default kernel log buffer size as 128KB for Loongson3

[Huacai Chen]

On Tue, Oct 15, 2019 at 10:12 AM Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
>
> When I update kernel with loongson3_defconfig based on the Loongson 3A3000
> platform, then using dmesg command to show kernel ring buffer, the initial
> kernel messages have disappeared due to the log buffer is too small, it is
> better to change the default kernel log buffer size from 16KB to 128KB.
>
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> ---
>  arch/mips/configs/loongson3_defconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/mips/configs/loongson3_defconfig b/arch/mips/configs/loongson3_defconfig
> index 90ee008..3aa2201 100644
> --- a/arch/mips/configs/loongson3_defconfig
> +++ b/arch/mips/configs/loongson3_defconfig
> @@ -12,7 +12,7 @@ CONFIG_TASKSTATS=y
>  CONFIG_TASK_DELAY_ACCT=y
>  CONFIG_TASK_XACCT=y
>  CONFIG_TASK_IO_ACCOUNTING=y
> -CONFIG_LOG_BUF_SHIFT=14
> +CONFIG_LOG_BUF_SHIFT=17
Hi, Tiezhu,

Why you choose 128KB but not 64KB or 256KB? I found 64KB is enough for
our cases. And if you really need more, I think 256KB could be better
because there are many platforms choose 256KB.

Huacai

>  CONFIG_MEMCG=y
>  CONFIG_MEMCG_SWAP=y
>  CONFIG_BLK_CGROUP=y
> --
> 2.1.0
>
>

Re: [PATCH] MIPS: Loongson: Make default kernel log buffer size as 128KB for Loongson3

[Tiezhu Yang]

On 10/15/2019 11:36 AM, Huacai Chen wrote:
> On Tue, Oct 15, 2019 at 10:12 AM Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
>> When I update kernel with loongson3_defconfig based on the Loongson 3A3000
>> platform, then using dmesg command to show kernel ring buffer, the initial
>> kernel messages have disappeared due to the log buffer is too small, it is
>> better to change the default kernel log buffer size from 16KB to 128KB.
>>
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>> ---
>>   arch/mips/configs/loongson3_defconfig | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/mips/configs/loongson3_defconfig b/arch/mips/configs/loongson3_defconfig
>> index 90ee008..3aa2201 100644
>> --- a/arch/mips/configs/loongson3_defconfig
>> +++ b/arch/mips/configs/loongson3_defconfig
>> @@ -12,7 +12,7 @@ CONFIG_TASKSTATS=y
>>   CONFIG_TASK_DELAY_ACCT=y
>>   CONFIG_TASK_XACCT=y
>>   CONFIG_TASK_IO_ACCOUNTING=y
>> -CONFIG_LOG_BUF_SHIFT=14
>> +CONFIG_LOG_BUF_SHIFT=17
> Hi, Tiezhu,
>
> Why you choose 128KB but not 64KB or 256KB? I found 64KB is enough for
> our cases. And if you really need more, I think 256KB could be better
> because there are many platforms choose 256KB.

Hi Huacai,

Thanks for your reply and suggestion, I will send a v2 patch.

Thanks,

Tiezhu Yang

>
> Huacai
>
>>   CONFIG_MEMCG=y
>>   CONFIG_MEMCG_SWAP=y
>>   CONFIG_BLK_CGROUP=y
>> --
>> 2.1.0
>>
>>

Re: [PATCH] MIPS: Loongson: Make default kernel log buffer size as 128KB for Loongson3

[Paul Burton]

Hi Tiezhu & Huacai,

On Tue, Oct 15, 2019 at 12:00:25PM +0800, Tiezhu Yang wrote:
> On 10/15/2019 11:36 AM, Huacai Chen wrote:
> > On Tue, Oct 15, 2019 at 10:12 AM Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
> > > When I update kernel with loongson3_defconfig based on the Loongson 3A3000
> > > platform, then using dmesg command to show kernel ring buffer, the initial
> > > kernel messages have disappeared due to the log buffer is too small, it is
> > > better to change the default kernel log buffer size from 16KB to 128KB.
> > > 
> > > Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
> > > ---
> > >   arch/mips/configs/loongson3_defconfig | 2 +-
> > >   1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/arch/mips/configs/loongson3_defconfig b/arch/mips/configs/loongson3_defconfig
> > > index 90ee008..3aa2201 100644
> > > --- a/arch/mips/configs/loongson3_defconfig
> > > +++ b/arch/mips/configs/loongson3_defconfig
> > > @@ -12,7 +12,7 @@ CONFIG_TASKSTATS=y
> > >   CONFIG_TASK_DELAY_ACCT=y
> > >   CONFIG_TASK_XACCT=y
> > >   CONFIG_TASK_IO_ACCOUNTING=y
> > > -CONFIG_LOG_BUF_SHIFT=14
> > > +CONFIG_LOG_BUF_SHIFT=17
> > Hi, Tiezhu,
> > 
> > Why you choose 128KB but not 64KB or 256KB? I found 64KB is enough for
> > our cases. And if you really need more, I think 256KB could be better
> > because there are many platforms choose 256KB.
> 
> Hi Huacai,
> 
> Thanks for your reply and suggestion, I will send a v2 patch.

Thanks for the patches.

I actually have a slight preference for 128KB if you've no specific
need, since 128KB is the default. Some quick grepping says that of 405
defconfigs in tree (as of v5.4-rc3), we have:

  LOG_BUF_SHIFT  Count
             12  1
	     13  3
	     14  235
	     15  18
	     16  39
	     17  90
	     18  13
	     19  2
	     20  4

ie. 16KiB is by far the most common, then second most common is the
default 128KiB. 256KiB is comparatively rare.

However, I don't think your v1 patch is quite right Tiezhu - since 17 is
the default it shouldn't be specified in the defconfig at all. Did you
manually make the change in the loongson3_defconfig file? If so please
take a look at the savedefconfig make target & try something like this:

  make ARCH=mips loongson3_defconfig
  make ARCH=mips menuconfig
  # Change LOG_BUF_SHIFT
  make ARCH=mips savedefconfig
  mv defconfig arch/mips/configs/loongson3_defconfig
  git add -i arch/mips/configs/loongson3_defconfig
  # Stage the relevant changes, drop the others

You should end up with the CONFIG_LOG_BUF_SHIFT line just getting
deleted.

If on the other hand you really do prefer 256KiB for these systems
please describe why in the commit message. It could be something as
simple as "we have lots of memory so using 256KiB isn't a big deal, and
gives us a better chance of preserving boot messages until they're
examined". But if your log is getting this big before you look at it (or
before something like systemd copies it into its journal), there's
probably something fishy going on.

Thanks,
    Paul

Re: [PATCH] MIPS: Loongson: Make default kernel log buffer size as 128KB for Loongson3

[Tiezhu Yang]

On 10/16/2019 02:53 AM, Paul Burton wrote:
> Hi Tiezhu & Huacai,
>
> On Tue, Oct 15, 2019 at 12:00:25PM +0800, Tiezhu Yang wrote:
>> On 10/15/2019 11:36 AM, Huacai Chen wrote:
>>> On Tue, Oct 15, 2019 at 10:12 AM Tiezhu Yang <yangtiezhu@loongson.cn> wrote:
>>>> When I update kernel with loongson3_defconfig based on the Loongson 3A3000
>>>> platform, then using dmesg command to show kernel ring buffer, the initial
>>>> kernel messages have disappeared due to the log buffer is too small, it is
>>>> better to change the default kernel log buffer size from 16KB to 128KB.
>>>>
>>>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>>>> ---
>>>>    arch/mips/configs/loongson3_defconfig | 2 +-
>>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/arch/mips/configs/loongson3_defconfig b/arch/mips/configs/loongson3_defconfig
>>>> index 90ee008..3aa2201 100644
>>>> --- a/arch/mips/configs/loongson3_defconfig
>>>> +++ b/arch/mips/configs/loongson3_defconfig
>>>> @@ -12,7 +12,7 @@ CONFIG_TASKSTATS=y
>>>>    CONFIG_TASK_DELAY_ACCT=y
>>>>    CONFIG_TASK_XACCT=y
>>>>    CONFIG_TASK_IO_ACCOUNTING=y
>>>> -CONFIG_LOG_BUF_SHIFT=14
>>>> +CONFIG_LOG_BUF_SHIFT=17
>>> Hi, Tiezhu,
>>>
>>> Why you choose 128KB but not 64KB or 256KB? I found 64KB is enough for
>>> our cases. And if you really need more, I think 256KB could be better
>>> because there are many platforms choose 256KB.
>> Hi Huacai,
>>
>> Thanks for your reply and suggestion, I will send a v2 patch.
> Thanks for the patches.
>
> I actually have a slight preference for 128KB if you've no specific
> need, since 128KB is the default. Some quick grepping says that of 405
> defconfigs in tree (as of v5.4-rc3), we have:
>
>    LOG_BUF_SHIFT  Count
>               12  1
> 	     13  3
> 	     14  235
> 	     15  18
> 	     16  39
> 	     17  90
> 	     18  13
> 	     19  2
> 	     20  4
>
> ie. 16KiB is by far the most common, then second most common is the
> default 128KiB. 256KiB is comparatively rare.

Hi Paul,

Thank you very much for your detailed explanation. I think 128KB is 
enough to save
the boot messages because the dmesg output size is about 40KB after 
reboot in my
system.

>
> However, I don't think your v1 patch is quite right Tiezhu - since 17 is
> the default it shouldn't be specified in the defconfig at all. Did you
> manually make the change in the loongson3_defconfig file? If so please
> take a look at the savedefconfig make target & try something like this:
>
>    make ARCH=mips loongson3_defconfig
>    make ARCH=mips menuconfig
>    # Change LOG_BUF_SHIFT
>    make ARCH=mips savedefconfig
>    mv defconfig arch/mips/configs/loongson3_defconfig
>    git add -i arch/mips/configs/loongson3_defconfig
>    # Stage the relevant changes, drop the others
>
> You should end up with the CONFIG_LOG_BUF_SHIFT line just getting
> deleted.

Yes, you are right, I will make a new patch followed with your steps and 
send a v3
patch: just delete the CONFIG_LOG_BUF_SHIFT line to use the default 
value 128KB and
update the commit message.

>
> If on the other hand you really do prefer 256KiB for these systems
> please describe why in the commit message. It could be something as
> simple as "we have lots of memory so using 256KiB isn't a big deal, and
> gives us a better chance of preserving boot messages until they're
> examined". But if your log is getting this big before you look at it (or
> before something like systemd copies it into its journal), there's
> probably something fishy going on.

Hi Huacai,

What do you think? If you have any more suggestion, please let me know.

Thanks,

Tiezhu Yang

>
> Thanks,
>      Paul