Re: [Intel-gfx] [PATCH i-g-t v4 0/4] Use C11 atomics
On 25/06/2019 11:54, Ser, Simon wrote: > On Mon, 2019-06-24 at 17:22 +0100, Guillaume Tucker wrote: >> This series replaces calls to the __sync_* functions with the more >> recent atomic_* ones defined in stdatomic.h in gem_create and >> sw_sync. It also adds dependency on libatomic when required, that is >> to say when the CPU architecture doesn't provide native support for >> some atomic operations. This makes the tests more portable, in >> particular for 32-bit MIPS which doesn't support 64-bit atomics. >> > Thanks for this series! Pushed: > > To gitlab.freedesktop.org:drm/igt-gpu-tools.git >a6f5cc854efb..5eafa33dbdb1 master -> master Awesome! The patch I sent separately to enable MIPS in Gitlab CI should now be ready to be pushed as well, I'll follow up on the other thread. Guillaume ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH i-g-t v4 0/4] Use C11 atomics
On Mon, 2019-06-24 at 17:22 +0100, Guillaume Tucker wrote: > This series replaces calls to the __sync_* functions with the more > recent atomic_* ones defined in stdatomic.h in gem_create and > sw_sync. It also adds dependency on libatomic when required, that is > to say when the CPU architecture doesn't provide native support for > some atomic operations. This makes the tests more portable, in > particular for 32-bit MIPS which doesn't support 64-bit atomics. Thanks for this series! Pushed: To gitlab.freedesktop.org:drm/igt-gpu-tools.git a6f5cc854efb..5eafa33dbdb1 master -> master > v2: > - add linker test to only add dependency on libatomic when needed > - only add libatomic dependency to gem_create and sw_sync > - use stdatomic.h and _Atomic type modifier > - explicitly require libatomic in all Docker images > > v3: > - use sub-arch libatomic1 in Debian docker images > - use null_dep in meson.build > > v4: > - rebase with changes in Dockerfile.debian-minimal > > Guillaume Tucker (4): > meson: add libatomic dependency > gitlab-ci: add libatomic to docker images > i915/gem_create: use atomic_* instead of __sync_* > tests/sw_sync: use atomic_* instead of __sync_* > > Dockerfile.debian-arm64 | 1 + > Dockerfile.debian-armhf | 1 + > Dockerfile.debian-minimal | 1 + > Dockerfile.fedora | 2 +- > meson.build | 14 ++ > tests/Makefile.am | 3 ++- > tests/i915/gem_create.c | 16 > tests/meson.build | 17 +++-- > tests/sw_sync.c | 12 ++-- > 9 files changed, 53 insertions(+), 14 deletions(-) > > -- > 2.20.1 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[Intel-gfx] [PATCH i-g-t v4 0/4] Use C11 atomics
This series replaces calls to the __sync_* functions with the more recent atomic_* ones defined in stdatomic.h in gem_create and sw_sync. It also adds dependency on libatomic when required, that is to say when the CPU architecture doesn't provide native support for some atomic operations. This makes the tests more portable, in particular for 32-bit MIPS which doesn't support 64-bit atomics. v2: - add linker test to only add dependency on libatomic when needed - only add libatomic dependency to gem_create and sw_sync - use stdatomic.h and _Atomic type modifier - explicitly require libatomic in all Docker images v3: - use sub-arch libatomic1 in Debian docker images - use null_dep in meson.build v4: - rebase with changes in Dockerfile.debian-minimal Guillaume Tucker (4): meson: add libatomic dependency gitlab-ci: add libatomic to docker images i915/gem_create: use atomic_* instead of __sync_* tests/sw_sync: use atomic_* instead of __sync_* Dockerfile.debian-arm64 | 1 + Dockerfile.debian-armhf | 1 + Dockerfile.debian-minimal | 1 + Dockerfile.fedora | 2 +- meson.build | 14 ++ tests/Makefile.am | 3 ++- tests/i915/gem_create.c | 16 tests/meson.build | 17 +++-- tests/sw_sync.c | 12 ++-- 9 files changed, 53 insertions(+), 14 deletions(-) -- 2.20.1 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx