Hi Zhangjin,
On Wed, Jul 19, 2023 at 02:49:12PM +0800, Zhangjin Wu wrote:
Hi, Willy
Hi Zhangjin,
On Wed, Jul 19, 2023 at 07:56:37AM +0800, Zhangjin Wu wrote:
It made me recalled I have at last disabled (not enabled for tinyconfig) the following options:
CONFIG_ALTIVEC CONFIG_VSX --> This option enables kernel support for the Vector Scaler extensionsOr we can disable the vsx instructions explicitly:
-mno-vsxBoth of them work well, but I prefer -mno-vsx for to get a faster build, what about you?
+CFLAGS_powerpc64 = -m64 -mbig-endian -Wl,-EB,-melf64ppc -mno-vsx +CFLAGS_powerpc64le = -m64 -mlittle-endian -Wl,-EL,-melf64ppc -mno-vsxSo, this patch itself is wrong, let's drop it from the next revision.
Better explicitly disable it in the CFLAGS (2nd option) if we want to make sure we don't want to rely on this, at least for portability purposes.
Ok, thanks, have updated CFLAGS in these two patches locally:
[PATCH v1 7/8] selftests/nolibc: add test support for powerpc64le [PATCH v1 8/8] selftests/nolibc: add test support for powerpc64what about the other ones? I'm ready to send v2 ;-)
I have not had the time to review them yet. Please just don't send another series yet, that just adds more noise and makes it hard to distinguish all of them. I hope to be able to check these and hopefully the tinyconfig series by the week-end.
Thanks, Willy