Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-22 Thread Andrei Borzenkov
Отправлено с iPhone > 22 сент. 2015 г., в 17:03, Mark H Weaver написал(а): > > Andrei Borzenkov writes: >> Did you also test cross-compile > > No. My Yeeloong 8101B (based on the Loongson 2F processor) is my > (relatively) high security machine, and I prefer to avoid running any > code on i

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-22 Thread Mark H Weaver
Andrei Borzenkov writes: > Did you also test cross-compile No. My Yeeloong 8101B (based on the Loongson 2F processor) is my (relatively) high security machine, and I prefer to avoid running any code on it that was not built natively on it from source code. That's also the reason I have not yet

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-21 Thread Andrei Borzenkov
Did you also test cross-compile (building for BE MIPS in your case)? Отправлено с iPhone > 21 сент. 2015 г., в 18:09, Mark H Weaver написал(а): > > Andrei Borzenkov writes: > >> 14.09.2015 02:37, Mark H Weaver пишет: >>> >>> The little-endian-option test fails with GCC 5.2.0. The relevant e

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-21 Thread Mark H Weaver
Andrei Borzenkov writes: > 14.09.2015 02:37, Mark H Weaver пишет: >> >> The little-endian-option test fails with GCC 5.2.0. The relevant errors >> from config.log are: >> >>gcc: error: unrecognized command line option '-target' >>gcc: error: unrecognized command line option '-mlittle-end

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-19 Thread Andrei Borzenkov
Sorry for delay. 14.09.2015 02:37, Mark H Weaver пишет: The little-endian-option test fails with GCC 5.2.0. The relevant errors from config.log are: gcc: error: unrecognized command line option '-target' gcc: error: unrecognized command line option '-mlittle-endian' Oh, well ... let'

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-13 Thread Mark H Weaver
Andrei Borzenkov writes: > 13.09.2015 05:41, Mark H Weaver пишет: >> I'm glad to report that with your new patch, GRUB successfully builds on >> GuixSD mips64el with TARGET_CC=clang. I haven't yet tried booting it >> though. >> > > Did you have chance to test whether compiled grub actually work?

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-13 Thread Andrei Borzenkov
13.09.2015 05:41, Mark H Weaver пишет: Andrei Borzenkov writes: ld: /tmp/conftest-efc530.o: compiled for a big endian system and target is little endian ld: /tmp/conftest-efc530.o: endianness incompatible with that of the selected emulation This is the same problem we had on powerpc. Your

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-12 Thread Mark H Weaver
Hi Andrei, Here's a patch on top of yours that omits the link step from the new autoconf test. This eliminates the warnings I encountered before: diff --git a/configure.ac b/configure.ac index f3c9b90..cd9fa8d 100644 --- a/configure.ac +++ b/configure.ac @@ -605,14 +605,10 @@ if test "x$target_c

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-12 Thread Mark H Weaver
Andrei Borzenkov writes: >> ld: /tmp/conftest-efc530.o: compiled for a big endian system and target is >> little endian >> ld: /tmp/conftest-efc530.o: endianness incompatible with that of the >> selected emulation > > This is the same problem we had on powerpc. Your binutils are set to > use li

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-09-11 Thread Andrei Borzenkov
25.08.2015 21:18, Mark H Weaver пишет: Andrei Borzenkov writes: 24.08.2015 11:12, Andrei Borzenkov пишет: Could you test attached patch for both gcc and clang? Use configure TARGET_CC=clang Sorry, was broken, fixed version attached. Your patch works for me with gcc, although config.log s

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-25 Thread Mark H Weaver
Andrei Borzenkov writes: > 24.08.2015 11:12, Andrei Borzenkov пишет: >> Could you test attached patch for both gcc and clang? Use >> >> configure TARGET_CC=clang > > Sorry, was broken, fixed version attached. Your patch works for me with gcc, although config.log shows this warning when building

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-24 Thread Mark H Weaver
Andrei Borzenkov writes: > Could you paste "clang -v" output for both cases? I believe I > understand what's going on. Okay, I've included it below. It might be relevant to mention that the MIPS port of GNU Guix is not a 'multilib' system: it includes support only for the N32 ABI. However, our

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-24 Thread Andrei Borzenkov
22.08.2015 21:04, Mark H Weaver пишет: Andrei Borzenkov writes: 19.08.2015 21:35, Mark H Weaver пишет: Include -mabi=32 in CFLAGS_PLATFORM and CCASFLAGS_PLATFORM to compile code for the O32 ABI when targetting MIPS, since the MIPS assembly code in GRUB assumes this. This flag is also needed

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-24 Thread Andrei Borzenkov
24.08.2015 11:12, Andrei Borzenkov пишет: 24.08.2015 10:57, Mark H Weaver пишет: Andrei Borzenkov writes: Could you paste "clang -v" output for both cases? I believe I understand what's going on. Okay, I've included it below. It might be relevant to mention that the MIPS port of GNU Guix is

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-24 Thread Andrei Borzenkov
24.08.2015 10:57, Mark H Weaver пишет: Andrei Borzenkov writes: Could you paste "clang -v" output for both cases? I believe I understand what's going on. Okay, I've included it below. It might be relevant to mention that the MIPS port of GNU Guix is not a 'multilib' system: it includes suppo

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-23 Thread Mark H Weaver
Andrei Borzenkov writes: > 22.08.2015 21:04, Mark H Weaver пишет: >> Andrei Borzenkov writes: >> >>> 19.08.2015 21:35, Mark H Weaver пишет: Include -mabi=32 in CFLAGS_PLATFORM and CCASFLAGS_PLATFORM to compile code for the O32 ABI when targetting MIPS, since the MIPS assembly code

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-23 Thread Andrei Borzenkov
22.08.2015 21:04, Mark H Weaver пишет: Andrei Borzenkov writes: 19.08.2015 21:35, Mark H Weaver пишет: Include -mabi=32 in CFLAGS_PLATFORM and CCASFLAGS_PLATFORM to compile code for the O32 ABI when targetting MIPS, since the MIPS assembly code in GRUB assumes this. This flag is also needed

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-22 Thread Mark H Weaver
Andrei Borzenkov writes: > 19.08.2015 21:35, Mark H Weaver пишет: >> Include -mabi=32 in CFLAGS_PLATFORM and CCASFLAGS_PLATFORM to compile >> code for the O32 ABI when targetting MIPS, since the MIPS assembly code >> in GRUB assumes this. This flag is also needed when compiling >> asm-tests/mips

Re: [PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-20 Thread Andrei Borzenkov
19.08.2015 21:35, Mark H Weaver пишет: Include -mabi=32 in CFLAGS_PLATFORM and CCASFLAGS_PLATFORM to compile code for the O32 ABI when targetting MIPS, since the MIPS assembly code in GRUB assumes this. This flag is also needed when compiling asm-tests/mips.S from configure, because GNU as rejec

[PATCH] Ensure that MIPS target code is compiled for the O32 ABI.

2015-08-19 Thread Mark H Weaver
Include -mabi=32 in CFLAGS_PLATFORM and CCASFLAGS_PLATFORM to compile code for the O32 ABI when targetting MIPS, since the MIPS assembly code in GRUB assumes this. This flag is also needed when compiling asm-tests/mips.S from configure, because GNU as rejects MIPS register names such as $t2 unless