Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
From:   Thomas Gleixner <tglx@linutronix.de>
To:     David Hildenbrand <david@redhat.com>,
        Adrian-Ken Rueegsegger <ken@codelabs.ch>,
        dave.hansen@linux.intel.com, osalvador@suse.de
Cc:     luto@kernel.org, peterz@infradead.org, mingo@redhat.com,
        bp@alien8.de, hpa@zytor.com, x86@kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] x86/mm: Fix marking of unused sub-pmd ranges
In-Reply-To: <8f2af450-1080-2dcd-9c85-6190e7e14f27@redhat.com>
References: <20220509090637.24152-1-ken@codelabs.ch>
 <20220509090637.24152-2-ken@codelabs.ch>
 <8f2af450-1080-2dcd-9c85-6190e7e14f27@redhat.com>
Date:   Thu, 12 May 2022 18:54:50 +0200
Message-ID: <87bkw2vgh1.ffs@tglx>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

On Thu, May 12 2022 at 11:04, David Hildenbrand wrote:
> On 09.05.22 11:06, Adrian-Ken Rueegsegger wrote:
>>  static void __meminit vmemmap_use_new_sub_pmd(unsigned long start, unsigned long end)
>>  {
>> +	const unsigned long page = ALIGN_DOWN(start, PMD_SIZE);
>> +
>>  	vmemmap_flush_unused_pmd();
>>  
>>  	/*
>> @@ -914,8 +916,7 @@ static void __meminit vmemmap_use_new_sub_pmd(unsigned long start, unsigned long
>>  	 * Mark with PAGE_UNUSED the unused parts of the new memmap range
>>  	 */
>>  	if (!IS_ALIGNED(start, PMD_SIZE))
>> -		memset((void *)start, PAGE_UNUSED,
>> -			start - ALIGN_DOWN(start, PMD_SIZE));
>> +		memset((void *)page, PAGE_UNUSED, start - page);
>>  
>>  	/*
>>  	 * We want to avoid memset(PAGE_UNUSED) when populating the vmemmap of
>
> As the x86 code was based on my s390x code, I assume that this was
> accidentally introduced in the x86 variant.
>
> We'd be marking the wrong range PAGE_UNUSED.
>
> Your fix looks correct to me:
>
> Reviewed-by: David Hildenbrand <david@redhat.com>
>
> Do we want to cc stable?

Yes, we'll add it when picking it up.

I really have to ask why this duplicated code exists in the first
place. There is zero architecture specific code neither in the s390 nor
in the x86 version.

The x86 version is just copy & pasta & fatfinger, if I'm not missing
something here.

Thanks,

        tglx