From: Peter Seiderer <ps.report@gmx.net>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2] pulseaudio: bump version to 10.0
Date: Tue, 27 Jun 2017 22:03:53 +0200 [thread overview]
Message-ID: <20170627220353.7624f94d@gmx.net> (raw)
In-Reply-To: <20170627211557.76aa88e6@gmx.net>
Hello Bernd,
On Tue, 27 Jun 2017 21:15:57 +0200, Peter Seiderer <ps.report@gmx.net> wrote:
> Hello Bernd,
>
>
> On Tue, 27 Jun 2017 06:19:00 +0200, Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
>
> > Am Tue, 27 Jun 2017 00:25:58 +0200 schrieb Peter Seiderer:
> >
> > > ToDo/FollowUp: pulseaudio uses now a built-in json parser (see [2]),
> > > drop the json-c (and maybe the BR2_TOOLCHAIN_HAS_SYNC_4) dependency.
> >
> > Hi Peter,
> >
> > in http://patchwork.ozlabs.org/patch/718184/ I tried that but the
>
> Ups, missed your patch, sorry for the noise...
>
> > configure logic of pulseaudio seems not to be an easy one. Can you help
> > here?
>
> Did take a quick look today at it...yes seems tricky, no quick solution
> from my side...
>
Did take a second look at the configure.ac file, I think the wanted (and
straight forward) solution is to to relay on the __sync_ built-ins (as
before Thomas Petazzoni patch 'pulseaudio: remove BR2_ARCH_HAS_ATOMICS dependency'
(see [1]), to avoid the special atomic-arm-linux-helpers/atomic-arm-memory-barrier
handling and/or the handling in the else case of the __sync_ test....
The solution would be a partly revert of Thomas commit with a change to BR2_TOOLCHAIN_HAS_SYNC_4
(or similar, not sure how to check which are needed)...
Regards,
Peter
[1] https://git.buildroot.net/buildroot/commit/?id=7290dc1e87b23fe21c75c50b2238004273a0ec5c
$ grep -R __sync_ pulseaudio-10.0/
pulseaudio-10.0/src/pulsecore/atomic.h: __sync_synchronize();
pulseaudio-10.0/src/pulsecore/atomic.h: __sync_synchronize();
pulseaudio-10.0/src/pulsecore/atomic.h: return __sync_fetch_and_add(&a->value, i);
pulseaudio-10.0/src/pulsecore/atomic.h: return __sync_fetch_and_sub(&a->value, i);
pulseaudio-10.0/src/pulsecore/atomic.h: return __sync_bool_compare_and_swap(&a->value, old_i, new_i);
pulseaudio-10.0/src/pulsecore/atomic.h: __sync_synchronize();
pulseaudio-10.0/src/pulsecore/atomic.h: __sync_synchronize();
pulseaudio-10.0/src/pulsecore/atomic.h: return __sync_bool_compare_and_swap(&a->value, (long) old_p, (long) new_p);
> Regards,
> Peter
>
> >
> > Regards, Bernd
> >
> > _______________________________________________
> > buildroot mailing list
> > buildroot at busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
prev parent reply other threads:[~2017-06-27 20:03 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-06-26 22:25 [Buildroot] [PATCH v2] pulseaudio: bump version to 10.0 Peter Seiderer
2017-06-27 4:19 ` Bernd Kuhls
2017-06-27 19:15 ` Peter Seiderer
2017-06-27 20:03 ` Peter Seiderer [this message]
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=20170627220353.7624f94d@gmx.net \
--to=ps.report@gmx.net \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox