All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2] erlang: selectively enable libatomic_ops
Date: Sun, 4 Jan 2015 15:21:05 +0100	[thread overview]
Message-ID: <20150104142105.GA4137@free.fr> (raw)
In-Reply-To: <1419295981-30100-1-git-send-email-fhunleth@troodon-software.com>

Frank, All,

On 2014-12-22 19:53 -0500, Frank Hunleth spake thusly:
> Erlang can use either the system libatomic_ops or its own version.
> However, if using the system version, it must find implementations for
> AO_compare_and_swap() and other calls.  Since these are not available on
> ARM versions before armv6+, selectively enable the use of the system
> libatomic_ops.
> 
> Fixes:
>   http://autobuild.buildroot.net/results/eba/eba5bf85e9ff98916cd3638b5f13ab5bd54564d8/
>   http://autobuild.buildroot.net/results/c74/c743cfca629fb332bbb1327cf4ea360f0a4e4bfe/
>   http://autobuild.buildroot.net/results/095/095348d4d309c2a408a3b297b4325348ed1dd7e8/
> 
> Signed-off-by: Frank Hunleth <fhunleth@troodon-software.com>

I've sent an alternate solution to this libatomic_ops issue:
    http://patchwork.ozlabs.org/patch/425174/

This is much simpler, Buildroot-wise, as it essentially patches erlang,
so that it properly use libatomic_ops.

This is the result of a past discussion with Thomas P. on IRC, where he
pointed me to a patch we have against libunwind, that does require CAS
too, and for which we add `#define AO_REQUIRE_CAS', as instructed in the
libatomic_ops documentation.

Could you please test that new patch and report on it, please?

Thanks!

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2015-01-04 14:21 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-23  0:53 [Buildroot] [PATCH v2] erlang: selectively enable libatomic_ops Frank Hunleth
2015-01-04 14:21 ` Yann E. MORIN [this message]
2015-01-05  5:15   ` Frank Hunleth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150104142105.GA4137@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.