On Tue, Feb 18, 2025 at 03:49:54PM +0530, Anshuman Khandual wrote:

From: Ryan Roberts <ryan.roberts@arm.com>

ioremap_prot() currently accepts pgprot_val parameter as an unsigned long,
thus implicitly assuming that pgprot_val and pgprot_t could never be bigger
than unsigned long. But this assumption soon will not be true on arm64 when
using D128 pgtables. In 128 bit page table configuration, unsigned long is
64 bit, but pgprot_t is 128 bit.

Passing platform abstracted pgprot_t argument is better as compared to size
based data types. Let's change the parameter to directly pass pgprot_t like
another similar helper generic_ioremap_prot().

Without this change in place, D128 configuration does not work on arm64 as
the top 64 bits gets silently stripped when passing the protection value to
this function.

Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-snps-arc@lists.infradead.org
Cc: linux-riscv@lists.infradead.org
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-parisc@vger.kernel.org
Cc: linux-csky@vger.kernel.org
Cc: linux-mips@vger.kernel.org
Cc: linux-s390@vger.kernel.org
Cc: linux-arch@vger.kernel.org
Cc: loongarch@lists.linux.dev
Cc: linux-sh@vger.kernel.org
Cc: linux-mm@kvack.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Ryan Roberts <ryan.roberts@arm.com>
Co-developed-by: Anshuman Khandual <redacted>
Signed-off-by: Anshuman Khandual <redacted>

For arm64:

Acked-by: Catalin Marinas <catalin.marinas@arm.com>