Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Buildroot 2023.02 kernel 5.10.174 #5256

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

shenki
Copy link
Member

@shenki shenki commented Mar 14, 2023

No description provided.

@op-jenkins
Copy link
Contributor

Can one of the admins verify this patch?

@shenki shenki force-pushed the buildroot-2023.02-kernel-5.10.174 branch 3 times, most recently from a7a761c to d0f66e0 Compare March 15, 2023 03:47
Loadkeys is a copy of kbd to reduce the filesystem size. It doesn't
build with GCC 11, so update to the latest kdb version.

Signed-off-by: Joel Stanley <[email protected]>
Building for power8 is now the default for ppc64le.

GCC 8 is no longer available so drop the option to a void an error with
legacy config. GCC 10 will now be used.

Signed-off-by: Joel Stanley <[email protected]>
Update to the latest v5.10 kernel for security and bug fixes.

Signed-off-by: Joel Stanley <[email protected]>
@shenki shenki force-pushed the buildroot-2023.02-kernel-5.10.174 branch from d0f66e0 to a0b660a Compare March 15, 2023 03:49
@maheshsal
Copy link

Can the kernel version be bumped to v5.10.181 ?

@runlevel5
Copy link
Contributor

runlevel5 commented Aug 17, 2023

Looks good to me. Perhaps we could bump to v5.10.191?

Ref: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/diff/?id=v5.10.191&id2=v5.10.174&dt=2

@runlevel5
Copy link
Contributor

runlevel5 commented Aug 19, 2023

I tried to build this on my Fedora 38 ppc64le and got following error:

>>> hcode hw062823a.opmst Building
CONFIG_FILE=/home/tle/Work/op-build/openpower/configs/hcode/ LD_LIBRARY_PATH=/home/tle/Work/op-build/output/host/usr/lib OPENPOWER_BUILD=1 CROSS_COMPILER_PATH=/home/tle/Work/op-build/output/host/powerpc64le-buildroot-linux-gnu/sysroot/ppe42-toolchain/linux PPE_TOOL_PATH=/home/tle/Work/op-build/output/host/powerpc64le-buildroot-linux-gnu/sysroot/ppe42-toolchain/linux PPE_PREFIX=/home/tle/Work/op-build/output/host/powerpc64le-buildroot-linux-gnu/sysroot/ppe42-toolchain/linux/bin/powerpc-eabi- RINGFILEPATH=/home/tle/Work/op-build/output/host/powerpc64le-buildroot-linux-gnu/sysroot/hostboot_binaries __EKB_PREFIX= CONFIG_IONV_FILE_LOCATION=/home/tle/Work/op-build/output/host/powerpc64le-buildroot-linux-gnu/sysroot/hostboot_binaries/"ionv.bin" CONFIG_INCLUDE_IONV=y bash -c 'cd /home/tle/Work/op-build/output/build/hcode-hw062823a.opmst && source ./env.bash && /usr/bin/make  '
Searching for Gerrit Host...
make[1]: Entering directory '/home/tle/Work/op-build/output/build/hcode-hw062823a.opmst'
make[3]: '../../import/../output/bin/ppetracepp.exe' is up to date.
make[3]: '../../import/../output/bin/ppetracepp.exe' is up to date.
make[3]: '../../import/../output/bin/cmeImgEdit.exe' is up to date.
make[3]: '../../import/../output/bin/stop_gpeImgEdit.exe' is up to date.
make[3]: '../../import/../output/bin/pstate_gpeImgEdit.exe' is up to date.
    DEP        p9_scan_compression.dep
    C++        p9_scan_compression.o
../../import/chips/p9/utils/imageProcs//p9_scan_compression.C: In function ‘int rs4_embed_cmsk(CompressedScanData**, CompressedScanData*)’:
../../import/chips/p9/utils/imageProcs//p9_scan_compression.C:935:26: error: pointer ‘embeddedAddr’ may be used after ‘void* realloc(void*, size_t)’ [-Werror=use-after-free]
  935 |     memmove(embeddedAddr + embeddedSize,
      |             ~~~~~~~~~~~~~^~~~~~~~~~~~~~
../../import/chips/p9/utils/imageProcs//p9_scan_compression.C:927:43: note: call to ‘void* realloc(void*, size_t)’ here
  927 |     *io_rs4 = (CompressedScanData*)realloc(*io_rs4, totalSize);
      |                                    ~~~~~~~^~~~~~~~~~~~~~~~~~~~
../../import/chips/p9/utils/imageProcs//p9_scan_compression.C:940:11: error: pointer ‘embeddedAddr’ may be used after ‘void* realloc(void*, size_t)’ [-Werror=use-after-free]
  940 |     memcpy(embeddedAddr,
      |     ~~~~~~^~~~~~~~~~~~~~
  941 |            i_rs4Cmsk,
      |            ~~~~~~~~~~
  942 |            embeddedSize);
      |            ~~~~~~~~~~~~~
../../import/chips/p9/utils/imageProcs//p9_scan_compression.C:927:43: note: call to ‘void* realloc(void*, size_t)’ here
  927 |     *io_rs4 = (CompressedScanData*)realloc(*io_rs4, totalSize);
      |                                    ~~~~~~~^~~~~~~~~~~~~~~~~~~~
../../import/chips/p9/utils/imageProcs//p9_scan_compression.C:935:12: error: pointer ‘embeddedAddr’ may be used after ‘void* realloc(void*, size_t)’ [-Werror=use-after-free]
  935 |     memmove(embeddedAddr + embeddedSize,
      |     ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  936 |             embeddedAddr,
      |             ~~~~~~~~~~~~~
  937 |             be16toh((*io_rs4)->iv_size) - sizeof(CompressedScanData));
      |             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../import/chips/p9/utils/imageProcs//p9_scan_compression.C:927:43: note: call to ‘void* realloc(void*, size_t)’ here
  927 |     *io_rs4 = (CompressedScanData*)realloc(*io_rs4, totalSize);
      |                                    ~~~~~~~^~~~~~~~~~~~~~~~~~~~
cc1plus: all warnings being treated as errors
make[3]: *** [../../import/chips/p9/utils/imageProcs/p9_scan_compression.mk:32: ../../import/../output/obj/p9_scan_compression/p9_scan_compression.o] Error 1
make[2]: *** [common.dir/order.final.mk:41: _BUILD/MODULE_TARGETS] Error 2
make[1]: *** [Makefile:37: all] Error 2
make[1]: Leaving directory '/home/tle/Work/op-build/output/build/hcode-hw062823a.opmst'
make: *** [package/pkg-generic.mk:251: /home/tle/Work/op-build/output/build/hcode-hw062823a.opmst/.stamp_built] Error 2
make: Leaving directory '/home/tle/Work/op-build/buildroot'

Both GCC 8.4.0 and 9.4.0 produce same error.

I've reported this issue with hcode side open-power/hcode#4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants