Re: [tip: x86/urgent] x86/ioremap: Use is_ioremap_addr() in iounmap()
From: Max R
Date: Wed Aug 21 2024 - 06:18:48 EST
Apparently, x86-32 doesn't provide VMALLOC_START in asm/pgtable.h,
and after a quick glance it seems to be the only arch to do so...
Probably the correct solution is to add include asm/vmalloc.h to
include/linux/ioremap.h, considering it uses VMALLOC_START. Will
resubmit the v4 version of patch later today.
Best regards,
Max
On Tue, 20 Aug 2024 at 23:35, Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:
>
> On Fri, Aug 16 2024 at 09:43, tip-bot wrote:
> > The following commit has been merged into the x86/urgent branch of tip:
> >
> > Commit-ID: 7b02ad32d83c16abd4961d79f3154b734d1d5d9c
> > Gitweb: https://git.kernel.org/tip/7b02ad32d83c16abd4961d79f3154b734d1d5d9c
> > Author: Max Ramanouski <max8rr8@xxxxxxxxx>
> > AuthorDate: Thu, 15 Aug 2024 23:56:07 +03:00
> > Committer: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> > CommitterDate: Fri, 16 Aug 2024 11:33:33 +02:00
> >
> > x86/ioremap: Use is_ioremap_addr() in iounmap()
>
> This has been removed as it fails on a 32-bit defconfig:
>
> include/linux/ioremap.h: In function ‘is_ioremap_addr’:
> include/linux/ioremap.h:14:25: error: ‘VMALLOC_START’ undeclared (first use in this function); did you mean ‘KMALLOC_DMA’?
> 14 | #define IOREMAP_START VMALLOC_START
>
> Can you please have a look?
>
> Thanks,
>
> tglx