Re: [PATCH V3 0/2] arm64/mm: Fix pfn_valid() for ZONE_DEVICE based memory

[Veronika Kabatova]

----- Original Message -----
> From: "Veronika Kabatova" <vkabatov@xxxxxxxxxx>
> To: "Anshuman Khandual" <anshuman.khandual@xxxxxxx>
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, "Catalin Marinas"
> <catalin.marinas@xxxxxxx>, "Will Deacon" <will@xxxxxxxxxx>, "Ard Biesheuvel" <ardb@xxxxxxxxxx>, "Mark Rutland"
> <mark.rutland@xxxxxxx>, "James Morse" <james.morse@xxxxxxx>, "Robin Murphy" <robin.murphy@xxxxxxx>, "Jérôme Glisse"
> <jglisse@xxxxxxxxxx>, "Dan Williams" <dan.j.williams@xxxxxxxxx>, "David Hildenbrand" <david@xxxxxxxxxx>, "Mike
> Rapoport" <rppt@xxxxxxxxxxxxx>
> Sent: Friday, March 5, 2021 1:28:40 PM
> Subject: Re: [PATCH V3 0/2] arm64/mm: Fix pfn_valid() for ZONE_DEVICE based memory
> 
> 
> 
> ----- Original Message -----
> > From: "Anshuman Khandual" <anshuman.khandual@xxxxxxx>
> > To: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx,
> > linux-mm@xxxxxxxxx
> > Cc: "Catalin Marinas" <catalin.marinas@xxxxxxx>, "Will Deacon"
> > <will@xxxxxxxxxx>, "Ard Biesheuvel" <ardb@xxxxxxxxxx>,
> > "Mark Rutland" <mark.rutland@xxxxxxx>, "James Morse" <james.morse@xxxxxxx>,
> > "Robin Murphy" <robin.murphy@xxxxxxx>,
> > "Jérôme Glisse" <jglisse@xxxxxxxxxx>, "Dan Williams"
> > <dan.j.williams@xxxxxxxxx>, "David Hildenbrand"
> > <david@xxxxxxxxxx>, "Mike Rapoport" <rppt@xxxxxxxxxxxxx>, "Veronika
> > Kabatova" <vkabatov@xxxxxxxxxx>
> > Sent: Friday, March 5, 2021 6:38:14 AM
> > Subject: Re: [PATCH V3 0/2] arm64/mm: Fix pfn_valid() for ZONE_DEVICE based
> > memory
> > 
> > 
> > On 3/5/21 10:54 AM, Anshuman Khandual wrote:
> > > This series fixes pfn_valid() for ZONE_DEVICE based memory and also
> > > improves
> > > its performance for normal hotplug memory. While here, it also
> > > reorganizes
> > > pfn_valid() on CONFIG_SPARSEMEM. This series is based on v5.12-rc1.
> > > 
> > > Cc: Catalin Marinas <catalin.marinas@xxxxxxx>
> > > Cc: Will Deacon <will@xxxxxxxxxx>
> > > Cc: Ard Biesheuvel <ardb@xxxxxxxxxx>
> > > Cc: Mark Rutland <mark.rutland@xxxxxxx>
> > > Cc: James Morse <james.morse@xxxxxxx>
> > > Cc: Robin Murphy <robin.murphy@xxxxxxx>
> > > Cc: Jérôme Glisse <jglisse@xxxxxxxxxx>
> > > Cc: Dan Williams <dan.j.williams@xxxxxxxxx>
> > > Cc: David Hildenbrand <david@xxxxxxxxxx>
> > > Cc: Mike Rapoport <rppt@xxxxxxxxxxxxx>
> > > Cc: Veronika Kabatova <vkabatov@xxxxxxxxxx>
> > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> > > Cc: linux-mm@xxxxxxxxx
> > > Cc: linux-kernel@xxxxxxxxxxxxxxx
> > > 
> > > Changes in V3:
> > > 
> > > - Validate the pfn before fetching mem_section with __pfn_to_section() in
> > > [PATCH 2/2]
> > 
> > Hello Veronica,
> > 
> > Could you please help recreate the earlier failure [1] but with this
> > series applies on v5.12-rc1. Thank you.
> > 
> 
> Hello Anshuman,
> 
> the machine in question is currently loaned to a developer. I'll reach
> out to them and will let you know once I have any results.
> 

Hi,

I'm happy to report the kernel boots with these new patches. I used the
5.12.0-rc1 kernel (commit 280d542f6ffac0) as a base. The full console log
from the boot process is available at

https://gitlab.com/-/snippets/2086833

in case you want to take a look. Note that there are some call traces
starting around line 3220, but they are NOT introduced by these two
patches and are also present with the base mainline kernel.


Veronika

> 
> Veronika
> 
> > [1]
> > https://lore.kernel.org/linux-arm-kernel/cki.8D1CB60FEC.K6NJMEFQPV@xxxxxxxxxx/
> > 
> > - Anshuman
> > 
> > 
>