Source: jemalloc Severity: normal Tags: upstream patch Hi,
I am building with "CPPFLAGS='-DLG_QUANTUM=4'" at the moment, and something similar could be applied in debian/rules (as it is done in other architectures), but I guess that it would be better if this had support upstream so not only Debian benefits from this. I provide a patch here: --------------- diff -ur jemalloc.orig/include/jemalloc/internal/jemalloc_internal.h.in jemalloc.new/include/jemalloc/internal/jemalloc_internal.h.in --- jemalloc.orig/include/jemalloc/internal/jemalloc_internal.h.in 2014-03-31 17:37:49.000000000 +0100 +++ jemalloc.new/include/jemalloc/internal/jemalloc_internal.h.in 2014-05-23 00:25:24.338661693 +0100 @@ -268,6 +268,9 @@ # ifdef __mips__ # define LG_QUANTUM 3 # endif +# ifdef __or1k__ +# define LG_QUANTUM 4 +# endif # ifdef __powerpc__ # define LG_QUANTUM 4 # endif --------------- Source file (with some explanation about what this variable is supposed to mean): http://sources.debian.net/src/jemalloc/3.6.0-1/include/jemalloc/internal/jemalloc_internal.h.in I actually don't know if the size should be 3 (__mips__ and __arm__ use this) or 4, which all the rest use, 32 and 64 bits arches alike. Perhaps Christian (in CC) knows better. I tried to compile and builds mostly fine, but fails later for unrelated reasons (I think that it's not honoring "DEB_BUILD_OPTIONS=notest", by the way). It would be useful if you could add this patch soonish and upload to Debian, since we are bootstrapping the OpenRISC/or1k architecture, there are many reverse build-depends for this package, and would let us to not have to carry local patches. Cheers and thanks in advance. -- Manuel -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
