Re: ✗ GitLab.Pipeline: warning for lib/gpgpu: add shader support (rev2)

[Andrzej Hajda <andrzej.hajda@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 3187 bytes --]



On 15.05.2024 12:52, Patchwork wrote:
> == Series Details ==
>
> Series: lib/gpgpu: add shader support (rev2)
> URL   :https://patchwork.freedesktop.org/series/133020/
> State : warning
>
> == Summary ==
>
> Pipeline status: FAILED.
>
> seehttps://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/1178181  for the overview.
>
> build:tests-fedora-clang has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/58765598):

Apparently clang still (?) does not support raw string literals, shall I 
switch to old ugly c literals:

+ emit_iga64_code(shdr, eot, "\n\ +(W) mov (8|M0) r112.0<1>:ud 
r0.0<8;8,1>:ud \n\+#if GEN_VER < 1250\n\+(W) send.ts (16|M0) null r112 
null 0x10000000 0x02000010 {EOT,@1} // wr:1+0, rd:0; end of 
thread\n\+#else\n\+(W) send.gtwy (8|M0) null r112 src1_null 0 0x02000000 
{EOT}\n\+#endif\n\+ "); or since it is warning can we ignore it? :)

>    #define ARRAY_SIZE(arr) (sizeof(arr)/sizeof(arr[0]))
>                                   ^~~~~
>    ../lib/gpgpu_shader.c:312:3: error: expected ';' after expression
>            )ASM", 2, y_offset, 3, value, value, value, value);
>             ^
>             ;
>    ../lib/gpgpu_shader.c:312:6: warning: missing terminating '"' character [-Winvalid-pp-token]
>            )ASM", 2, y_offset, 3, value, value, value, value);
>                ^
>    ../lib/gpgpu_shader.c:312:3: error: use of undeclared identifier 'ASM'
>            )ASM", 2, y_offset, 3, value, value, value, value);
>             ^
>    6 warnings and 12 errors generated.
>    ninja: build stopped: subcommand failed.
>    section_end:1715769978:step_script
>    section_start:1715769978:cleanup_file_variables
>    Cleaning up project directory and file based variables
>    section_end:1715769979:cleanup_file_variables
>    ERROR: Job failed: exit code 1
>    
>
> build:tests-fedora-oldest-meson has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/58765596):
>    Checking for function "outb" : YES
>    Checking if "cpuid.h" links: YES
>    Header <unistd.h> has symbol "gettid": YES
>    Checking whether type "struct sysinfo" has member "totalram" : YES
>    Checking for function "memfd_create" : YES
>    Configuring config.h using configuration
>    Program python3 found: YES (/usr/bin/python3)
>    lib/meson.build:236: WARNING: Identifier 'in' will become a reserved keyword in a future release. Please rename it.
>    
>    lib/meson.build:236:9: ERROR:  Expecting eol got id.
>        if f in iga64_assembly_sources
>             ^

Grepping internet suggests that meson should be updated :) I have not 
spotted better solution.
I guess it just matter to replace line in meson.build:
meson_version : '>=0.47.2'
with sth bigger.

Regards
Andrzej

>    
>    A full log can be found at /builds/gfx-ci/igt-ci-tags/build/meson-logs/meson-log.txt
>    section_end:1715769970:step_script
>    section_start:1715769970:cleanup_file_variables
>    Cleaning up project directory and file based variables
>    section_end:1715769971:cleanup_file_variables
>    ERROR: Job failed: exit code 1
>
> == Logs ==
>
> For more details see:https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/1178181

[-- Attachment #2: Type: text/html, Size: 6175 bytes --]