Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 24 Apr 2012 12:05:40 +0530
From:      "Jayachandran C." <jchandra@freebsd.org>
To:        Jason Evans <jasone@freebsd.org>
Cc:        freebsd-mips@freebsd.org
Subject:   Re: svn commit: r234370 - in head: contrib/jemalloc contrib/jemalloc/doc contrib/jemalloc/include contrib/jemalloc/include/jemalloc contrib/jemalloc/include/jemalloc/internal contrib/jemalloc/src inclu...
Message-ID:  <CA%2B7sy7D1XLJy8s2i59naqMP=XSL%2BkqXLJg87QonaW9oS22Axmg@mail.gmail.com>
In-Reply-To: <201204170722.q3H7ME6A070934@svn.freebsd.org>
References:  <201204170722.q3H7ME6A070934@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Apr 17, 2012 at 12:52 PM, Jason Evans <jasone@freebsd.org> wrote:
> Author: jasone
> Date: Tue Apr 17 07:22:14 2012
> New Revision: 234370
> URL: http://svn.freebsd.org/changeset/base/234370

[....]
> +diff --git a/include/jemalloc/jemalloc_FreeBSD.h b/include/jemalloc/jema=
lloc_FreeBSD.h
> +new file mode 100644
> +index 0000000..2c5797f
> +--- /dev/null
> ++++ b/include/jemalloc/jemalloc_FreeBSD.h
> +@@ -0,0 +1,76 @@
[....]
> ++#ifdef __mips__
> ++# =A0define LG_SIZEOF_PTR =A0 =A0 =A0 =A0 =A0 =A0 =A0 2
> ++#endif

This breaks 64-bit mips platforms. I have a simple patch that fixes
this at http://people.freebsd.org/~jchandra/jemalloc.patch

Another possible fix suggested by Juli is to do
#define  LG_SIZEOF_PTR _MIPS_SZPTR

Can you please check-in one of these?  If you are busy, I can do the
check-in if you approve the changes.

Thanks,
JC.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CA%2B7sy7D1XLJy8s2i59naqMP=XSL%2BkqXLJg87QonaW9oS22Axmg>