Hi, Willy, Hi Thomas
I'm so happy to share with you, we have solved all of the left found issues, include the ones about ppc and the missing poweroff options for the tinyconfig series, will renew both series ;-)
Further compared the preprocessed files, found the root cause is the new compiler using 'no_stack_protector' instead of '__optimize__("-fno-stack-protector")'. And the attribute 'no_stack_protector' breaks our "omit-frame-pointer" like the failure with '-O0' we fixed before.
I checked some of the other architectures, they didn't have the same issue, but test shows the 'no_stack_protector' attribute does have such compability issue here.
I learned the commit message of tools/include/nolibc/compiler.h, seems __optimize__("-fno-stack-protector") is enough for all of the nolibc supported architectures? is it ok for us to simply give up 'no_stack_protector' eventully? otherwise, we should manually disable 'no_stack_protector' for ppc32:
#define __no_stack_protector __attribute__((__optimize__("-fno-stack-protector")))
Hello, any suggestion here? ;-)
BTW, we have found another -O0 + "omit-frame-pointer" regression with gcc 13.1.0 on all architectures ("omit-frame-pointer" will be completely ignored when -O0 passed whatever other internal optimize attribute like 'Os' we used together in commit '4dc4401991fc' for gcc < 11.1.0), Yuan is doing more tests to confirm it, will open another discussion for it.
Benefit from the __no_stack_protetor macro, we have no need to input __optimize__("-fno-stack-protector") for every new architecture, it may be the time to only use __optimize__("-fno-stack-protector"), am I missing something?
out that your willingness to work around some of the oddities met on this one have resulted in a lot of drift away from the "keep it simple" principle and significantly delayed the possibilities to integrate the rest of your work. It would be much saner to only focus on ppc64/ppc64le and figure later how to make ppc32 work (or decide not to support it if it's not worth the effort in the end).
Another test shows, the run target of ppc64le with the big endian gcc 13.1.0 toolchain has test startup failure (although we have verified run-user before, run-user ok! and run with local powerpc64le toolchain ok too!), I'm debugging this now, hope it not cost too much.
Just solved this, when -mabi=elfv2 supported by the toolchain (gcc 13.1.0 has this option), kernel will compile with this ABI, to align with kernel, our user space application should use -mabi=elfv2 too if toolchain support it, otherwise, test will simply fails at startup, this solves it:
$(call cc-option,-mabi=elfv2)
Another good news is, Yuan has found a working defconfig for ppc32 which has poweroff support and he is helping me narrowing down the exact options, this will eventually make us happy ;-)
We eventually find the missing poweroff option for ppc32, will update the patches with them.
we also found such options for all of the nolibc supported architectures, will add them in their tinyconfig support patches.
Thanks, Zhangjin
Thanks & Best regards, Zhangjin
Regards, willy