Re: ✗ CI.KUnit: failure for drm/xe: Use fault injection infrastructure to find issues at probe time (rev2)

[Francois Dugast <francois.dugast@intel.com>]

On Fri, Sep 20, 2024 at 01:52:17PM +0000, Patchwork wrote:
> == Series Details ==
> 
> Series: drm/xe: Use fault injection infrastructure to find issues at probe time (rev2)
> URL   : https://patchwork.freedesktop.org/series/138654/
> State : failure
> 
> == Summary ==
> 
> + trap cleanup EXIT
> + /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
> ERROR:root:In file included from ../drivers/gpu/drm/xe/xe_guc_ads.c:8:
> ../include/linux/fault-inject.h:97:15: error: unknown type name ‘bool’
>    97 | static inline bool should_fail_alloc_page(gfp_t gfp_mask, unsigned int order)
>       |               ^~~~
> ../include/linux/fault-inject.h:97:43: error: unknown type name ‘gfp_t’
>    97 | static inline bool should_fail_alloc_page(gfp_t gfp_mask, unsigned int order)
>       |                                           ^~~~~
> ../include/linux/fault-inject.h:106:57: error: unknown type name ‘gfp_t’
>   106 | static inline int should_failslab(struct kmem_cache *s, gfp_t gfpflags)
>       |                                                         ^~~~~
> make[7]: *** [../scripts/Makefile.build:244: drivers/gpu/drm/xe/xe_guc_ads.o] Error 1

This build error occurs when CONFIG_FAULT_INJECTION=n due to missing inclusion
of linux/types.h in linux/fault-inject.h. It is solved with Jani's series:

https://lore.kernel.org/intel-xe/20240813121237.2382534-1-jani.nikula@intel.com/

Francois

> make[7]: *** Waiting for unfinished jobs....
> make[6]: *** [../scripts/Makefile.build:485: drivers/gpu/drm/xe] Error 2
> make[6]: *** Waiting for unfinished jobs....
> ../lib/iomap.c:156:5: warning: no previous prototype for ‘ioread64_lo_hi’ [-Wmissing-prototypes]
>   156 | u64 ioread64_lo_hi(const void __iomem *addr)
>       |     ^~~~~~~~~~~~~~
> ../lib/iomap.c:163:5: warning: no previous prototype for ‘ioread64_hi_lo’ [-Wmissing-prototypes]
>   163 | u64 ioread64_hi_lo(const void __iomem *addr)
>       |     ^~~~~~~~~~~~~~
> ../lib/iomap.c:170:5: warning: no previous prototype for ‘ioread64be_lo_hi’ [-Wmissing-prototypes]
>   170 | u64 ioread64be_lo_hi(const void __iomem *addr)
>       |     ^~~~~~~~~~~~~~~~
> ../lib/iomap.c:178:5: warning: no previous prototype for ‘ioread64be_hi_lo’ [-Wmissing-prototypes]
>   178 | u64 ioread64be_hi_lo(const void __iomem *addr)
>       |     ^~~~~~~~~~~~~~~~
> ../lib/iomap.c:264:6: warning: no previous prototype for ‘iowrite64_lo_hi’ [-Wmissing-prototypes]
>   264 | void iowrite64_lo_hi(u64 val, void __iomem *addr)
>       |      ^~~~~~~~~~~~~~~
> ../lib/iomap.c:272:6: warning: no previous prototype for ‘iowrite64_hi_lo’ [-Wmissing-prototypes]
>   272 | void iowrite64_hi_lo(u64 val, void __iomem *addr)
>       |      ^~~~~~~~~~~~~~~
> ../lib/iomap.c:280:6: warning: no previous prototype for ‘iowrite64be_lo_hi’ [-Wmissing-prototypes]
>   280 | void iowrite64be_lo_hi(u64 val, void __iomem *addr)
>       |      ^~~~~~~~~~~~~~~~~
> ../lib/iomap.c:288:6: warning: no previous prototype for ‘iowrite64be_hi_lo’ [-Wmissing-prototypes]
>   288 | void iowrite64be_hi_lo(u64 val, void __iomem *addr)
>       |      ^~~~~~~~~~~~~~~~~
> make[5]: *** [../scripts/Makefile.build:485: drivers/gpu/drm] Error 2
> make[4]: *** [../scripts/Makefile.build:485: drivers/gpu] Error 2
> make[3]: *** [../scripts/Makefile.build:485: drivers] Error 2
> make[3]: *** Waiting for unfinished jobs....
> make[2]: *** [/kernel/Makefile:1926: .] Error 2
> make[1]: *** [/kernel/Makefile:224: __sub-make] Error 2
> make: *** [Makefile:224: __sub-make] Error 2
> 
> [13:51:49] Configuring KUnit Kernel ...
> Generating .config ...
> Populating config with:
> $ make ARCH=um O=.kunit olddefconfig
> [13:51:53] Building KUnit Kernel ...
> Populating config with:
> $ make ARCH=um O=.kunit olddefconfig
> Building with:
> $ make ARCH=um O=.kunit --jobs=48
> + cleanup
> ++ stat -c %u:%g /kernel
> + chown -R 1003:1003 /kernel
> 
>