Re: [PATCH] arm: vdso: pass --be8 to linker if necessary

2019-06-18 Thread Masahiro Yamada
Hi. On Fri, May 31, 2019 at 5:20 PM Jason A. Donenfeld wrote: > > Hey Masahiro, > > I'm not sure exactly. I did just notice another place --be8 is being added: That is not my question. I just asked about your commit log: "big-endian ARM was relying on gcc to translate its -mbe8 option into ld's

Re: [PATCH] arm: vdso: pass --be8 to linker if necessary

2019-06-13 Thread Jason A. Donenfeld
Hey Masahiro, Considering ARM big endian userland is pretty badly broken without this, we should probably have this merged in the next rc or rather soon. Was there additional information you needed? Would you prefer Russell queues up my patch or did you want to make further build system changes?

Re: [PATCH] arm: vdso: pass --be8 to linker if necessary

2019-05-31 Thread Russell King - ARM Linux admin
On Fri, May 31, 2019 at 11:01:23AM +0900, Masahiro Yamada wrote: > Hi Jason, > > Thanks for catching this. > > On Thu, May 30, 2019 at 3:26 AM Jason A. Donenfeld wrote: > > > > The commit fe00e50b2db8 ("ARM: 8858/1: vdso: use $(LD) instead of $(CC) > > to link VDSO") removed the passing of CFLAG

Re: [PATCH] arm: vdso: pass --be8 to linker if necessary

2019-05-31 Thread Jason A. Donenfeld
Hey Masahiro, I'm not sure exactly. I did just notice another place --be8 is being added: ifeq ($(CONFIG_CPU_ENDIAN_BE8),y) LDFLAGS_vmlinux += --be8 KBUILD_LDFLAGS_MODULE += --be8 endif I suppose it's possible that this is kbuild related where one of those isn't winding up in the right place.

Re: [PATCH] arm: vdso: pass --be8 to linker if necessary

2019-05-30 Thread Masahiro Yamada
Hi Jason, Thanks for catching this. On Thu, May 30, 2019 at 3:26 AM Jason A. Donenfeld wrote: > > The commit fe00e50b2db8 ("ARM: 8858/1: vdso: use $(LD) instead of $(CC) > to link VDSO") removed the passing of CFLAGS, since ld doesn't take > those directly. However, prior, big-endian ARM was rel

[PATCH] arm: vdso: pass --be8 to linker if necessary

2019-05-29 Thread Jason A. Donenfeld
The commit fe00e50b2db8 ("ARM: 8858/1: vdso: use $(LD) instead of $(CC) to link VDSO") removed the passing of CFLAGS, since ld doesn't take those directly. However, prior, big-endian ARM was relying on gcc to translate its -mbe8 option into ld's --be8 option. Lacking this, ld generated be32 code, m