Bert,<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Nov 26, 2012 at 5:16 AM, Bert Freudenberg <span dir="ltr"><<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
On 2012-11-25, at 17:03, "David T. Lewis" <<a href="mailto:lewis@mail.msen.com">lewis@mail.msen.com</a>> wrote:<br>
<br>
><br>
> To summarize my understanding of this issue (see below for background):<br>
><br>
> - powerpcspe is variant of powerpc that lacks certain parts of the powerpc<br>
> instruction set, as explained here: <a href="http://wiki.debian.org/PowerPCSPEPort" target="_blank">http://wiki.debian.org/PowerPCSPEPort</a><br>
><br>
> - The FFI plugin has architecture-specific assembler modules that set<br>
> up the entry points for FFI calls. The assembly module for powerpc is<br>
> platforms/unix/plugins/SqueakFFIPrims/ppc-sysv-asm.S<br>
><br>
> - The ppc-sysv-asm.S module uses one or more instructions ('lfd', possibly<br>
> others) that are not available on the powerpcspe processor, and the the<br>
> module cannot be compiled on powerpcspe.<br>
><br>
> - The solution for the Debian build is to not build the FFI plugin when<br>
> compiling for powerpcspe. This is done by passing the --without-SqueakFFIPrims<br>
> option to the cmake configure when building on powerpcspe.<br>
><br>
> - The configuration is done in a file called debian/rules, and the provided<br>
> squeak-vm.patch is a patch for that rules file. This file is not part of<br>
> the Squeak VM platforms sources and is presumably maintained externally<br>
> by the Debian folks.<br>
><br>
> - The right way to fix the build problem in the platforms sources would be<br>
> to modify platforms/unix/plugins/SqueakFFIPrims/config.cmake to disable<br>
> the FFI plugin for powerpcspe at configure time. This is probably easy to<br>
> do, although I do not have a way to try it myself.<br>
><br>
> - Extra credit: If someone wanted FFI to work on powerpcspe, the way to make<br>
> this happen would be to write a variation of ppc-sysv-asm.S for powerpcspe<br>
> and add it to config.cmake.<br>
><br>
> Dave<br>
<br>
</div></div>Instead of disabling FFI it might be possible to fall back to libffi, as is done for some other architectures (e.g. ARM).<br></blockquote><div><br></div><div>IMO, much better would be to add PPC support to ThreadedFFIPlugin. This is a framework for cross-platform FFI, currently instantiated for x86, but it has all the hooks one needs to write a PPC interface. This uses minimal assembler, essentially only the occasional asm statement, e.g. to set the flag that indicates a call has active floating-point parameters, or to adjust the stack pointer after an alloca. Being written in terms of the C library's alloca stack allocator to create the outgoing argument frame it is a) more efficient since arguments are not copied from a marshalling space to the outgoing stack frame as occurs in the old FFI, and b) reentrant, since there is no static area used for marshalling.</div>
<div><br></div><div>I've written support for PPC for VisualWorks in the same way so I do know this approach does work and would be happy to help anyone with a PPC mahcine who was interested in biting the bullet. The tricky issues are the need to pass register arguments alongside the stacked arguments, and the need to avoid overwriting the marshalling area of the alloca'ed argument frame when calling functions involved in marshalling arguments. The first issue is solved by always calling the external function as if it had 8 register arguments, and passing 8 arguments, hence loading the argument registers. The second issue is solved by over allocating and cutting back the stack with an asm statement just before the call.</div>
<div><br></div><div>You'll find that the framework was written with this in mind even if the support for PPC and ARM (and SPARC and AXP and MIPS, all of which I wrote for the VisualWorks VM) isn't complete yet.</div>
<div><br></div><div>cheers</div><div>Eliot</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
- Bert -<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
><br>
> On Mon, Nov 19, 2012 at 06:38:10PM +0100, Bert Freudenberg wrote:<br>
>><br>
>> Begin forwarded message:<br>
>><br>
>>> Resent-From: Roland Stigge <<a href="mailto:stigge@antcom.de">stigge@antcom.de</a>><br>
>>> From: Roland Stigge <<a href="mailto:stigge@antcom.de">stigge@antcom.de</a>><br>
>>> Subject: Bug#693633: squeak-vm: FTBFS on powerpcspe (asm stfd/lfd)<br>
>>> Date: 18. November 2012 21:10:04 MEZ<br>
>>> Resent-To: <a href="mailto:debian-bugs-dist@lists.debian.org">debian-bugs-dist@lists.debian.org</a><br>
>>> To: Debian Bug Tracking System <<a href="mailto:submit@bugs.debian.org">submit@bugs.debian.org</a>><br>
>>> Resent-Cc: Debian Squeak Team <<a href="mailto:pkg-squeak-devel@lists.alioth.debian.org">pkg-squeak-devel@lists.alioth.debian.org</a>><br>
>>> Reply-To: Roland Stigge <<a href="mailto:stigge@antcom.de">stigge@antcom.de</a>>, <a href="mailto:693633@bugs.debian.org">693633@bugs.debian.org</a><br>
>>><br>
>>> Package: squeak-vm<br>
>>> Version: 1:4.4.7.2357-1.1<br>
>>> Severity: wishlist<br>
>>> Tags: sid patch<br>
>>> User: debian-powerpcspe@breakpoint.cc<br>
>>> Usertags: powerpcspe<br>
>>><br>
>>> Hi,<br>
>>><br>
>>> squeak-vm FTBFS on powerpcspe [1] like this:<br>
>>><br>
>>> [ 82%] Building ASM object SqueakFFIPrims/CMakeFiles/SqueakFFIPrims.dir/<C2><AB>PKGBUILDDIR<C2><BB>/unix/plugins/SqueakFFIPrims/ppc-sysv-asm.S.o<br>
>>> /PKGBUILDDIR/unix/plugins/SqueakFFIPrims/ppc-sysv-asm.S: Assembler messages:<br>
>>> /PKGBUILDDIR/unix/plugins/SqueakFFIPrims/ppc-sysv-asm.S:109: Error: Unrecognized opcode: `lfd'<br>
>>> [...]<br>
>>><br>
>>> There are some asm statements that only work on powerpc but not on powerpcspe.<br>
>>><br>
>>> Attaching a patch that fixes it by disabling the SqueakFFIPrims plugin on<br>
>>> powerpcspe.<br>
>>><br>
>>> Thanks,<br>
>>><br>
>>> Roland<br>
>>><br>
>>><br>
>>> [1] <a href="http://wiki.debian.org/PowerPCSPEPort" target="_blank">http://wiki.debian.org/PowerPCSPEPort</a><br>
>>><br>
>>> -- System Information:<br>
>>> Debian Release: wheezy/sid<br>
>>> APT prefers unstable<br>
>>> APT policy: (500, 'unstable'), (1, 'experimental')<br>
>>> Architecture: amd64 (x86_64)<br>
>>><br>
>>> Kernel: Linux 3.5.0 (SMP w/4 CPU cores)<br>
>>> Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8) (ignored: LC_ALL set to en_GB.UTF-8)<br>
>>> Shell: /bin/sh linked to /bin/dash<br>
><br>
><br>
>>><br>
>><br>
><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>best,<div>Eliot</div><br>
</div>