Re: [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t,1/4] lib/igt_kms: Add writeback support

From: Petri Latvala <petri.latvala@intel.com>
To: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t,1/4] lib/igt_kms: Add writeback support
Date: Tue, 5 Nov 2019 15:41:43 +0200	[thread overview]
Message-ID: <20191105134143.GL25209@platvala-desk.ger.corp.intel.com> (raw)
In-Reply-To: <20191105125046.h2j4xsyhlfp76gxb@ahiler-desk1.fi.intel.com>

On Tue, Nov 05, 2019 at 02:50:46PM +0200, Arkadiusz Hiler wrote:
> On Tue, Nov 05, 2019 at 12:41:40PM +0000, Patchwork wrote:
> > == Series Details ==
> > 
> > Series: series starting with [i-g-t,1/4] lib/igt_kms: Add writeback support
> > URL   : https://patchwork.freedesktop.org/series/68992/
> > State : warning
> > 
> > == Summary ==
> > 
> > Did not get list of undocumented tests for this run, something is wrong!
> > 
> > Other than that, pipeline status: FAILED.
> > 
> > see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/76331 for more details
> > 
> > == Logs ==
> > 
> > For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/76331
> 
> fails to compile for armhf and mips
> 
> FAILED: tests/59830eb@@kms_writeback@exe/kms_writeback.c.o 
> /usr/bin/arm-linux-gnueabihf-gcc -Itests/59830eb@@kms_writeback@exe -Itests -I../tests -I../include/drm-uapi -Ilib -I../lib -I../lib/stubs/syscalls -I. -I../ -I../lib/stubs/drm -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib/arm-linux-gnueabihf/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/libdrm -I/usr/include/valgrind -I/usr/include/alsa -I/usr/include/libdrm/nouveau -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -O2 -g -D_GNU_SOURCE -include config.h -D_FORTIFY_SOURCE=2 -Wbad-function-cast -Wdeclaration-after-statement -Wformat=2 -Wlogical-op -Wmissing-declarations -Wmissing-format-attribute -Wmissing-noreturn -Wmissing-prototypes -Wnested-externs -Wold-style-definition -Wpointer-arith -Wredundant-decls -Wshadow -Wstrict-prototypes -Wuninitialized -Wunused -Wno-clobbered -Wno-maybe-uninitialized -Wno-missing-field-initializers -Wno-pointer-arith -Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-unused-result -Werror=address -Werror=array-bounds -Werror=implicit -Werror=init-self -Werror=int-to-pointer-cast -Werror=main -Werror=missing-braces -Werror=nonnull -Werror=pointer-to-int-cast -Werror=return-type -Werror=sequence-point -Werror=trigraphs -Werror=write-strings -fno-builtin-malloc -fno-builtin-calloc -pthread  -MD -MQ 'tests/59830eb@@kms_writeback@exe/kms_writeback.c.o' -MF 'tests/59830eb@@kms_writeback@exe/kms_writeback.c.o.d' -o 'tests/59830eb@@kms_writeback@exe/kms_writeback.c.o' -c ../tests/kms_writeback.c
> In file included from ../lib/igt.h:39:0,
>                  from ../tests/kms_writeback.c:30:
> ../tests/kms_writeback.c: In function ‘do_writeback_test’:
> ../tests/kms_writeback.c:152:75: error: cast from pointer to integer of different size [-Werror=pointer-to-int-cast]
>   igt_output_set_prop_value(output, IGT_CONNECTOR_WRITEBACK_OUT_FENCE_PTR, (uint64_t)out_fence_ptr);
>                                                                            ^
> ../lib/igt_kms.h:650:29: note: in definition of macro ‘igt_output_set_prop_value’
>    (output)->values[prop] = (value); \

lib/ioctl_wrappers has from_user_pointer() and to_user_pointer() for that conversion.

-- 
Petri Latvala
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev