Re: [tip: x86/urgent] x86/ioremap: Use is_ioremap_addr() in iounmap()

[Max R]

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