Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 20 May 2022 11:56:02 +0100 (BST)
From:   "Maciej W. Rozycki" <macro@orcam.me.uk>
To:     =?UTF-8?B?6ZmI5a2m5YW1?= <chenxuebing@jari.cn>
cc:     Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        linux-mips@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] [MIPS] DEC:do not initialise statics to 0
In-Reply-To: <4d4c0b64.993.180d7a6945a.Coremail.chenxuebing@jari.cn>
Message-ID: <alpine.DEB.2.21.2205200308000.52080@angie.orcam.me.uk>
References: <4d4c0b64.993.180d7a6945a.Coremail.chenxuebing@jari.cn>
User-Agent: Alpine 2.21 (DEB 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
Precedence: bulk

On Wed, 18 May 2022, 陈学兵 wrote:

> ​Static variables do not need to be initialised to 0,

 This is true.

> because compiler
> will initialise all uninitialised statics to 0.

 This is however not (even though such zeroing is mandated by the C 
language standard), because it is this code:

	PTR_LA		t0, __bss_start		# clear .bss
	LONG_S		zero, (t0)
	PTR_LA		t1, __bss_stop - LONGSIZE
1:
	PTR_ADDIU	t0, LONGSIZE
	LONG_S		zero, (t0)
	bne		t0, t1, 1b

in arch/mips/kernel/head.S that does it in Linux (in userland code this 
will be in CRT startup, but it's not a part of the compiler either).  So 
please avoid spreading misleading information in your change description.

> diff --git a/arch/mips/dec/prom/identify.c b/arch/mips/dec/prom/identify.c
> index 80cd14cd1a63..89e736670ccf 100644
> --- a/arch/mips/dec/prom/identify.c
> +++ b/arch/mips/dec/prom/identify.c
> @@ -45,7 +45,7 @@ const char *get_system_type(void)
>  {
>  #define STR_BUF_LEN 64
>   static char system[STR_BUF_LEN];
> - static int called = 0;
> + static int called;

 Barring the inaccuracy of the change description your code update is 
correct, however your patch has been whitespace-damaged in transit and 
cannot be applied as it stands.

 Please send an updated version with the problems addressed.

 NB the same concerns apply to your other such change submitted.

  Maciej