Re: [PATCH] mm: remove io-mapping

[Lorenzo Stoakes <lorenzo.stoakes@oracle.com>]

On Sat, Jul 26, 2025 at 10:39:52AM -0700, Andrew Morton wrote:
> On Sat, 26 Jul 2025 18:15:44 +0800 kernel test robot <lkp@intel.com> wrote:
>
> > kernel test robot noticed the following build errors:
> >
> > [auto build test ERROR on akpm-mm/mm-everything]
> >
> > url:    https://github.com/intel-lab-lkp/linux/commits/Lorenzo-Stoakes/mm-remove-io-mapping/20250724-225556
> > base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-everything
> > patch link:    https://lore.kernel.org/r/20250724145313.65920-1-lorenzo.stoakes%40oracle.com
> > patch subject: [PATCH] mm: remove io-mapping
> > config: x86_64-buildonly-randconfig-003-20250725 (https://download.01.org/0day-ci/archive/20250726/202507261811.4FJjX1bu-lkp@intel.com/config)
> > compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
> > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250726/202507261811.4FJjX1bu-lkp@intel.com/reproduce)
> >
> > If you fix the issue in a separate patch/commit (i.e. not just a new version of
> > the same patch/commit), kindly add following tags
> > | Reported-by: kernel test robot <lkp@intel.com>
> > | Closes: https://lore.kernel.org/oe-kbuild-all/202507261811.4FJjX1bu-lkp@intel.com/
> >
> > All errors (new ones prefixed by >>):
> >
> > >> drivers/net/ethernet/mellanox/mlx4/main.c:43:10: fatal error: 'linux/io-mapping.h' file not found
> >       43 | #include <linux/io-mapping.h>
> >          |          ^~~~~~~~~~~~~~~~~~~~
>
> I'm all confused.  include/linux/io-mapping.h is present in every tree
> I can find.  This report doesn't identify the HEAD commit which was
> used?

It's testing against this original version of the series that incorrectly
removed the header (long week :>). The v2 of this series does the right
thing (just remove the one dead code function, file it's in and related
Makefile, Kconfig stuff).

Anyway yes this test report is totally redundant!

Cheers, Lorenzo