Re: ✗ CI.KUnit: failure for Fix xe_force_wake_get() failure handling (rev6)

[Ghimiray, Himal Prasad]

On 30-09-2024 11:33, Patchwork wrote:
> == Series Details ==
> 
> Series: Fix xe_force_wake_get() failure handling (rev6)
> URL   : https://patchwork.freedesktop.org/series/137982/
> State : failure
> 
> == Summary ==
> 
> + trap cleanup EXIT
> + /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
> ERROR:root:../drivers/gpu/drm/xe/xe_gt_idle.c:56:27: error: redefinition of ‘str_up_down’
>     56 | static inline const char *str_up_down(bool v)

Author: Michal Wajdeczko <michal.wajdeczko at intel.com>
Date:   10 weeks ago

     lib/string_choices: Add str_up_down() helper

     Add str_up_down() helper to return "up" or "down" string literal.

     Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
     Link: 
https://lore.kernel.org/r/20240725101841.574-1-michal.wajdeczko@intel.com
     Signed-off-by: Kees Cook <kees at kernel.org>

Is merged to drm-tip needs to be backported to drm-xe-next, along with 
removing local copy of str_up_down from xe_gt_idle.c.

Failure is not related https://patchwork.freedesktop.org/series/137982/.

>        |                           ^~~~~~~~~~~
> In file included from ../include/linux/string_helpers.h:7,
>                   from ../drivers/gpu/drm/xe/xe_assert.h:9,
>                   from ../drivers/gpu/drm/xe/xe_force_wake.h:9,
>                   from ../drivers/gpu/drm/xe/xe_gt_idle.c:8:
> ../include/linux/string_choices.h:62:27: note: previous definition of ‘str_up_down’ with type ‘const char *(bool)’ {aka ‘const char *(_Bool)’}
>     62 | static inline const char *str_up_down(bool v)
>        |                           ^~~~~~~~~~~
> make[7]: *** [../scripts/Makefile.build:229: drivers/gpu/drm/xe/xe_gt_idle.o] Error 1
> make[7]: *** Waiting for unfinished jobs....
> make[6]: *** [../scripts/Makefile.build:478: 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:478: drivers/gpu/drm] Error 2
> make[4]: *** [../scripts/Makefile.build:478: drivers/gpu] Error 2
> make[3]: *** [../scripts/Makefile.build:478: drivers] Error 2
> make[3]: *** Waiting for unfinished jobs....
> make[2]: *** [/kernel/Makefile:1936: .] Error 2
> make[1]: *** [/kernel/Makefile:224: __sub-make] Error 2
> make: *** [Makefile:224: __sub-make] Error 2
> 
> [06:03:18] Configuring KUnit Kernel ...
> Generating .config ...
> Populating config with:
> $ make ARCH=um O=.kunit olddefconfig
> [06:03:23] Building KUnit Kernel ...
> Populating config with:
> $ make ARCH=um O=.kunit olddefconfig
> Building with:
> $ make all compile_commands.json ARCH=um O=.kunit --jobs=48
> + cleanup
> ++ stat -c %u:%g /kernel
> + chown -R 1003:1003 /kernel
> 
>