这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@Ansuel
Copy link
Owner

@Ansuel Ansuel commented Jan 11, 2023

Thanks for your contribution to OpenWrt!

To help keep the codebase consistent and readable,
and to help people review your contribution,
we ask you to follow the rules you find in the wiki at this link
https://openwrt.org/submitting-patches

Please remove this message before posting the pull request.

Package external toolchain after correct build.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Add option to configure container to use for build and build-tools test.
By default the tools and buildbot container is used if no option is
provided.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Build and push container with external toolchain embedded in the
container image.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Use alpine container as base image for everything.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Add support to use container included external toolchain and skip
redownloading external sdk for each test.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Backport patch fixing compile on musl libc to improve support for alpine
image.

Ref: https://sourceware.org/bugzilla/show_bug.cgi?id=29477
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Alone7z fails with error

cc      -O2 -c -Wall -Werror -Wextra  -DNDEBUG -D_REENTRANT -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -fPIC  -o _o/CpuArch.o ../../../../C/CpuArch.c
cc      -O2 -c -Wall -Werror -Wextra  -DNDEBUG -D_REENTRANT -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -fPIC  -o _o/Delta.o ../../../../C/Delta.c
cc      -O2 -c -Wall -Werror -Wextra  -DNDEBUG -D_REENTRANT -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -fPIC  -o _o/LzFind.o ../../../../C/LzFind.c
cc      -O2 -c -Wall -Werror -Wextra  -DNDEBUG -D_REENTRANT -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -fPIC  -o _o/Lzma2Dec.o ../../../../C/Lzma2Dec.c
cc      -O2 -c -Wall -Werror -Wextra  -DNDEBUG -D_REENTRANT -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -fPIC  -o _o/Lzma2DecMt.o ../../../../C/Lzma2DecMt.c
In file included from ../../../../C/MtDec.h:10,
                 from ../../../../C/Lzma2DecMt.c:30:
../../../../C/Threads.h:80:9: error: unknown type name 'cpu_set_t'
   80 | typedef cpu_set_t CCpuSet;
      |         ^~~~~~~~~
make[3]: *** [../../7zip_gcc.mak:1118: _o/Lzma2DecMt.o] Error 1
make[3]: Leaving directory '/__w/openwrt/openwrt/openwrt/build_dir/host/7z-22.01/CPP/7zip/Bundles/Alone7z'

It seems -D_GNU_SOURCE is needed for cpu_set_t type to correctly work
with musl libc.
Also we need to use a portable variant of pthread_setaffinity that can
work with both glibc and musl libc.

Add the 2 required patch to make 7zip correctly compile with musl libc.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Add concurrency limits for pull request test so that on pull request
refresh old jobs are cancelled.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
@Ansuel Ansuel force-pushed the master branch 9 times, most recently from bf418c1 to d86de67 Compare May 26, 2024 17:18
@Ansuel Ansuel force-pushed the master branch 7 times, most recently from 8c6fb7f to 2fd3c99 Compare June 13, 2024 16:26
@Ansuel Ansuel force-pushed the master branch 6 times, most recently from 118b46f to a575158 Compare June 16, 2024 16:15
@Ansuel Ansuel force-pushed the master branch 4 times, most recently from 12b0c50 to e5cf548 Compare July 4, 2024 14:06
Ansuel pushed a commit that referenced this pull request Jul 5, 2024
@Ansuel Ansuel force-pushed the master branch 2 times, most recently from e73fda1 to b3e7969 Compare August 1, 2024 12:27
Ansuel pushed a commit that referenced this pull request May 25, 2025
Fixes the following warning:
[  180.314652] NOHZ tick-stop error: local softirq work is pending, handler #8!!!

Signed-off-by: Felix Fietkau <nbd@nbd.name>
Ansuel pushed a commit that referenced this pull request Sep 7, 2025
Fixes the following warning:
[  180.314652] NOHZ tick-stop error: local softirq work is pending, handler #8!!!

Signed-off-by: Felix Fietkau <nbd@nbd.name>
(cherry picked from commit 610dd87)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants