Bug#660525: libffi on m68k failures
Matthias Klose dixit:
> Please provide the patch for the version in experimental.
Sure. @porters: Similar to the sid version, we’re left with:
FAIL: libffi.call/nested_struct2.c -O0 -W -Wall execution test
FAIL: libffi.call/nested_struct2.c -O2 execution test
FAIL: libffi.call/return_sc.c -O2 execution test
FAIL: libffi.call/nested_struct2.c -O3 execution test
FAIL: libffi.call/return_sc.c -O3 execution test
FAIL: libffi.call/nested_struct2.c -Os execution test
FAIL: libffi.call/return_sc.c -Os execution test
FAIL: libffi.call/nested_struct2.c -O2 -fomit-frame-pointer execution test
FAIL: libffi.call/return_sc.c -O2 -fomit-frame-pointer execution test
And then, there’s…
dh_makeshlibs -plibffi6 --add-udeb=libffi6-udeb
dpkg-gensymbols: warning: some new symbols appeared in the symbols file: see diff output below
dpkg-gensymbols: warning: debian/libffi6/DEBIAN/symbols doesn't match completely debian/libffi6.symbols.m68k
--- debian/libffi6.symbols.m68k (libffi6_3.0.11~rc1-5+m68k.1_m68k)
+++ dpkg-gensymbols0plREZ 2012-02-24 19:28:19.000000000 +0000
@@ -13,6 +13,7 @@
ffi_prep_args@Base 3.0.4
ffi_prep_cif@Base 3.0.4
ffi_prep_cif_machdep@Base 3.0.4
+ ffi_prep_cif_var@Base 3.0.11~rc1-5+m68k.1
ffi_prep_closure@Base 3.0.4
ffi_prep_closure_loc@Base 3.0.4
ffi_prep_java_raw_closure@Base 3.0.4
I’ve only built that for testing, not uploaded anything yet.
bye,
//mirabilos
--
Solange man keine schmutzigen Tricks macht, und ich meine *wirklich*
schmutzige Tricks, wie bei einer doppelt verketteten Liste beide
Pointer XORen und in nur einem Word speichern, funktioniert Boehm ganz
hervorragend. -- Andreas Bogk über boehm-gc in d.a.s.r
diff -Nru libffi-3.0.11~rc1/debian/changelog libffi-3.0.11~rc1/debian/changelog
--- libffi-3.0.11~rc1/debian/changelog 2011-10-12 19:46:01.000000000 +0000
+++ libffi-3.0.11~rc1/debian/changelog 2012-02-24 18:29:49.000000000 +0000
@@ -1,3 +1,9 @@
+libffi (3.0.11~rc1-5+m68k.1) local; urgency=low
+
+ * Fix m68k floats (Andreas Schwab) and err_bad_abi (Alan Hourihane).
+
+ -- Thorsten Glaser <tg@mirbsd.de> Fri, 24 Feb 2012 18:28:42 +0000
+
libffi (3.0.11~rc1-5) experimental; urgency=low
* Fix powerpc and ppc64 builds (Kyle Moffett).
diff -Nru libffi-3.0.11~rc1/debian/patches/m68k-fix.diff libffi-3.0.11~rc1/debian/patches/m68k-fix.diff
--- libffi-3.0.11~rc1/debian/patches/m68k-fix.diff 1970-01-01 00:00:00.000000000 +0000
+++ libffi-3.0.11~rc1/debian/patches/m68k-fix.diff 2012-02-24 18:28:08.000000000 +0000
@@ -0,0 +1,72 @@
+Index: libffi-3.0.11~rc1/src/m68k/ffi.c
+===================================================================
+--- libffi-3.0.11~rc1.orig/src/m68k/ffi.c 2011-07-28 19:45:45.000000000 +0000
++++ libffi-3.0.11~rc1/src/m68k/ffi.c 2012-02-24 18:27:55.000000000 +0000
+@@ -261,7 +261,8 @@
+ void *user_data,
+ void *codeloc)
+ {
+- FFI_ASSERT (cif->abi == FFI_SYSV);
++ if (cif->abi != FFI_SYSV)
++ return FFI_BAD_ABI;
+
+ *(unsigned short *)closure->tramp = 0x207c;
+ *(void **)(closure->tramp + 2) = codeloc;
+Index: libffi-3.0.11~rc1/src/m68k/sysv.S
+===================================================================
+--- libffi-3.0.11~rc1.orig/src/m68k/sysv.S 2011-07-28 19:45:45.000000000 +0000
++++ libffi-3.0.11~rc1/src/m68k/sysv.S 2012-02-24 18:27:03.000000000 +0000
+@@ -109,7 +109,7 @@
+ retfloat:
+ btst #2,%d2
+ jbeq retdouble
+-#if defined(__MC68881__)
++#if defined(__MC68881__) || defined(__HAVE_68881__)
+ fmove.s %fp0,(%a1)
+ #else
+ move.l %d0,(%a1)
+@@ -119,7 +119,7 @@
+ retdouble:
+ btst #3,%d2
+ jbeq retlongdouble
+-#if defined(__MC68881__)
++#if defined(__MC68881__) || defined(__HAVE_68881__)
+ fmove.d %fp0,(%a1)
+ #else
+ move.l %d0,(%a1)+
+@@ -130,7 +130,7 @@
+ retlongdouble:
+ btst #4,%d2
+ jbeq retpointer
+-#if defined(__MC68881__)
++#if defined(__MC68881__) || defined(__HAVE_68881__)
+ fmove.x %fp0,(%a1)
+ #else
+ move.l %d0,(%a1)+
+@@ -199,7 +199,7 @@
+ move.l (%a0),%d1
+ jra .Lcls_epilogue
+ .Lcls_ret_float:
+-#if defined(__MC68881__)
++#if defined(__MC68881__) || defined(__HAVE_68881__)
+ fmove.s (%a0),%fp0
+ #else
+ move.l (%a0),%d0
+@@ -209,7 +209,7 @@
+ lsr.l #2,%d0
+ jne 1f
+ jcs .Lcls_ret_ldouble
+-#if defined(__MC68881__)
++#if defined(__MC68881__) || defined(__HAVE_68881__)
+ fmove.d (%a0),%fp0
+ #else
+ move.l (%a0)+,%d0
+@@ -217,7 +217,7 @@
+ #endif
+ jra .Lcls_epilogue
+ .Lcls_ret_ldouble:
+-#if defined(__MC68881__)
++#if defined(__MC68881__) || defined(__HAVE_68881__)
+ fmove.x (%a0),%fp0
+ #else
+ move.l (%a0)+,%d0
diff -Nru libffi-3.0.11~rc1/debian/patches/series libffi-3.0.11~rc1/debian/patches/series
--- libffi-3.0.11~rc1/debian/patches/series 2011-10-07 10:06:54.000000000 +0000
+++ libffi-3.0.11~rc1/debian/patches/series 2012-02-24 18:25:31.000000000 +0000
@@ -3,3 +3,4 @@
sparc-v8-fix.diff
kfreebsd-config.diff
powerpc-softfloat.diff
+m68k-fix.diff
Reply to: