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

From: Veronika Kabatova <vkabatov@redhat.com>
To: Anshuman Khandual <anshuman.khandual@arm.com>,
	Will Deacon <will@kernel.org>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	"Catalin Marinas" <catalin.marinas@arm.com>,
	"Ard Biesheuvel" <ardb@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"James Morse" <james.morse@arm.com>,
	"Robin Murphy" <robin.murphy@arm.com>,
	"Jérôme Glisse" <jglisse@redhat.com>,
	"Dan Williams" <dan.j.williams@intel.com>,
	"David Hildenbrand" <david@redhat.com>,
	"Mike Rapoport" <rppt@linux.ibm.com>
Subject: Re: [PATCH V3 0/2] arm64/mm: Fix pfn_valid() for ZONE_DEVICE based memory
Date: Fri, 5 Mar 2021 13:16:28 -0500 (EST)	[thread overview]
Message-ID: <1890909388.30244858.1614968188922.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <2001895867.30197818.1614947320141.JavaMail.zimbra@redhat.com>

----- Original Message -----
> From: "Veronika Kabatova" <vkabatov@redhat.com>
> To: "Anshuman Khandual" <anshuman.khandual@arm.com>
> Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, "Catalin Marinas"
> <catalin.marinas@arm.com>, "Will Deacon" <will@kernel.org>, "Ard Biesheuvel" <ardb@kernel.org>, "Mark Rutland"
> <mark.rutland@arm.com>, "James Morse" <james.morse@arm.com>, "Robin Murphy" <robin.murphy@arm.com>, "Jérôme Glisse"
> <jglisse@redhat.com>, "Dan Williams" <dan.j.williams@intel.com>, "David Hildenbrand" <david@redhat.com>, "Mike
> Rapoport" <rppt@linux.ibm.com>
> 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@arm.com>
> > To: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
> > linux-mm@kvack.org
> > Cc: "Catalin Marinas" <catalin.marinas@arm.com>, "Will Deacon"
> > <will@kernel.org>, "Ard Biesheuvel" <ardb@kernel.org>,
> > "Mark Rutland" <mark.rutland@arm.com>, "James Morse" <james.morse@arm.com>,
> > "Robin Murphy" <robin.murphy@arm.com>,
> > "Jérôme Glisse" <jglisse@redhat.com>, "Dan Williams"
> > <dan.j.williams@intel.com>, "David Hildenbrand"
> > <david@redhat.com>, "Mike Rapoport" <rppt@linux.ibm.com>, "Veronika
> > Kabatova" <vkabatov@redhat.com>
> > 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@arm.com>
> > > Cc: Will Deacon <will@kernel.org>
> > > Cc: Ard Biesheuvel <ardb@kernel.org>
> > > Cc: Mark Rutland <mark.rutland@arm.com>
> > > Cc: James Morse <james.morse@arm.com>
> > > Cc: Robin Murphy <robin.murphy@arm.com>
> > > Cc: Jérôme Glisse <jglisse@redhat.com>
> > > Cc: Dan Williams <dan.j.williams@intel.com>
> > > Cc: David Hildenbrand <david@redhat.com>
> > > Cc: Mike Rapoport <rppt@linux.ibm.com>
> > > Cc: Veronika Kabatova <vkabatov@redhat.com>
> > > Cc: linux-arm-kernel@lists.infradead.org
> > > Cc: linux-mm@kvack.org
> > > Cc: linux-kernel@vger.kernel.org
> > > 
> > > 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@redhat.com/
> > 
> > - Anshuman
> > 
> > 
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel