Subject: Re: [PATCH] m68k: Enable memtest kernel parameter
To:     Finn Thain <fthain@linux-m68k.org>,
        Geert Uytterhoeven <geert@linux-m68k.org>
Cc:     Mike Rapoport <rppt@kernel.org>, Sam Creasey <sammy@sammy.net>,
        linux-m68k@lists.linux-m68k.org, linux-kernel@vger.kernel.org
References: <a57524fe38123ea17a6741cdc282f6c84c30a009.1636593047.git.fthain@linux-m68k.org>
From:   Randy Dunlap <rdunlap@infradead.org>
Message-ID: <725c0578-d816-f0c9-175a-7400c4a01887@infradead.org>
Date:   Wed, 10 Nov 2021 18:04:48 -0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.13.0
MIME-Version: 1.0
In-Reply-To: <a57524fe38123ea17a6741cdc282f6c84c30a009.1636593047.git.fthain@linux-m68k.org>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 11/10/21 5:10 PM, Finn Thain wrote:
> Enable the memtest feature and rearrange some code to prevent it from
> clobbering the initrd.
> 
> The CONFIG_BLK_DEV_INITRD symbol was conditional on !defined(CONFIG_SUN3).
> For simplicity, remove that test on the basis that m68k_ramdisk.size == 0
> on Sun 3.
> 
> The SLIME source code athttp://sammy.net/  shows that no BI_RAMDISK entry
> is ever passed to the kernel due to #ifdef 0 around the relevant code.
> 
> Cc: Mike Rapoport<rppt@kernel.org>
> Cc: Sam Creasey<sammy@sammy.net>
> Signed-off-by: Finn Thain<fthain@linux-m68k.org>
> ---
> Are there any other Linux bootloaders on Sun 3?
> ---
>   arch/m68k/Kconfig           |  1 +
>   arch/m68k/kernel/setup_mm.c | 15 ++++++---------
>   arch/m68k/mm/motorola.c     |  2 ++
>   3 files changed, 9 insertions(+), 9 deletions(-)

Hi Finn,

Please also update Documentation/admin-guide/kernel-parameters.txt:

	memtest=	[KNL,X86,ARM,PPC,RISCV] Enable memtest

to include "M68K".

thanks.
-- 
~Randy