Re: [pulseaudio-discuss] [PACKAGERS] New dep on json-c
Dear Colin, Am Montag, den 16.05.2011, 11:56 +0100 schrieb Colin Guthrie: > I've just pushed Arun's (mostly, Pierre-Louis also had a hand!) > passthrough work to master. > > This carries with it a new dependency: json-c > > We may yet remove this and write our own parser for the simple subset of > JSON formatting we use but there may ultimately be other reasons to keep > this longer term. > > But in the interests of simplicity, I'd certainly not be against any > native implementation patches that came along provided they were simple > and clean. > > Please test these changes. I'm sure Arun will post more details and test > requests in due course. looking at the XBMC commit log [1] they seem to move from JsonCpp [2] to YAJL [3]. Thanks, Paul [1] https://github.com/xbmc/xbmc/commit/56ac2a40cc3d721d428f026bfe11173b27596fe3/ [2] http://lloyd.github.com/yajl/ [3] http://jsoncpp.sourceforge.net/ signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH] loopback: Add a modarg for disabling remixing.
Am Sonntag, den 15.05.2011, 15:12 +0100 schrieb Colin Guthrie: > Thought this one may be coming :) > > In my tree now. Does some documentation need to be added? Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] unambiguous tag names in the repository
Dear PulseAudio folks, I just read the blog post Please use unambiguous tag names in your DVCS [1] which for security reasons suggests to use unambiguous tag names. Therefore I suggest to change our tag name scheme to the following. pa_0.9.22 or pa_1.0 Thanks, Paul [1] http://www.debian-administration.org/users/dkg/weblog/81 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] ORC buildsystem problems
Am Dienstag, den 03.05.2011, 10:40 +0200 schrieb Maarten Bosmans: > 2011/5/3 Paul Menzel : > > Am Dienstag, den 03.05.2011, 08:39 +0530 schrieb Arun Raghavan: > >> On Mon, 2011-05-02 at 11:52 +0200, Maarten Bosmans wrote: […] > >> > Secondly, there is a problem when cross-compiling. The pkg-config > >> > check for ORC is used to find the usual include files and linking > >> > flags, but also to find the location of orcc. This is a problem, > >> > because when configure is run with the correct configuration, such > >> > that pkg-config finds the host package, it also finds the host orcc > >> > (in the case of my mingw32 test, it finds orcc.exe), which is of > >> > course useless in the build environment. > >> > >> Why is it useless in your environment? The files generated by orcc are > >> architecture-neutral. > > The binary that make ends up trying to use (orcc.exe) is useless, > because it's a win32 binary on a Linux platform. So the > architecture-neutral files can't even be generated. > > > For your second issue please take a look at the thread on the > > gstreamer-devel list [1]. > > Ah, yes. It seems that my workaround of setting ORCC= is the advised > solution. Still it would be better to autodetect a broken return value > of pkg-config. I could make a patch to orc.m4 that does just that, but > I can't seem to find the file in the orc git repo. Is orc.m4 managed > locally in the pulse tree, or is there some upstream that improvements > should also be submitted to? As far as I know `orc.m4` is managed locally by all projects depending on Orc like PulseAudio, Gstreamer or Schroedinger. I guess you or I (if your are not subscribed) could forward your patch to the gstreamer-devel list. Thanks, Paul > > [1] > > http://lists.freedesktop.org/archives/gstreamer-devel/2011-March/030983.html signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] ORC buildsystem problems
Am Dienstag, den 03.05.2011, 08:39 +0530 schrieb Arun Raghavan: > On Mon, 2011-05-02 at 11:52 +0200, Maarten Bosmans wrote: > > Recently, I encountered some problems when enabling orc in some less > > usual situations. > > > > When compiling with --enable-orc from a tarball generated from a > > --disable-orc configured tree, the following error occurs. > > make[2]: *** No rule to make target `pulsecore/svolume-orc-gen.c', > > needed by `all'. Stop > > I haven't really looked at a solution. Perhaps the nodist_ prefix for > > some files inside if HAVE_ORC in src/Makefile.am should be dropped? > > May be Colin needs to do his make distchecks with this situation, in > > order to catch it earlier. > > There should not be a dist'ed tarball without those generated files. > Whether they are used or not is a configure-time option, then. > > > Secondly, there is a problem when cross-compiling. The pkg-config > > check for ORC is used to find the usual include files and linking > > flags, but also to find the location of orcc. This is a problem, > > because when configure is run with the correct configuration, such > > that pkg-config finds the host package, it also finds the host orcc > > (in the case of my mingw32 test, it finds orcc.exe), which is of > > course useless in the build environment. > > Why is it useless in your environment? The files generated by orcc are > architecture-neutral. For your second issue please take a look at the thread on the gstreamer-devel list [1]. Thanks, Paul [1] http://lists.freedesktop.org/archives/gstreamer-devel/2011-March/030983.html signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH] bluetooth: Pull a2dp-codecs.h grom BlueZ
Am Donnerstag, den 31.03.2011, 00:56 +0530 schrieb Arun Raghavan: Typo in commit summary: s/grom/from/. > This pulls a2dp-codecs.h from BlueZ which contains the capabilities > structures for SBC and MPEG. We currently have these manually added to > ipc.h, so pulling this header makes our files identical to upstream. > --- > src/Makefile.am |2 +- > src/modules/bluetooth/a2dp-codecs.h | 116 > +++ > src/modules/bluetooth/bluetooth-util.c |8 +- > src/modules/bluetooth/ipc.h | 28 -- > src/modules/bluetooth/module-bluetooth-device.c |8 +- > 5 files changed, 123 insertions(+), 39 deletions(-) > create mode 100644 src/modules/bluetooth/a2dp-codecs.h > > diff --git a/src/Makefile.am b/src/Makefile.am > index f3717ce..a7fc1ca 100644 > --- a/src/Makefile.am > +++ b/src/Makefile.am > @@ -1851,7 +1851,7 @@ libbluetooth_ipc_la_SOURCES = modules/bluetooth/ipc.c > modules/bluetooth/ipc.h > libbluetooth_ipc_la_LDFLAGS = -avoid-version > libbluetooth_ipc_la_LIBADD = $(AM_LIBADD) libpulsecore-@PA_MAJORMINOR@.la > libpulsecommon-@PA_MAJORMINOR@.la libpulse.la > libbluetooth_ipc_la_CFLAGS = $(AM_CFLAGS) > -BLUETOOTH_IPC_FILES = $(subst > modules/bluetooth/,,$(libbluetooth_ipc_la_SOURCES)) rtp.h > +BLUETOOTH_IPC_FILES = $(subst > modules/bluetooth/,,$(libbluetooth_ipc_la_SOURCES)) rtp.h a2dp-codecs.h Sort these alphabetically? […] Otherwise this looks good. Do you have a public Git tree with this patch applied? Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH] bluetooth: run `make update-sbc` to pull in build fix for thumb mode
Date: Tue, 29 Mar 2011 12:14:27 +0200 This update pulls in commit c495077c [1] to fix a build error. commit c495077cf8a8c37afd90875ec5a5b16b294be15e Author: Siarhei Siamashka Date: Tue Mar 29 01:57:39 2011 +0300 sbc: better compatibility with ARM thumb/thumb2 ARM assembly optimizations fail to compile in thumb mode, but are fine for thumb2. Update ifdefs in the code to make use of ARM assembly only when it is safe and also make sure that no optimizations are missed when compiling for thumb2. The problem was reported by Paul Menzel: https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-February/009022.html This patch is tested with OpenEmbedded using `minimal-uclibc` for `MACHINE = "at91sam9260ek"`. Note that changes to ipc.h from 8f3ef04b had to be manually reapplied. [1] http://git.kernel.org/?p=bluetooth/bluez.git;a=commit;h=c495077cf8a8c37afd90875ec5a5b16b294be15e Signed-off-by: Paul Menzel CC: Luiz Augusto von Dentz CC: Arun Raghavan --- Please update the commit summary to something that is technically correct. --- src/modules/bluetooth/sbc/sbc_math.h |2 +- src/modules/bluetooth/sbc/sbc_primitives_armv6.h |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/modules/bluetooth/sbc/sbc_math.h b/src/modules/bluetooth/sbc/sbc_math.h index 9f126c6..5476860 100644 --- a/src/modules/bluetooth/sbc/sbc_math.h +++ b/src/modules/bluetooth/sbc/sbc_math.h @@ -48,7 +48,7 @@ typedef int32_t sbc_fixed_t; #define SBC_FIXED_0(val) { val = 0; } #define MUL(a, b)((a) * (b)) -#ifdef __arm__ +#if defined(__arm__) && (!defined(__thumb__) || defined(__thumb2__)) #define MULA(a, b, res) ({ \ int tmp = res; \ __asm__(\ diff --git a/src/modules/bluetooth/sbc/sbc_primitives_armv6.h b/src/modules/bluetooth/sbc/sbc_primitives_armv6.h index 1862aed..6a9efe5 100644 --- a/src/modules/bluetooth/sbc/sbc_primitives_armv6.h +++ b/src/modules/bluetooth/sbc/sbc_primitives_armv6.h @@ -40,8 +40,8 @@ #if !defined(SBC_HIGH_PRECISION) && (SCALE_OUT_BITS == 15) && \ defined(__GNUC__) && defined(SBC_HAVE_ARMV6) && \ - defined(__ARM_EABI__) && !defined(__thumb__) && \ - !defined(__ARM_NEON__) + defined(__ARM_EABI__) && !defined(__ARM_NEON__) && \ + (!defined(__thumb__) || defined(__thumb2__)) #define SBC_BUILD_WITH_ARMV6_SUPPORT -- 1.7.4.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH] .gitignore: add `.tarball-version`
Date: Tue, 29 Mar 2011 11:49:52 +0200 `.tarball-version` is created by `.git-version-gen`. Signed-off-by: Paul Menzel --- .gitignore |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/.gitignore b/.gitignore index 3a840d9..eb33e91 100644 --- a/.gitignore +++ b/.gitignore @@ -1,3 +1,4 @@ +.tarball-version .version .*.swp ABOUT-NLS -- 1.7.4.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH] sbc_math.h: add explicit check for ARMv6 instructions
Dear BlueZ folks, Am Montag, den 28.03.2011, 18:41 +0300 schrieb Luiz Augusto von Dentz: > On Fri, Mar 25, 2011 at 1:34 PM, Paul Menzel wrote: > > Am Samstag, den 19.03.2011, 16:14 +0530 schrieb Arun Raghavan: > > > >> On Wed, 2011-02-23 at 01:07 +0530, Arun Raghavan wrote: > >> [...] > >> > The correct fix for this, imo, is in bluez (there is a new > >> > sbc_primitives_armv6.h that can probably be used at least as a > >> > template). We need to do an sbc-udpate on the PA side anyway, and can > >> > pull this when we do. > >> > >> Could you see if the attached patch works for you? If it does, I can > >> push this to the bluez folks. > > > > My tag on the other thread [1] is ambiguous. The error is indeed caused > > by your patch. I will try to implement your recommended changes but will > > probably not get to it before Sunday. > > Im including Siarhei Siamashka since he did most, if not all, of this > code and linux-bluetooth to cc. to fix [1] I tried to add `#include "sbc_tables.h"` to `src/modules/bluetooth/sbc/sbc_primitives.h` diff --git a/src/modules/bluetooth/sbc/sbc_primitives.h b/src/modules/bluetooth/ index 3fec8d5..9544826 100644 --- a/src/modules/bluetooth/sbc/sbc_primitives.h +++ b/src/modules/bluetooth/sbc/sbc_primitives.h @@ -24,6 +24,8 @@ * */ +#include "sbc_tables.h" + #ifndef __SBC_PRIMITIVES_H #define __SBC_PRIMITIVES_H But this seems to result in a need for `sbc_math.h` which leads to a circular inclusion. […] In file included from modules/bluetooth/sbc/sbc_primitives.h:27:0, from modules/bluetooth/sbc/sbc_primitives_armv6.h:30, from modules/bluetooth/sbc/sbc_math.h:27, from modules/bluetooth/sbc/sbc.c:46: modules/bluetooth/sbc/sbc_tables.h:50:2: warning: implicit declaration of function 'ASR' [-Wimplicit-function-declaration] modules/bluetooth/sbc/sbc_tables.h:50:2: error: 'SCALE_SPROTO4_TBL' undeclared here (not in a function) modules/bluetooth/sbc/sbc_tables.h:58:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:58:2: error: (near initialization for 'sbc_proto_4_40m1[0]') modules/bluetooth/sbc/sbc_tables.h:58:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:58:2: error: (near initialization for 'sbc_proto_4_40m1[1]') modules/bluetooth/sbc/sbc_tables.h:58:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:58:2: error: (near initialization for 'sbc_proto_4_40m1[2]') modules/bluetooth/sbc/sbc_tables.h:58:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:58:2: error: (near initialization for 'sbc_proto_4_40m1[3]') modules/bluetooth/sbc/sbc_tables.h:59:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:59:2: error: (near initialization for 'sbc_proto_4_40m1[4]') modules/bluetooth/sbc/sbc_tables.h:59:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:59:2: error: (near initialization for 'sbc_proto_4_40m1[5]') modules/bluetooth/sbc/sbc_tables.h:59:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:59:2: error: (near initialization for 'sbc_proto_4_40m1[6]') modules/bluetooth/sbc/sbc_tables.h:59:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:59:2: error: (near initialization for 'sbc_proto_4_40m1[7]') modules/bluetooth/sbc/sbc_tables.h:60:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:60:2: error: (near initialization for 'sbc_proto_4_40m1[8]') modules/bluetooth/sbc/sbc_tables.h:60:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:60:2: error: (near initialization for 'sbc_proto_4_40m1[9]') modules/bluetooth/sbc/sbc_tables.h:60:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:60:2: error: (near initialization for 'sbc_proto_4_40m1[10]') modules/bluetooth/sbc/sbc_tables.h:60:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:60:2: error: (near initialization for 'sbc_proto_4_40m1[11]') modules/bluetooth/sbc/sbc_tables.h:61:2: error: initializer element is not constant modules/bluetooth/sbc/sbc_tables.h:61:2: error: (near initialization for 'sbc_proto_4_40m1[12]
Re: [pulseaudio-discuss] [PATCH] sbc_math.h: add explicit check for ARMv6 instructions
Dear Arun, Am Samstag, den 19.03.2011, 16:14 +0530 schrieb Arun Raghavan: > On Wed, 2011-02-23 at 01:07 +0530, Arun Raghavan wrote: > [...] > > The correct fix for this, imo, is in bluez (there is a new > > sbc_primitives_armv6.h that can probably be used at least as a > > template). We need to do an sbc-udpate on the PA side anyway, and can > > pull this when we do. > > Could you see if the attached patch works for you? If it does, I can > push this to the bluez folks. My tag on the other thread [1] is ambiguous. The error is indeed caused by your patch. I will try to implement your recommended changes but will probably not get to it before Sunday. Thanks, Paul [1] https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-March/009594.html signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH] build: Bump Orc dependency to 0.4.11
Am Freitag, den 25.03.2011, 10:59 + schrieb Colin Guthrie: > 'Twas brillig, and Arun Raghavan at 25/03/11 10:46 did gyre and gimble: > > 0.4.9 errors out at compile time, The error is the following [1]. svolume_orc.c: error: line 67: unknown directive: .longparam > > and might as well bump to 0.4.11 since > > that's the version being tested with and has been around for a while > > now. Thanks to Paul Menzel for > > pointing this out. Well I am undecided if the depends should reflect the necessary version or the recommended one. Orc 0.4.11 seems to fix some bugs [2], but it should be the job of the distribution/person to decide what version to use? For example OpenEmbedded still carries a patch to decrease the version of Autoconf [3] and it seems to work alright. > Thanks. Pushed. This is f49711c9 [4]. Thanks, Paul [1] https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-March/009519.html [2] http://code.entropywave.com/git?p=orc.git;a=blob;f=RELEASE;h=a2629c6e339398b3dc5da700dee325c25feef26a;hb=HEAD [3] http://cgit.openembedded.org/cgit.cgi/openembedded/commit/recipes/pulseaudio/pulseaudio-0.9.21/autoconf_version.patch?id=0e0de60f4443c143fa92068932722f5d75b0999b [4] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=f49711c99a86075de0a2337c4ea71049a9aa146a signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] svolume_orc.c: error: line 67: unknown directive: .longparam
Am Donnerstag, den 24.03.2011, 11:56 +0530 schrieb Arun Raghavan: > On Mon, 2011-03-21 at 00:06 +0100, Paul Menzel wrote: > [...] > > I guess it has something to do with > > > > commit 4cd90d9e32ca9a23e3c0f7615974ea0c55ff3e49 > > Author: Arun Raghavan > > Date: Mon Oct 25 17:59:08 2010 +0100 > > > > volume: Add Orc-based optimised volume scaling > > > > This adds volume scaling for 1- and 2-channel software volume > > scaling > > using Orc. While testing the MMX and SSE backends on a Core2, I > > see an > > ~2x performance benefit over the hand-rolled MMX and SSE code. > > Since I > > haven't been able to test on other architectures, the Orc code > > is only > > used when MMX/SSE* is present. This can be changed in the > > future after > > testing on AMD and ARM machines. > > > > but I do not know anything about this. > > > > I am using OpenEmbedded with `minimal` or `minimal-uclibc` for `MACHINE > > = "at91sam9260ek"`. ORCC 0.4.9 is used on this system. > > Could you try with Orc 0.4.10? Unfortunately, I don't have a quick way > to downgrade my local Orc version to verify? I verified that it works with Orc 0.4.10. > If this is not possible, we > can just bump the patch to 0.4.11, which was released a while ago (and > is what I'm using). You send the patch today and it was already pushed in f49711c9 [1]. > BTW, for 0.9.22 and current stable-queue, the Orc stuff will not get > used for anything on ARM. Understood. Thanks, Paul [1] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=f49711c99a86075de0a2337c4ea71049a9aa146a signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [solved, invalid] modules/bluetooth/sbc/sbc_primitives.h:41:22: error: expected ':', ',', '; ', '}' or '__attribute__' before 'X'
Am Freitag, den 25.03.2011, 11:10 +0530 schrieb Arun Raghavan: > On Thu, 2011-03-24 at 23:20 +0100, Paul Menzel wrote: > > I get the following error with latest master (a9c8f904). > > > > CC libbluetooth_sbc_la-sbc.lo > > In file included from > > modules/bluetooth/sbc/sbc_primitives_armv6.h:30:0, > > from modules/bluetooth/sbc/sbc_math.h:27, > > from modules/bluetooth/sbc/sbc.c:46: > > modules/bluetooth/sbc/sbc_primitives.h:41:22: error: expected ':', > > ',', ';', '}' or '__attribute__' before 'X' > > modules/bluetooth/sbc/sbc.c: In function 'sbc_synthesize_four': > > [...] > > modules/bluetooth/sbc/sbc.c: In function > > 'sbc_get_implementation_info': > > modules/bluetooth/sbc/sbc.c:1216:24: error: 'struct > > sbc_encoder_state' has no member named 'implementation_info' > > modules/bluetooth/sbc/sbc.c:1217:1: warning: control reaches end of > > non-void function [-Wreturn-type] > > make[3]: *** [libbluetooth_sbc_la-sbc.lo] Error 1 > > make[3]: Leaving directory > > `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+a9c8f904b0c4a03e7bff004c103aa5910f8bea3d/git/src' > > make[2]: *** [all] Error 2 > > make[2]: Leaving directory > > `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+a9c8f904b0c4a03e7bff004c103aa5910f8bea3d/git/src' > > make[1]: *** [all-recursive] Error 1 > > make[1]: Leaving directory > > `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+a9c8f904b0c4a03e7bff004c103aa5910f8bea3d/git' > > > > I am using OpenEmbedded with `minimal` using EGLIBC or `minimal-uclibc` > > using uClibc for `MACHINE = "beagleboard"`. > > > > Unfortunately I do not know what syntax error is causing this. > > > > modules/bluetooth/sbc/sbc_primitives.h:41:22: error: expected ':', > > ',', ';', '}' or '__attribute__' before 'X' > > I'm guessing that you've patched sbc_math.h with my earlier patch? If > yes, looks like you need to add a > > #include "sbc_tables.h" > > in sbc_primitives.h since it depends on some defines from that file. I > guess this ends up happening in all the other instances where that file > is used. I am sorry. I had still two patches applied (your `sbc_math.h` and a revert of 4cd90d9e32ca9a23e3c0f7615974ea0c55ff3e49) which I overlooked. > In the mean time, I'll try to unbreak my cross-compile > environment so I can test this stuff locally as well. What environment do you use? I am sorry again and thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] modules/bluetooth/sbc/sbc_primitives.h:41:22: error: expected ':', ',', '; ', '}' or '__attribute__' before 'X'
Dear PulseAudio folks, I get the following error with latest master (a9c8f904). CC libbluetooth_sbc_la-sbc.lo In file included from modules/bluetooth/sbc/sbc_primitives_armv6.h:30:0, from modules/bluetooth/sbc/sbc_math.h:27, from modules/bluetooth/sbc/sbc.c:46: modules/bluetooth/sbc/sbc_primitives.h:41:22: error: expected ':', ',', ';', '}' or '__attribute__' before 'X' modules/bluetooth/sbc/sbc.c: In function 'sbc_synthesize_four': modules/bluetooth/sbc/sbc.c:577:18: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:577:18: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:577:18: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:577:18: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:577:18: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:577:18: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:577:18: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:577:18: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c: In function 'sbc_synthesize_eight': modules/bluetooth/sbc/sbc.c:619:29: warning: declaration of 'tmp' shadows a previous local [-Wshadow] modules/bluetooth/sbc/sbc.c:619:29: warning: shadowed declaration is here [-Wshadow] modules/bluetooth/sbc/sbc.c:619:29: warning: declaration of 'tmp' shadows a previou
[pulseaudio-discuss] svolume_orc.c: error: line 67: unknown directive: .longparam
Dear PulseAudio folks, I get the following error with latest master (c0392af2). | + do_compile | + base_do_compile | + '[' -e Makefile -o -e makefile -o -e GNUmakefile ']' | + oe_runmake | + oenote make | + echo NOTE: make | NOTE: make | + make | make all-recursive | make[1]: Entering directory `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv5te-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+c0392af20a504d7ced715cbd7152fd6d032125aa/git' | Making all in src | make[2]: Entering directory `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv5te-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+c0392af20a504d7ced715cbd7152fd6d032125aa/git/src' | ORCC pulsecore/svolume-orc-gen.c | In function pa_volume_s16ne_orc_2ch: | error: line 67: unknown directive: .longparam | error: variable "(null)" used before being written | Failed to compile pa_volume_s16ne_orc_2ch | Failed to compile pa_volume_s16ne_orc_2ch | make[2]: *** [pulsecore/svolume-orc-gen.c] Error 1 | make[2]: Leaving directory `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv5te-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+c0392af20a504d7ced715cbd7152fd6d032125aa/git/src' | make[1]: *** [all-recursive] Error 1 | make[1]: Leaving directory `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv5te-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+c0392af20a504d7ced715cbd7152fd6d032125aa/git' | make: *** [all] Error 2 | + die 'oe_runmake failed' | + oefatal 'oe_runmake failed' | + echo FATAL: 'oe_runmake failed' | FATAL: oe_runmake failed | + exit 1 I guess it has something to do with commit 4cd90d9e32ca9a23e3c0f7615974ea0c55ff3e49 Author: Arun Raghavan Date: Mon Oct 25 17:59:08 2010 +0100 volume: Add Orc-based optimised volume scaling This adds volume scaling for 1- and 2-channel software volume scaling using Orc. While testing the MMX and SSE backends on a Core2, I see an ~2x performance benefit over the hand-rolled MMX and SSE code. Since I haven't been able to test on other architectures, the Orc code is only used when MMX/SSE* is present. This can be changed in the future after testing on AMD and ARM machines. but I do not know anything about this. I am using OpenEmbedded with `minimal` or `minimal-uclibc` for `MACHINE = "at91sam9260ek"`. ORCC 0.4.9 is used on this system. Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] More patches for OS X
Am Sonntag, den 20.03.2011, 20:07 +0200 schrieb Tanu Kaskinen: > On Sun, 2011-03-20 at 18:39 +0100, Daniel Mack wrote: > > osx: re-order module locations > > This patch didn't reach the list for some reason. I got it around ten minutes after your message. Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH] src/modules/module-tunnel.c: remove unneeded `{`
Date: Sun, 20 Mar 2011 23:01:13 +0100 • fix up for commit 36e7f8a92e8b340dd4034a39a655ba93bcd41886 Author: Colin Guthrie Date: Sun Mar 20 14:09:14 2011 + tunnel: Fix tunnel streams with recent servers Prompted by bug #750. Thanks to Emil Renner Berthing for the original patch. --- src/modules/module-tunnel.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/modules/module-tunnel.c b/src/modules/module-tunnel.c index a8af6c1..6cb22e0 100644 --- a/src/modules/module-tunnel.c +++ b/src/modules/module-tunnel.c @@ -1689,7 +1689,7 @@ static void setup_complete_callback(pa_pdispatch *pd, uint32_t command, uint32_t if (u->version >= 17) pa_tagstruct_put_boolean(reply, FALSE); /* relative volume */ -if (u->version >= 18) { +if (u->version >= 18) pa_tagstruct_put_boolean(reply, FALSE); /* passthrough stream */ #endif -- 1.7.4.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH] src/modules/module-tunnel.c: fix typo (s/passthough/passthrough/)
Date: Sun, 20 Mar 2011 23:00:19 +0100 --- src/modules/module-tunnel.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/modules/module-tunnel.c b/src/modules/module-tunnel.c index 65fb67d..a8af6c1 100644 --- a/src/modules/module-tunnel.c +++ b/src/modules/module-tunnel.c @@ -1690,7 +1690,7 @@ static void setup_complete_callback(pa_pdispatch *pd, uint32_t command, uint32_t pa_tagstruct_put_boolean(reply, FALSE); /* relative volume */ if (u->version >= 18) { -pa_tagstruct_put_boolean(reply, FALSE); /* passthough stream */ +pa_tagstruct_put_boolean(reply, FALSE); /* passthrough stream */ #endif pa_pstream_send_tagstruct(u->pstream, reply); -- 1.7.4.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] What tests on AMD and ARM are needed for Orc-based optimised volume scaling?
Dear Arun, your commit messages of commit 4cd90d9e [1] says the following. … Since I haven't been able to test on other architectures, the Orc code is only used when MMX/SSE* is present. This can be changed in the future after testing on AMD and ARM machines. What tests need to be performed or what tests did you run to figure out that it works. Thanks, Paul [1] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=4cd90d9e32ca9a23e3c0f7615974ea0c55ff3e49 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Nvidia hdmi passthrough patch
Dear Kelly, Am Freitag, den 04.03.2011, 14:50 -0700 schrieb Kelly Anderson: > Here's a patch that I created for git that adds passthrough support for > the Nvidia hdmi port. Pretty much just copy/paste from the iec958 sections. > > --- ./src/modules/alsa/alsa-sink.c.orig2011-02-16 22:58:11.0 > -0700 > +++ ./src/modules/alsa/alsa-sink.c2011-02-17 02:53:21.452857569 -0700 > @@ -1708,7 +1708,8 @@ static int setup_mixer(struct userdata * your MUA seems to have mangled your patch. Please turn off automatic line breaks or use `git send-email` to send the patch. To ease the work for developers please also add a commit summary and commit message (maybe with a link to the list discussion). […] Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH 1/2] Add a target to the PA log feature
Dear VincentX, please note in the subject what iteration your patch is, e. g. »[PATCH 1/2 v2]«. See `--subject-prefix` in `git help format-patch`. Am Donnerstag, den 24.02.2011, 16:30 + schrieb Becker, VincentX: > From 55f84afd8575dadba697c746daa61ee07b333c57 Mon Sep 17 00:00:00 2001 > From: Vincent Becker > Date: Thu, 24 Feb 2011 16:52:05 +0100 > Subject: [PATCH] Add a new log target to a file descriptor in daemon > configuration > Signed-off-by: Vincent Becker > This patches adds the option to log pulseaudio messages into a file > descriptor. > The file descriptor can be a regular file or other type (character, block..). > If the file given in parameter already exists, it is automatically renamed > with current date infos. You are missing a blank line after the commit summary (first line). Applying your patch, e. g. with `git am …`, needs therefore manual editing. It is also common to add the Signed-off-by line at the end. Could you add an example to the commit message about how to use the new functionality so that people can easily test you new feature? > --- > src/daemon/cmdline.c | 98 +- > src/daemon/daemon-conf.c | 108 > +- > src/pulsecore/log.c |6 +++ > src/pulsecore/log.h |5 ++ > 4 files changed, 166 insertions(+), 51 deletions(-) > > diff --git a/src/daemon/cmdline.c b/src/daemon/cmdline.c > index f6cdcdc..7f72b8c 100644 > --- a/src/daemon/cmdline.c > +++ b/src/daemon/cmdline.c > @@ -114,59 +114,59 @@ void pa_cmdline_help(const char *argv0) { > > printf(_("%s [options]\n\n" > "COMMANDS:\n" > - " -h, --helpShow this help\n" > - " --version Show version\n" > - " --dump-conf Dump default > configuration\n" > - " --dump-modulesDump list of available > modules\n" > - " --dump-resample-methods Dump available resample > methods\n" > - " --cleanup-shm Cleanup stale shared > memory segments\n" > - " --start Start the daemon if it > is not running\n" > - " -k --killKill a running daemon\n" > - " --check Check for a running > daemon (only returns exit code)\n\n" > + " -h, --help Show this > help\n" > + " --versionShow > version\n" > + " --dump-conf Dump default > configuration\n" > + " --dump-modules Dump list of > available modules\n" > + " --dump-resample-methods Dump > available resample methods\n" > + " --cleanup-shmCleanup stale > shared memory segments\n" > + " --start Start the > daemon if it is not running\n" > + " -k --kill Kill a > running daemon\n" > + " --check Check for a > running daemon (only returns exit code)\n\n" > > "OPTIONS:\n" > - " --system[=BOOL] Run as system-wide > instance\n" > - " -D, --daemonize[=BOOL]Daemonize after > startup\n" > - " --fail[=BOOL] Quit when startup > fails\n" > - " --high-priority[=BOOL]Try to set high nice > level\n" > - "(only available as root, > when SUID or\n" > - "with elevated > RLIMIT_NICE)\n" > - " --realtime[=BOOL] Try to enable realtime > scheduling\n" > - "(only available as root, > when SUID or\n" > - "with elevated > RLIMIT_RTPRIO)\n" > - " --disallow-module-loading[=BOOL] Disallow module user > requested module\n" > - "loading/unloading after > startup\n" > - " --disallow-exit[=BOOL]Disallow user requested > exit\n" > - " --exit-idle-time=SECS Terminate the daemon > when idle and this\n" > - "time passed\n" > - " --module-idle-time=SECS Unload autoloaded > modules when idle and\n" > - "this time passed\n" > - " --scache-idle-time=SECS Unload autoloaded > samples when idle and\n" > - "
Re: [pulseaudio-discuss] [PATCH] sbc_math.h: add explicit check for ARMv6 instructions
Am Dienstag, den 22.02.2011, 09:48 + schrieb Colin Guthrie: > 'Twas brillig, and Arun Raghavan at 21/02/11 20:31 did gyre and gimble: > > On Sun, 2011-02-20 at 19:20 +0100, Paul Menzel wrote: > >> Date: Sun, 20 Feb 2011 15:57:55 +0100 > >> > >> Building PulseAudio 051d8213 [1] using OpenEmbedded with distribution > >> `minimal-uclibc` for `MACHINE="om-gta01"` having an ARMv4T architecture > >> (armv4t-oe-linux-uclibceabi) compilation fails with the following error. > > [...] > >>{standard input}:4099: Error: selected processor does not support Thumb > >> mode `mla r3,r2,r6,r3' > >>{standard input}:4114: Error: selected processor does not support Thumb > >> mode `mla r3,r2,ip,r3' > >>{standard input}:4125: Error: selected processor does not support Thumb > >> mode `mla r3,r6,r2,r3' > >>make[3]: *** [libbluetooth_sbc_la-sbc.lo] Error 1 > >> > >> Apply the same fix as in [2], which is similar to the patch applied in > >> OpenEmbedded since commit 976ab4b5 [3]. > >> > >> [1] > >> http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=051d82133f0ae6a57bf66fd200bc8e3591a7d5ca > >> [2] > >> http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=12b900858ae82d435c100d6eb94cb7bb22fe5e29 > >> [3] > >> http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=976ab4b4587d548c0483a274058c5359cb72bf1b > >> > >> Signed-off-by: Paul Menzel > >> CC: Marcel Holtmann > >> --- > >> I do not know if these files are just copied from linux-bluetooth > >> upstream, so I am adding Marcel to CC anyway because he is mentioned in > >> the copyright section. > > > > It does appear that the files are a copy of the bluez package. If we're > > fixing this on the PulseAudio side, I think we should include something > > like diff below configure.ac, so that we can actually verify that the > > mla instruction is available for the arch being used. > > > > -- Arun > > > > > > diff --git a/configure.ac b/configure.ac > > index 08c947a..77e9823 100644 > > --- a/configure.ac > > +++ b/configure.ac > > @@ -257,13 +257,14 @@ case $host in > > asm volatile ("ldr r0, %2 \n" > > "ldr r2, %3 \n" > > "ldr r3, %4 \n" > > + "mla r4, r1, r2, r3 \n" > > "ssat r1, #8, r0 \n" > > "str r1, %0 \n" > > "pkhbt r1, r3, r2, LSL #8 \n" > > "str r1, %1 \n" > > : "=m" (a), "=m" (b) > > : "m" (a), "m" (b), "m" (c) > > - : "r0", "r1", "r2", "r3", "cc"); > > + : "r0", "r1", "r2", "r3", "r4", "cc"); > > return (a == -128 && b == 0xaabb) ? 0 : -1; > > ]]), > > [pulseaudio_cv_support_armv6=yes], > > Paul, can you confirm if this avoids the problem for you? If so, then > Arun, can you make a proper patch? You can? Awesome, thanks :) Does not Arun’s patch just improve the check for `HAVE_ARMV6`? My patch just adds the check `if defined(HAVE_ARMV6)` to `sbc_math.h`. Without that addition all the configure checks are not evaluated at all in `sbc_math.h`. Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] patches in OpenEmbedded (was: [PATCH] src/Makefile.am: add missing space to fix build using uClibc)
Am Dienstag, den 22.02.2011, 10:02 + schrieb Colin Guthrie: > 'Twas brillig, and Paul Menzel at 19/02/11 20:19 did gyre and gimble: > > Concerning OpenEmbedded you can see the patches in the repository [1]. I > > am trying to send them upstream during the next days. > > > > [1] > > http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/pulseaudio/ > > I think the most interesting one there is the ARM fixes: > http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/pulseaudio/pulseaudio-0.9.21/armv4+v5asm.patch That is exactly what »[PATCH] sbc_math.h: add explicit check for ARMv6 instructions« [1] is about. I even referenced that link in the commit message. > Arun, as my new resident go-to-guy for ARM stuff, what's your take on it? Please see the other thread. Thanks, Paul [1] https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-February/009022.html signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH] sbc_math.h: add explicit check for ARMv6 instructions
Date: Sun, 20 Feb 2011 15:57:55 +0100 Building PulseAudio 051d8213 [1] using OpenEmbedded with distribution `minimal-uclibc` for `MACHINE="om-gta01"` having an ARMv4T architecture (armv4t-oe-linux-uclibceabi) compilation fails with the following error. […] modules/bluetooth/sbc.c:485:10: warning: cannot optimize possibly infinite loops [-Wunsafe-loop-optimizations] {standard input}: Assembler messages: {standard input}:3628: Error: selected processor does not support Thumb mode `mla r3,r0,r1,r3' {standard input}:3635: Error: selected processor does not support Thumb mode `mla r3,r2,r0,r3' {standard input}:3643: Error: selected processor does not support Thumb mode `mla r3,r1,r0,r3' {standard input}:3705: Error: selected processor does not support Thumb mode `mla r3,r7,ip,r3' {standard input}:3717: Error: selected processor does not support Thumb mode `mla r3,r7,ip,r3' {standard input}:3729: Error: selected processor does not support Thumb mode `mla r3,r7,ip,r3' {standard input}:3741: Error: selected processor does not support Thumb mode `mla r3,r7,ip,r3' {standard input}:3753: Error: selected processor does not support Thumb mode `mla r3,r7,ip,r3' {standard input}:3767: Error: selected processor does not support Thumb mode `mla r3,r7,ip,r3' {standard input}:3780: Error: selected processor does not support Thumb mode `mla r3,r7,ip,r3' {standard input}:3786: Error: selected processor does not support Thumb mode `mla r3,r4,r7,r3' {standard input}:3794: Error: selected processor does not support Thumb mode `mla r3,r0,r4,r3' {standard input}:3895: Error: selected processor does not support Thumb mode `mla r1,r5,r6,r1' {standard input}:3902: Error: selected processor does not support Thumb mode `mla r1,r5,r6,r1' {standard input}:3909: Error: selected processor does not support Thumb mode `mla r1,r5,r6,r1' {standard input}:3916: Error: selected processor does not support Thumb mode `mla r1,r5,r6,r1' {standard input}:3923: Error: selected processor does not support Thumb mode `mla r1,r5,r6,r1' {standard input}:3930: Error: selected processor does not support Thumb mode `mla r1,r5,r6,r1' {standard input}:3937: Error: selected processor does not support Thumb mode `mla r1,r5,r6,r1' {standard input}:4018: Error: selected processor does not support Thumb mode `mla r3,r6,r2,r3' {standard input}:4034: Error: selected processor does not support Thumb mode `mla r3,r2,ip,r3' {standard input}:4045: Error: selected processor does not support Thumb mode `mla r3,r2,r6,r3' {standard input}:4061: Error: selected processor does not support Thumb mode `mla r3,r2,ip,r3' {standard input}:4072: Error: selected processor does not support Thumb mode `mla r3,r2,r6,r3' {standard input}:4088: Error: selected processor does not support Thumb mode `mla r3,r2,ip,r3' {standard input}:4099: Error: selected processor does not support Thumb mode `mla r3,r2,r6,r3' {standard input}:4114: Error: selected processor does not support Thumb mode `mla r3,r2,ip,r3' {standard input}:4125: Error: selected processor does not support Thumb mode `mla r3,r6,r2,r3' make[3]: *** [libbluetooth_sbc_la-sbc.lo] Error 1 Apply the same fix as in [2], which is similar to the patch applied in OpenEmbedded since commit 976ab4b5 [3]. [1] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=051d82133f0ae6a57bf66fd200bc8e3591a7d5ca [2] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=12b900858ae82d435c100d6eb94cb7bb22fe5e29 [3] http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=976ab4b4587d548c0483a274058c5359cb72bf1b Signed-off-by: Paul Menzel CC: Marcel Holtmann --- I do not know if these files are just copied from linux-bluetooth upstream, so I am adding Marcel to CC anyway because he is mentioned in the copyright section. --- src/modules/bluetooth/sbc_math.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/modules/bluetooth/sbc_math.h b/src/modules/bluetooth/sbc_math.h index b87bc81..b0f46e5 100644 --- a/src/modules/bluetooth/sbc_math.h +++ b/src/modules/bluetooth/sbc_math.h @@ -47,7 +47,7 @@ typedef int32_t sbc_fixed_t; #define SBC_FIXED_0(val) { val = 0; } #define MUL(a, b)((a) * (b)) -#ifdef __arm__ +#if defined(__arm__) && defined(HAVE_ARMV6) #define MULA(a, b, res) ({ \ int tmp = res; \ __asm__(\ @@ -57,4 +57,4 @@ typedef int32_t sbc_fixed_t; tmp; }) #else #define MULA(a, b, res) ((a) *
Re: [pulseaudio-discuss] [PATCH] src/Makefile.am: add missing space to fix build using uClibc
Am Samstag, den 19.02.2011, 14:19 +0100 schrieb Maarten Bosmans: > 2011/2/19 Paul Menzel : > > From: Henning Heinold > > Date: Fri Jul 10 12:33:30 2009 +0200 > > > > Using uClibc > > > >AM_LIBADD = $(PTHREAD_LIBS) $(INTLLIBS) > > > > is not empty because `$(INTLLIBS)` is set to `-lintl`. This uncovered a > > missing space in commit ef0cc745 which resulted in the following build > > error. > > > >[…] > >CC libbluetooth_ipc_la-ipc.lo > >modules/bluetooth/ipc.c: In function 'bt_audio_service_open': > >modules/bluetooth/ipc.c:65:26: warning: dereferencing type-punned > > pointer might break strict-aliasing rules [-Wstrict-aliasing] > >modules/bluetooth/ipc.c: In function 'bt_audio_service_get_data_fd': > >modules/bluetooth/ipc.c:110:13: warning: cast increases required > > alignment of target type [-Wcast-align] > >modules/bluetooth/ipc.c:110:4: warning: dereferencing type-punned > > pointer will break strict-aliasing rules [-Wstrict-aliasing] > >make[3]: *** No rule to make target `-lintl', needed by > > `libbluetooth-ipc.la'. Stop. > >make[3]: Leaving directory > > `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git/src' > >make[2]: *** [all] Error 2 > >make[2]: Leaving directory > > `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git/src' > >make[1]: *** [all-recursive] Error 1 > >make[1]: Leaving directory > > `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git' > >make: *** [all] Error 2 > > > > This patch is taken from OpenEmbedded where it has been present since 2009 > > for PulseAudio 0.9.15 and greater [1]. > > Hmm, yes indeed. What ever happened to upstreaming patches? Well what I saw in OpenEmbedded is that there only some dozens of people seem to maintain a whole frame work for building distributions and so they seem to not have much time to get patches upstream. Maybe Henning – keep him in CC please – can comment what could be improved. Additionally there is no maintainer per recipe so it takes even more time to figure out how patches have to be formatted and submitted. And at least for PulseAudio the Web site says or suggests – as far as I know – to open a Trac ticket for which you have to register and operate through a Web interface. And if there is a list you often have to subscribe to send messages. Well it is good to know that patches can be sent to the list too. > Is there any more interesting stuff OpenEmbedded (or any other > distro/downstream for that matter) that isn't in git master yet? Concerning OpenEmbedded you can see the patches in the repository [1]. I am trying to send them upstream during the next days. […] Thanks, Paul [1] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/pulseaudio/ signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH] src/Makefile.am: add missing space to fix build using uClibc
From: Henning Heinold Date: Fri Jul 10 12:33:30 2009 +0200 Using uClibc AM_LIBADD = $(PTHREAD_LIBS) $(INTLLIBS) is not empty because `$(INTLLIBS)` is set to `-lintl`. This uncovered a missing space in commit ef0cc745 which resulted in the following build error. […] CC libbluetooth_ipc_la-ipc.lo modules/bluetooth/ipc.c: In function 'bt_audio_service_open': modules/bluetooth/ipc.c:65:26: warning: dereferencing type-punned pointer might break strict-aliasing rules [-Wstrict-aliasing] modules/bluetooth/ipc.c: In function 'bt_audio_service_get_data_fd': modules/bluetooth/ipc.c:110:13: warning: cast increases required alignment of target type [-Wcast-align] modules/bluetooth/ipc.c:110:4: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] make[3]: *** No rule to make target `-lintl', needed by `libbluetooth-ipc.la'. Stop. make[3]: Leaving directory `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git/src' make[2]: *** [all] Error 2 make[2]: Leaving directory `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git' make: *** [all] Error 2 This patch is taken from OpenEmbedded where it has been present since 2009 for PulseAudio 0.9.15 and greater [1]. [1] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=ef0cc74567b3bb98378c17f6a523bf18ba132ed7 [2] http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=7c33f2e906a20e139d53b4f2d8fbc2773a4725b3 Signed-off-by: Henning Heinold Signed-off-by: Paul Menzel CC: Henning Heinold --- src/Makefile.am |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/Makefile.am b/src/Makefile.am index 0687e40..0ee7abc 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -1804,7 +1804,7 @@ BLUETOOTH_SBC_FILES = $(subst modules/bluetooth/,,$(libbluetooth_sbc_la_SOURCES) libbluetooth_ipc_la_SOURCES = modules/bluetooth/ipc.c modules/bluetooth/ipc.h libbluetooth_ipc_la_LDFLAGS = -avoid-version -libbluetooth_ipc_la_LIBADD = $(AM_LIBADD)libpulsecore-@PA_MAJORMINOR@.la libpulsecommon-@PA_MAJORMINOR@.la libpulse.la +libbluetooth_ipc_la_LIBADD = $(AM_LIBADD) libpulsecore-@PA_MAJORMINOR@.la libpulsecommon-@PA_MAJORMINOR@.la libpulse.la libbluetooth_ipc_la_CFLAGS = $(AM_CFLAGS) BLUETOOTH_IPC_FILES = $(subst modules/bluetooth/,,$(libbluetooth_ipc_la_SOURCES)) rtp.h -- 1.7.4.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH 1/3] configure.ac: Typo. s/Console Kit/ConsoleKit/ [1]
Date: Fri, 18 Feb 2011 23:27:20 +0100 [1] http://www.freedesktop.org/wiki/Software/ConsoleKit --- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index 08c947a..b4fa78b 100644 --- a/configure.ac +++ b/configure.ac @@ -1688,7 +1688,7 @@ if test "${ENABLE_DBUS}" = "no" ; then = WARNING WARNING WARNING WARNING WARNING WARNING WARNING = You do not have DBUS support enabled. It is strongly recommended that you enable DBUS support if you platform supports it. -Many parts of PulseAudio use udev, from Console Kit interaction +Many parts of PulseAudio use udev, from ConsoleKit interaction to the Device Reservation Protocol to speak to JACK, Bluetooth support and even a native control protocol for communicating and controling the PulseAudio daemon itself. -- 1.7.4.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH 3/3] configure.ac: fix spelling and context in warning
Date: Fri, 18 Feb 2011 23:31:47 +0100 Signed-off-by: Paul Menzel --- configure.ac |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 7db5870..b80799f 100644 --- a/configure.ac +++ b/configure.ac @@ -1687,8 +1687,8 @@ if test "${ENABLE_DBUS}" = "no" ; then echo " = WARNING WARNING WARNING WARNING WARNING WARNING WARNING = You do not have D-Bus support enabled. It is strongly recommended -that you enable D-Bus support if you platform supports it. -Many parts of PulseAudio use udev, from ConsoleKit interaction +that you enable D-Bus support if your platform supports it. +Many parts of PulseAudio use D-Bus, from ConsoleKit interaction to the Device Reservation Protocol to speak to JACK, Bluetooth support and even a native control protocol for communicating and controling the PulseAudio daemon itself. @@ -1700,7 +1700,7 @@ if test "${ENABLE_UDEV}" = "no" ; then echo " = WARNING WARNING WARNING WARNING WARNING WARNING WARNING = You do not have udev support enabled. It is strongly recommended -that you enable udev support if you platform supports it as it is +that you enable udev support if your platform supports it as it is the primary method used to detect hardware audio devices (on Linux) and is thus a critical part of PulseAudio on that platform. = WARNING WARNING WARNING WARNING WARNING WARNING WARNING = -- 1.7.4.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH 2/3] configure.ac: Typo. s/DBUS/D-Bus/ [1]
Date: Fri, 18 Feb 2011 23:30:04 +0100 [1] http://www.freedesktop.org/wiki/Software/dbus Signed-off-by: Paul Menzel --- configure.ac |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index b4fa78b..7db5870 100644 --- a/configure.ac +++ b/configure.ac @@ -1259,7 +1259,7 @@ AC_SUBST(HAVE_DBUS) AM_CONDITIONAL([HAVE_DBUS], [test "x$HAVE_DBUS" = x1]) -# udev and HAL depend on DBUS: So double check if they were explicitly enabled. +# udev and HAL depend on D-Bus: So double check if they were explicitly enabled. if test "x$HAVE_DBUS" != "x1" ; then HAVE_HAL=0 if test "x${hal}" = xyes ; then @@ -1660,7 +1660,7 @@ echo " Enable Jack: ${ENABLE_JACK} Enable Async DNS: ${ENABLE_LIBASYNCNS} Enable LIRC: ${ENABLE_LIRC} -Enable DBUS: ${ENABLE_DBUS} +Enable D-Bus: ${ENABLE_DBUS} Enable HAL: ${ENABLE_HAL} Enable BlueZ:${ENABLE_BLUEZ} Enable udev: ${ENABLE_UDEV} @@ -1686,8 +1686,8 @@ echo " if test "${ENABLE_DBUS}" = "no" ; then echo " = WARNING WARNING WARNING WARNING WARNING WARNING WARNING = -You do not have DBUS support enabled. It is strongly recommended -that you enable DBUS support if you platform supports it. +You do not have D-Bus support enabled. It is strongly recommended +that you enable D-Bus support if you platform supports it. Many parts of PulseAudio use udev, from ConsoleKit interaction to the Device Reservation Protocol to speak to JACK, Bluetooth support and even a native control protocol for communicating and -- 1.7.4.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH] Small changes to make "module-coreaduio-detect" more safe
Dear Cai, Am Dienstag, den 21.12.2010, 14:11 +0800 schrieb caiiiyua: Typo in commit summary. s/coreaduio/coreaudio/ […] > From: Cai.Yuanqing Maybe replace ».« by space. Also in your Git configuration: git config user.name "Cai Yuanqing" […] Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Couple of PA problems
Am Sonntag, den 28.11.2010, 17:12 +0800 schrieb James Harkins: > At Sat, 27 Nov 2010 06:02:51 -0500, Sean McNamara wrote: > > Why they are not independent: Good question ;-) The answer is that > > PA's flat-volumes feature is enabled by default. > > It gets a little more weird. Today, I tried to reproduce the behavior after > opening rhythmbox. No problem -- the volume controls seemed to be independent. > > After a couple hours, and listening to some music, they are linked again. > > It seems like, if the master volume is at a high level, lowering the volume > in rhythm box doesn't change the master volume. But, if the master is turned > down, bringing up the rhythm box volume slider also increases the master. > > Changing the master (in either direction) also changes the rhythm box setting. > > I wonder... the problem started to reproduce after plugging in my iPod. I > can't imagine why that would make a difference, but this whole thing is > strange enough, anything is possible. Could you please post more information. 1. What version of PA do you run? 2. Did you try the latest release 0.9.22? 3. Does that happen with another media player, for example Banshee, too? 4. What version of Rhythmbox do you use? Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Couple of PA problems
Dear James, Am Samstag, den 27.11.2010, 18:24 +0800 schrieb James Harkins: […] > 2. As a workaround for problem #1, I thought I could turn down the > main system volume and then turn up the volume in client applications. > But I found, for instance, that changing the system volume also > changes the volume in rhythm box and vice versa. > > Why are these not independent? Where is the preference so that I can > make them independent? Please take a look at the Wiki page PulseAudioStoleMyVolumes [1] linked from [2]. Thanks, Paul [1] http://pulseaudio.org/wiki/PulseAudioStoleMyVolumes [2] http://pulseaudio.org/wiki/CannedResponses signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH] man pulse-daemon.conf: Reference correct `--use-pid-file` and fix typo (`s/Of/If/`).
Date: Sun, 31 Jan 2010 14:15:31 +0100 Signed-off-by: Paul Menzel --- man/pulse-daemon.conf.5.xml.in |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in index f0e1de4..299af6a 100644 --- a/man/pulse-daemon.conf.5.xml.in +++ b/man/pulse-daemon.conf.5.xml.in @@ -123,12 +123,12 @@ USA. use-pid-file= Create a PID file in - /tmp/pulse-$USER/pid. Of this is enabled you may + /tmp/pulse-$USER/pid. If this is enabled you may use commands like --kill or --check. If you are planning to start more than one PulseAudio process per user, you better disable this option since it effectively disables multiple instances. Takes a boolean argument, defaults - to yes. The --no-cpu-limit command line + to yes. The --use-pid-file command line option takes precedence. -- 1.7.2.3 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Choose remote default sink from commandline?
Am Freitag, den 12.11.2010, 21:55 -0500 schrieb Dave: […] > - How do I set this tunnel as the default sink (so apps launched from > gnome will use it)? As they wrote multiple times already. Just move the stream to this sink in `pavucontrol` and PA will remember that setting. Your subject line seems to require a solution for the command line, but since you used the *deprecated* `padevchooser` (annotation for users reading the archives) I guess you will be able to run `pavucontrol`. Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Add a new target to Pulseaudio log feature
Dear Vincent, Am Freitag, den 05.11.2010, 14:10 + schrieb Becker, VincentX: […] > Would it be possible to add them in Pulseaudio and shall I create a ticket > for this ? > > Could you provide me some details in the process of upstreaming on PA.org ? > > I can quickly provide the code for our needs. please send a patch to this mailing list for review. $ git format-patch -1 -s $ git send-email 0001… # not required, you can also attach the file Thanks, Paul PS: Please send only plain text messages. signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Compiler optimisation dependency?
Am Sonntag, den 24.10.2010, 21:11 +0100 schrieb tarantism: > On Sun, 2010-10-24 at 21:25 +0200, Paul Menzel wrote: > > > Any ideas? > > > > What versions of PulseAudio and your tool chain do you use? > PulseAudio 0.9.15 > sbox-arm-none-linux-gnueabi-gcc (GCC) 4.2.1 > ltmain.sh (GNU libtool) 2.2.6 > automake (GNU automake) 1.11.1 > > ... is that everything? Looks good to me, but I am not a developer. I think everybody will be interested if the latest development version works for you and if this problem is fixed there. Could you install from Git and try `stable-queue` (and `master`(?)) [1]? Thanks, Paul [1] http://pulseaudio.org/wiki/DownloadPulseAudio#Sourcecode [2] http://colin.guthr.ie/2010/09/compiling-and-running-pulseaudio-from-git/ signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Compiler optimisation dependency?
Am Sonntag, den 24.10.2010, 19:37 +0100 schrieb tarantism: > I'm building pulse under Scratchbox for Maemo 5 then copying the > module-sine.so file to Nokia N900 to test. > > If I use compiler options -O0 or -O1, it works fine. > If I use -O2 or -O3 I just get a sqwarking noise. > > Any ideas? What versions of PulseAudio and your tool chain do you use? Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Man page clarifications
Am Donnerstag, den 21.10.2010, 08:49 +0200 schrieb David Henningsson: > I'm relaying a bug comment to this list, the reporter has ideas about > the PulseAudio documentation. He originally filed a bug about that when > adding a pair of lines to ~/.pulse/default.pa and PulseAudio stopped > working, as he was unaware that it caused /etc/pulse/default.pa not > being used. I think it's healthy to get bug reports like these, could to > know what confuses users and what makes PA harder to use. > > Bug report link: https://bugs.launchpad.net/bugs/663019 > > = Comment from B Bobo begins here === […] I agree that it is good to get this user feedback. Please thank B Bobo for taking the time to write this down. Could you ask him, if he is willing to provide a patch with the changes he can do. $ git config --global user.name "Real name" $ git config --global user.email b.b...@example.org $ git clone git://git.0pointer.de/pulseaudio.git $ cd pulseaudio $ gedit *files for manpages* $ git commit -a $ git format-patch -n # n = number of commits $ # either send those files to the mailing list http://pulseaudio.org/wiki/Community#MailingLists $ # or use git send-email Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH 2/2] man/pulse-daemon.conf.5: Fix sentence.
Date: Sun, 17 Oct 2010 18:51:18 +0200 This section was added in commit 58bdd9 [1]. [1] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=58bdd97260b8acdc07bedd4cc3112786cb43ddbe Signed-off-by: Paul Menzel --- man/pulse-daemon.conf.5.xml.in |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in index e0ac038..96bec7b 100644 --- a/man/pulse-daemon.conf.5.xml.in +++ b/man/pulse-daemon.conf.5.xml.in @@ -447,8 +447,8 @@ USA. cause the resulting output sound to be momentarily too loud or too soft. So to ensure SW and HW volumes are applied concurrently without any glitches, their application needs to be synchronized. The sink -implementation needs to support synchronized volumes can use the following -parameters to refine the process. +implementation needs to support synchronized volumes. The following +parameters can be used to refine the process. enable-sync-volume= Enable sync volume for the sinks that -- 1.7.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH 1/2] man/pulse-daemon.conf.5.xml.in: Typo and uniform spelling. s/synchornised/synchronized/
Date: Sun, 17 Oct 2010 19:28:27 +0200 Signed-off-by: Paul Menzel --- man/pulse-daemon.conf.5.xml.in |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in index b3c3e5a..e0ac038 100644 --- a/man/pulse-daemon.conf.5.xml.in +++ b/man/pulse-daemon.conf.5.xml.in @@ -447,7 +447,7 @@ USA. cause the resulting output sound to be momentarily too loud or too soft. So to ensure SW and HW volumes are applied concurrently without any glitches, their application needs to be synchronized. The sink -implementation needs to support synchornised volumes can use the following +implementation needs to support synchronized volumes can use the following parameters to refine the process. -- 1.7.1 signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH] alsa-mixer: add profile for Native Instruments Korecontroller
Dear Daniel, Am Freitag, den 15.10.2010, 18:18 +0200 schrieb Daniel Mack: > --- > src/Makefile.am|3 +- > .../alsa/mixer/profile-sets/90-pulseaudio.rules|3 + > .../native-instruments-korecontroller.conf | 85 > > 3 files changed, 90 insertions(+), 1 deletions(-) > create mode 100644 > src/modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf > > diff --git a/src/Makefile.am b/src/Makefile.am > index e55c31c..87f1178 100644 > --- a/src/Makefile.am > +++ b/src/Makefile.am > @@ -1100,7 +1100,8 @@ dist_alsaprofilesets_DATA = \ > modules/alsa/mixer/profile-sets/maudio-fasttrack-pro.conf \ > > modules/alsa/mixer/profile-sets/native-instruments-audio4dj.conf \ > > modules/alsa/mixer/profile-sets/native-instruments-audio8dj.conf \ > - > modules/alsa/mixer/profile-sets/native-instruments-traktorkontrol-s4.conf > + > modules/alsa/mixer/profile-sets/native-instruments-traktorkontrol-s4.conf \ > + > modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf > > if HAVE_UDEV > dist_udevrules_DATA = \ > diff --git a/src/modules/alsa/mixer/profile-sets/90-pulseaudio.rules > b/src/modules/alsa/mixer/profile-sets/90-pulseaudio.rules > index 7a425b5..2c3821b 100644 > --- a/src/modules/alsa/mixer/profile-sets/90-pulseaudio.rules > +++ b/src/modules/alsa/mixer/profile-sets/90-pulseaudio.rules > @@ -22,7 +22,10 @@ KERNEL!="card*", GOTO="pulseaudio_end" > > SUBSYSTEMS=="usb", ATTRS{idVendor}=="17cc", ATTRS{idProduct}=="1978", > ENV{PULSE_PROFILE_SET}="native-instruments-audio8dj.conf" > SUBSYSTEMS=="usb", ATTRS{idVendor}=="17cc", ATTRS{idProduct}=="0839", > ENV{PULSE_PROFILE_SET}="native-instruments-audio4dj.conf" > +SUBSYSTEMS=="usb", ATTRS{idVendor}=="17cc", ATTRS{idProduct}=="1010", > ENV{PULSE_PROFILE_SET}="native-instruments-audio6dj.conf" > +SUBSYSTEMS=="usb", ATTRS{idVendor}=="17cc", ATTRS{idProduct}=="1021", > ENV{PULSE_PROFILE_SET}="native-instruments-audio10dj.conf" Did this slip in by accident? I do not see why this got added. > SUBSYSTEMS=="usb", ATTRS{idVendor}=="17cc", ATTRS{idProduct}=="baff", > ENV{PULSE_PROFILE_SET}="native-instruments-traktorkontrol-s4.conf" > +SUBSYSTEMS=="usb", ATTRS{idVendor}=="17cc", ATTRS{idProduct}=="4711", > ENV{PULSE_PROFILE_SET}="native-instruments-korecontroller.conf" > SUBSYSTEMS=="usb", ATTRS{idVendor}=="0763", ATTRS{idProduct}=="2012", > ENV{PULSE_PROFILE_SET}="maudio-fasttrack-pro.conf" > > LABEL="pulseaudio_end" > diff --git > a/src/modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf > b/src/modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf > new file mode 100644 > index 000..904357d > --- /dev/null > +++ > b/src/modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf […] Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH 5/5] man: sync_volume parameters to manual page
Dear Colin, Am Donnerstag, den 14.10.2010, 20:30 +0100 schrieb Colin Guthrie: > Subject: [PATCH] man: sync_volume parameters to manual page > > Some wording teaks by Colin Guthrie > > Signed-off-by: Jyri Sarha > Reviewed-by: Tanu Kaskinen > --- > man/pulse-daemon.conf.5.xml.in | 26 ++ > 1 files changed, 26 insertions(+), 0 deletions(-) > > diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in > index b5c9e92..f0a642b 100644 > --- a/man/pulse-daemon.conf.5.xml.in > +++ b/man/pulse-daemon.conf.5.xml.in > @@ -437,6 +437,32 @@ USA. > > > > + > + > +The flat volume feature sets the sink HW volume according to the > +highest volume input stream and reduces the volume any other streams > +(with lower volumes) in SW. To make this concurrent and ensure SW and > +HW volume work together without any glitches, their application needs to > +be synchronized. The sink implementation needs to support this > +feature in order to these parameters to take effect. > + > + > + enable-sync-volume= Enable sync volume for the sinks that > + support it. This feature is enabled by default. > + > + > + sync-volume-safety-margin-usec= The unit of time by which > + the HW volume increases are delayed and HW volume decreases are > advanced. > + Defaults to 8000 usec. I am not sure, but I would read it that I have to give the unit of time, i. e., seconds, hours, milliseconds, …, as a value. So I think the former wording is more correct. Since the unit of time is `usecs`. > + > + > + sync-volume-extra-delay-usec= The unit of time all HW > + volume changes are delayed. Negative values are also allowed. > + Defaults to 0. > + > + > + I am still thinking that a „normal“ users does not have a clue on how to determine to what value these options should be set. So it should be marked as expert value or something or a hint should be given on how one can determine what values should be used. > + > > The PulseAudio Developers <@PACKAGE_BUGREPORT@>; PulseAudio is > available from > Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] Wanted: native English speakers for review: [PATCH 5/5] man: sync_volume parameters to manual page
Am Mittwoch, den 13.10.2010, 19:40 +0300 schrieb o...@iki.fi: > From: Jyri Sarha > > Signed-off-by: Jyri Sarha > Reviewed-by: Tanu Kaskinen It would be great if some native English speakers could take a look at the following sentences. Maybe David (CCed) has some time too. I am giving it a shot too, but I do not know if everything is correct. > --- > man/pulse-daemon.conf.5.xml.in | 26 ++ > 1 files changed, 26 insertions(+), 0 deletions(-) > > diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in > index b5c9e92..ccc2db7 100644 > --- a/man/pulse-daemon.conf.5.xml.in > +++ b/man/pulse-daemon.conf.5.xml.in > @@ -437,6 +437,32 @@ USA. > > > > + > + > +The flat volume feature sets the sink HW volume according to according to *the* > +highest volume input stream and lowers the lower volume streams > +with SW volume. To make this concurrent use of SW and HW volume to > +work without a glitch their application needs to be > +synchronized. The sink implementation needs to support this > +feature in order to these parameters to take effect. > + > + > + enable-sync-volume= Enable sync volume for the sinks that > + support it. This feature is enabled by default. > + > + > + sync-volume-safety-margin-usec= The amount usecs the HW amount *of* usecs > + volume increases are delayed and HW volume decreases are advanced. > Defaults As someone who does not have a clue, could that be a little bit more elaborate explained. Why should increases be delayed? Could an example be give and would it be useful? > + to 8000 usec. > + > + > + sync-volume-extra-delay-usec= The amount usecs the The amount *of* usesc > + all HW volume changes are delayed. Negative values are also allowed. »the all«? > + Defaults to 0. > + > + > + > + > > The PulseAudio Developers <@PACKAGE_BUGREPORT@>; PulseAudio is > available from > Thanks, Paul signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [PATCH] Two identical USB sound-cards - avahi fails to add service.
Dear Ivar, Am Donnerstag, den 09.09.2010, 20:20 +0200 schrieb Ivar Mossin: […] thank you for your patch. Could you please send the patch using the Git tools (`git format-patch -s -1` (and maybe) `git send-email`). This way developers do not need to copy paste your commit message from your messages but just use the tools to get this done automatically. I do not know if a Signed-off-by line is required though. Thanks, Paul PS: And please also just send plain text messages. signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] How to save mixer settings?
Dear Jan, Am Montag, den 06.09.2010, 09:51 +0200 schrieb Jan Kasprzak: > On my laptop the "Front mic" input is routed to the speakers/headphones > after boot, causing unnecessary noise and sometimes even a feedback loop. > I can mute it using "amixer -D hw:0 set 'Front mic' mute", but it becomes > unmuted every time I suspend and resume the laptop, or every time I log in. > > Is it possible to save the mixer settings w/ pulse? I am sorry, I cannot answer your question. But I guess PA is not responsible for that and this should be ALSA’s responsibility. > My laptop is ASUS F3e with intel HDA sound with Realtek ALC660-VD codec chip > (snd-hda-intel module is loaded with "model=lenovo" switch because of > an unrelated problem described here: > http://www.fi.muni.cz/~kas/blog/index.cgi/computers/intel-hda-headphone-speakers.html > ). 1. Did you report that issue to the alsa-devel list [1][2]? 2. Your issue is from September 2008. What ALSA version did you use back then and what version do you use now? 3. Have you tried more recent ALSA versions? I think a lot of fixes have been made. 4. Since you have an ASUS system there might be a better value for `model`? 5. If nothing helps and the problem is still present, maybe HDA-Analyzer [3] can help you to fix it correctly. Thanks, Paul [1] http://mailman.alsa-project.org/mailman/listinfo/alsa-devel [2] http://alsa-project.org/main/index.php/Help_To_Debug [3] http://alsa-project.org/main/index.php/Help_To_Debug_Intel_HDA signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Volume control interactions
Am Sonntag, den 15.08.2010, 17:03 +0100 schrieb Dylan: > It seems to me that the behaviour of various volume controls and their > interactions is wrong. Let me explain ... Have you already read [1]? […] Thanks, Paul [1] http://pulseaudio.org/wiki/PulseAudioStoleMyVolumes signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] translation and transifex: Trying to fix a typo
Dear PA folks, I noticed a typo in the German translation and read [1], so instead of just sending a patch to this list, I registered at Transifex [2] and tried to submit the fix. Since trying to edit the translation using the Transifex Web interface did show to me the already translated strings, I went ahead, checked out the branch `master-tx`, edited `de.po` and chose at the overview »Submit a translation for po/de« and selected my edited (s/hochauslösend/hochauflösend/) file. But looking at the commit [3] I noticed that only meta data changed and my actual fix (s/hochauslösend/hochauflösend/) did not apply. I am pretty sure I saved the file. Still I am confused what happened. Can someone please advice me how to go on from here. Is there a way to revert that? Thanks, Paul [1] http://pulseaudio.org/wiki/Community#BugsPatchesTranslations [2] http://www.transifex.net/ [3] http://git.0pointer.de/?p=pulseaudio.git;a=blobdiff;f=po/de.po;h=38a50062a5249a65c1c1d5c9210b742d818fe614;hp=d08a3fa439d9d371aa6f74fb4eb050673fb9;hb=88681bd93d7bba693cbf6ff2e6380ead5aaa8901;hpb=2c888d747cdc1cec4200f27d59c4925264ee125e signature.asc Description: This is a digitally signed message part ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] 0.9.22 on Ubuntu Lucid gobbling up CPU
Am Donnerstag, den 13.05.2010, 20:57 + schrieb Brian J.Murrell: > I was so hoping that the PA on Ubuntu Lucid was going to be much more stable > than previous releases but alas, I don't think it's to be. Currently > installed > on a machine here is 0.9.22~0.9.21+stable-queue-32-g8478-0ubuntu14 which > presumably is some form of 0.9.22. > > I have it currently running with "-" due to the instability and desire to > at least log what it's doing. As such, I can provide a full log upon request. > > But currently, at this moment, there is not even any audio running through it > and it's using 90% of a pretty modern (and hence, somewhat beefy) processor. > > The only thing the log is showing currently, while this is happening: > > D: module-rtp-recv.c: Checking for dead streams ... > W: ratelimit.c: 3267 events suppressed > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > D: alsa-source.c: Wakeup from ALSA! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > W: ratelimit.c: 2993 events suppressed > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > D: alsa-source.c: Wakeup from ALSA! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > W: ratelimit.c: 3302 events suppressed > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > I: alsa-source.c: Overrun! > > The hardware in this machine is: > > $ lspci | grep -i audio > 04:01.0 Audio device: VIA Technologies, Inc. VT1708/A [Azalia HDAC] (VIA High > Definition Audio Controller) (rev 10) > 05:05.0 Multimedia audio controller: Ensoniq ES1371 [AudioPCI-97] (rev 07) > > $ cat /proc/asound/cards > 0 [VT82xx ]: HDA-Intel - HDA VIA VT82xx > HDA VIA VT82xx at 0xdfafc000 irq 17 > 1 [AudioPCI ]: ENS1371 - Ensoniq AudioPCI > Ensoniq AudioPCI ENS1371 at 0xaf00, irq 18 > $ uname -r > 2.6.32-22-generic > > Again, a full log is available if desired. > > Thots? You already posted the same question to the list four weeks ago [1]. Please keep the threading next time. Also you did not answer my questions back then. Thanks, Paul [1] https://tango.0pointer.de/pipermail/pulseaudio-discuss/2010-April/006857.html signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Pulse audio does not work with my hdmi output
Am Samstag, den 17.04.2010, 11:00 -0600 schrieb Dark Shadow: > I could not get a pulseaudio -vvv because every time I killed it it > re-spawned on it's own before I could start it. Did you search for »pulseaudio respawn« on the Web? You would have found [1]. Anyway the term used in PA seems to be »autospawn«. Reading `man pulse-client.conf` is also informational. (Although finding that is a little harder, because it is not referenced by `man pulseaudio`.) [2] and [3] in the PA Wiki are also good to read when debugging a problem. […] Thanks, Paul [1] https://fedoraproject.org/wiki/How_to_debug_PulseAudio_problems [2] http://pulseaudio.org/wiki/Community#BugsPatchesTranslations [3] http://pulseaudio.org/wiki/Troubleshooting signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] using near 100% CPU when doing nothing
Am Samstag, den 10.04.2010, 19:55 + schrieb Brian J.Murrell: > I have a(one of a number of)Ubuntu Karmic systems here running pulseaudio > 0.9.19. On this particular system, PA wants to eat as much CPU as is > available. It will use up near 100% on this system when it's otherwise idle. Did you try to record something or use the microphone? > This particular machine is the only one of several, as I say, with this > problem > and they are all up-to-date, so I wonder if this is particular to the sound > hardware in this machine. It has in it: > > 04:01.0 Audio device: VIA Technologies, Inc. VT1708/A [Azalia HDAC] (VIA High > Definition Audio Controller) (rev 10) > 05:05.0 Multimedia audio controller: Ensoniq ES1371 [AudioPCI-97] (rev 07) > > What other information can I gather to help diagnose this problem? Maybe it is related to ticket #758 [1] which commit [2][3][4] in ALSA fixed for me kind of. What CODEC chip do you have in your system? Thanks, Paul [1] http://pulseaudio.org/ticket/758 [2] http://mailman.alsa-project.org/pipermail/alsa-devel/2010-February/thread.html#25109 [3] http://git.kernel.org/?p=linux/kernel/git/tiwai/sound-2.6.git;a=commit;h=0708cc582f0fe2578eaab722841caf2b4f8cfe37 [4] http://sourceforge.net/mailarchive/forum.php?thread_name=1265550675.4642.24.camel%40mattotaupa&forum_name=alsa-user signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Pulseaudio and Skype
Dear Mark, Am Sonntag, den 14.03.2010, 09:56 + schrieb Mark Greenwood: […] > I'm running Skype 2.1 beta and Pulseaudio 0.9.19 on Kubuntu 9.10. […] > Any ideas on how to go about making it work? have you read [1] which is referenced by [2]. But before you report back with the debug information, could you please try the current release 0.9.21 or the one from Git (master or stable-queue). It should not be too hard to build that packages with `git-buildpackage` or `dpkg-buildpackage`. Thanks, Paul [1] http://pulseaudio.org/wiki/UbuntuBugs [2] http://pulseaudio.org/wiki/Community#BugsPatchesTranslations signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Frequent trivially-reproducible client hangs with stable-queue PA
Am Montag, den 22.02.2010, 18:50 + schrieb Nix: > Ever since I started using PA 0.9.21 (and possibly before: it's been > long enough that my memory has faded), lots of applications have started > hanging hard when fast-forwarded/rewound. This includes MPD and every > single video player I own capable of fullscreen playback. As you might > imagine, having to stop PA playing back sound and forcing all my video > and music players not to use PA renders it... less useful than it might > be, and the constant need to pay attention to ALSA volume levels makes > PA's otherwise admirable ALSA volume-level frotzing annoying as > well. (I'd use the ALSA->PA plugin, but as reported earlier, this > doesn't work either, stuttering like crazy and frequently locking up. > It may be that the ALSA plugin lockup is a special case of this one.) > > It seems unlikely to me that every one of these applications is buggy: > the problem more probably lies in PA itself. > > This is still reproducible with stable-queue HEAD. > > In all cases they seem to get stuck waiting for PA to send something > that it never sends (or, at least, all are stuck in poll()...) > > Examples, with client backtraces, include: […] Is there anything in the log files [1]? Thanks, Paul [1] http://pulseaudio.org/wiki/Community#BugsPatchesTranslations signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] How to get pulseaudio streaming to PS3?
Dear Nyall, Am Sonntag, den 07.02.2010, 16:35 +1100 schrieb Nyall Dawson: […] > WIth rygel installed I > see an option to change the default sink to DLNA/Upnp Streaming, where do you see that option? By the way, the Rygel start page is here [5]. > but > setting this option doesn't give me any audio from the ps3. Am I > missing something here or is there additional setup I should do to get > this working? Searching in the PulseAudio Wiki [1] for »rygel« there are three hits suggesting it is supported [2–4]. Please provide more information to us. For example what versions do you use? Can you please provide some log files [6]? Thanks, Paul [1] http://pulseaudio.org/search?q=rygel [2] http://pulseaudio.org/milestone/0.9.21 [3] http://pulseaudio.org/milestone/0.9.16 [4] http://pulseaudio.org/wiki/WhatIsWrongWithSystemMode [5] http://live.gnome.org/Rygel [6] http://pulseaudio.org/wiki/Community#BugsPatchesTranslations signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] [module-console-kit.c] SessionAdded/SessionRemoved messages every 5 seconds with `log-level = debug`
Am Dienstag, den 02.02.2010, 08:54 +0100 schrieb Lennart Poettering: > On Sun, 31.01.10 12:31, Paul Menzel (paulepan...@users.sourceforge.net) wrote: > > > If I understood the code correctly, it looks like the CK-module is > > initialized every five seconds on my system. Is that expected/normal? > > No. This is not what happens. The filter stays in place the entire > runtime. What apparently happens is that a CK session is set up and > teared down every 5s on your system. dbus-monitor might help you > figure out what session that actually is and what is going on. Thank you for your answer. I reported this issue to the Debian BTS [1]. Thanks, Paul [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=568076 signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] No or only random connections to PulseAudio server after `pulseaudio -k`.
Am Dienstag, den 02.02.2010, 09:42 + schrieb Colin Guthrie: > 'Twas brillig, and Lennart Poettering at 02/02/10 07:58 did gyre and gimble: > > On Sat, 30.01.10 15:59, Paul Menzel (paulepan...@users.sourceforge.net) > > wrote: […] > >> D: module-udev-detect.c: /dev/snd/controlC0 is accessible: yes > >> D: module-console-kit.c: dbus: > >> interface=org.freedesktop.ConsoleKit.Seat, > >> path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded > >> D: module-console-kit.c: dbus: > >> interface=org.freedesktop.ConsoleKit.Seat, > >> path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved > >> D: module-console-kit.c: dbus: > >> interface=org.freedesktop.ConsoleKit.Seat, > >> path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded > > > > There's something seriously wrong with your CK setup. > > FWIW, there is more debug regarding this issue in the ticket Paul opened > for it here: > http://pulseaudio.org/ticket/785 > > I tried to deconstruct the connection failures, but I'm failing. Please do not spend anymore time with this issue. I think it was a stupid configuration error on my site and then making wrong assumptions afterward. I am updating the report just now. > I suspect that the problem is in some way related to the consolekit stuff > as you point out. I reported the ConsoleKit error to the Debian BTS [1]. Thanks, Paul [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=568076 signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Pulseaudio spinning somewhere - should I report?
Dear Clemens, Am Sonntag, den 31.01.2010, 17:23 -0500 schrieb Clemens Eisserer: > After playing Tuxracer I found the pulseaudio-daemon was consuming 100% cpu. > I did some stack-traces and posted it here: http://pastebin.com/f7ffb6cfd > > I am running Fedora-12+latest updates. > > Is this a known problem, or should I open a bug-report? I do not know if this problem is known, but I suggest to open a bug report at Fedora or upstream, where you have to register [3]. Take a look at [1] or [2] where you find some information regarding such problems. Thanks, Paul [1] http://fedoraproject.org/wiki/How_to_debug_PulseAudio_problems [2] http://pulseaudio.org/wiki/HowToDebugCPULoadBugs [3] http://pulseaudio.org/ signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
Re: [pulseaudio-discuss] Local and remote sink
Dear Samuel, Am Samstag, den 30.01.2010, 18:57 +0100 schrieb samuel: […] > Actually I would like it to check if tarzanito is available, > if yes, use it, if not play locally. $ man pulse-client.conf […] default-server= The default sever to connect to. The environment vari‐ able $PULSE_SERVER takes precedence. Maybe you can check if tarzanito is available and set the environment variable accordingly during startup? > Also, suppose I would like to play > locally right now, I'd switch (but not changing the config file, but > with a quick command, a la pacmd or pactl...) preferrably using a > keyboard shortcut. (If I have pulseaudio set up, I'd know how to get > the shortcut) > > My question: Do I have to add sinks for tarzanito and my local alsa, > and if yes, how exactly do I do that? I tried messing with the > default.pa script, but everytime I change something, it gets messed > up... Sorry I have no ideas for this. Thanks, Paul signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [PATCH] man pages: correct formatting/markup of options
Signed-off-by: Paul Menzel --- man/pulse-client.conf.5.xml.in |2 +- man/pulse-daemon.conf.5.xml.in | 10 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/man/pulse-client.conf.5.xml.in b/man/pulse-client.conf.5.xml.in index c2a847f..739fae4 100644 --- a/man/pulse-client.conf.5.xml.in +++ b/man/pulse-client.conf.5.xml.in @@ -69,7 +69,7 @@ USA. autospawn= Autospawn a PulseAudio daemon when - needed. Takes a boolean value, defaults to "yes". + needed. Takes a boolean value, defaults to yes. diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in index e6b1e19..b5c9e92 100644 --- a/man/pulse-daemon.conf.5.xml.in +++ b/man/pulse-daemon.conf.5.xml.in @@ -53,14 +53,14 @@ USA. daemonize= Daemonize after startup. Takes a - boolean value, defaults to "no". The --daemonize + boolean value, defaults to no. The --daemonize command line option takes precedence. fail= Fail to start up if any of the directives in the configuration script default.pa - fail. Takes a boolean argument, defaults to "yes". The --fail command line + fail. Takes a boolean argument, defaults to yes. The --fail command line option takes precedence. @@ -191,7 +191,7 @@ USA. RLIMIT_NICE is used. root is dropped immediately after gaining the nice level on startup, thus it is presumably safe. See for more - information. Takes a boolean argument, defaults to "yes". The --high-priority + information. Takes a boolean argument, defaults to yes. The --high-priority command line option takes precedence. @@ -205,7 +205,7 @@ USA. real-time. The controlling thread is left a normally scheduled thread. Thus enabling the high-priority option is orthogonal. See for more - information. Takes a boolean argument, defaults to "yes". The + information. Takes a boolean argument, defaults to yes. The --realtime command line option takes precedence. @@ -215,7 +215,7 @@ USA. by default, 9 for clients. Thus it is recommended to choose the PulseAudio real-time priorities lower. Some PulseAudio threads might choose a priority a little lower or higher than the - specified value. Defaults to "5". + specified value. Defaults to 5. -- 1.6.6.1 signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] [module-console-kit.c] SessionAdded/SessionRemoved messages every 5 seconds with `log-level = debug`
Dear PulseAudio users, I know that setting `log-level = debug` should increase the messages written out to `log-target`. I noticed that the following messages are printed every five seconds to `log-target`. […] Jan 31 11:47:21 x pulseaudio[3489]: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded Jan 31 11:47:21 x pulseaudio[3489]: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved Jan 31 11:47:26 x pulseaudio[3489]: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded Jan 31 11:47:26 x pulseaudio[3489]: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved […] In `module-console-kit.c` [1] the module gets initialized, and the D-Bus connection gets a `filter_cb` added to it. int pa__init(pa_module*m) { […] if (!dbus_connection_add_filter(pa_dbus_connection_get(connection), filter_cb, u, NULL)) { pa_log_error("Failed to add filter function"); goto fail; } if (pa_dbus_add_matches( pa_dbus_connection_get(connection), &error, "type='signal',sender='org.freedesktop.ConsoleKit',interface='org.freedesktop.ConsoleKit.Seat',member='SessionAdded'", "type='signal',sender='org.freedesktop.ConsoleKit',interface='org.freedesktop.ConsoleKit.Seat',member='SessionRemoved'", NULL) < 0) { pa_log_error("Unable to subscribe to ConsoleKit signals: %s: %s", error.name, error.message); goto fail; } […] Looking at `filter_cb` there the message seems to be printed by `pa_log_debug(…)`. static DBusHandlerResult filter_cb(DBusConnection *bus, DBusMessage *message, void *userdata) { […] pa_log_debug("dbus: interface=%s, path=%s, member=%s\n", dbus_message_get_interface(message), dbus_message_get_path(message), dbus_message_get_member(message)); if (dbus_message_is_signal(message, "org.freedesktop.ConsoleKit.Seat", "SessionAdded")) { […] } add_session(u, path); } else if (dbus_message_is_signal(message, "org.freedesktop.ConsoleKit.Seat", "SessionRemoved")) { […] remove_session(u, path); } […] } If I understood the code correctly, it looks like the CK-module is initialized every five seconds on my system. Is that expected/normal? Thanks, Paul [1] http://git.0pointer.de/?p=pulseaudio.git;a=blob;f=src/modules/module-console-kit.c signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
[pulseaudio-discuss] No or only random connections to PulseAudio server after `pulseaudio -k`.
Dear PulseAudio folks, I want to debug PulseAudio problems as described in [1]. Therefore I added `autospawn = no` to `~/.pulse/client.conf` and issued `pulseaudio -k` and `pulseaudio -v`. But after doing so all my programs are not able to connect to the PulseAudio server(?) anymore. Putting `autospawn = yes` to `~/.pulse/client.conf` produces the same behavior besides with MPlayer, where it is sometimes able to connect to the server but fails in over 90 % of the attempts doing so. 1. For example running `gstreamer-properties` which has PulseAudio selected as output device gives an error message. $ LANG=C gstreamer-properties gstreamer-properties-Message: Error running pipeline 'PulseAudio Sound Server': Failed to connect: Connection refused [pulsesink.c(442): gst_pulseringbuffer_open_device (): /GstPipeline:pipeline0/GstPulseSink:pulsesink3] 2. Also MPlayer does not work anymore with PulseAudio as output. AO: [pulse] Init failed: Connection refused Failed to initialize audio driver 'pulse' Could not open/initialize audio device -> no sound. Audio: no sound Killing PulseAudio with `autospawn = yes` and feeding for example 12 audio files into MPlayer, then maybe half of the time running `mplayer -ao pulse *` is able to play a song at the end of the list (e. g. 9th song) and fails for the following songs again. Here is a excerpt from the debug messages. The whole can be found as an attachment for the ticket I filed [2] because of the size restrictions for attachments for this list. $ LANG=C pulseaudio -v 2> 20100130--pa-v.log […] D: module-udev-detect.c: /dev/snd/controlC0 is accessible: yes D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved D: alsa-sink.c: Wakeup from ALSA! I: module-suspend-on-idle.c: Source alsa_input.pci-_20_01.0.analog-stereo idle for too long, suspending ... D: source.c: Suspend cause of source alsa_input.pci-_20_01.0.analog-stereo is 0x0004, suspending I: alsa-source.c: Device suspended... I: module-suspend-on-idle.c: Sink alsa_output.pci-_20_01.0.analog-stereo idle for too long, suspending ... D: sink.c: Suspend cause of sink alsa_output.pci-_20_01.0.analog-stereo is 0x0004, suspending I: alsa-sink.c: Device suspended... D: reserve-wrap.c: Device lock status of reserve-monitor-wrap...@audio0 changed: not busy D: module-udev-detect.c: /dev/snd/controlC0 is accessible: yes D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved D: module-console-kit.c: dbus: interface=org.freedesktop.ConsoleKit.Seat, path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded D: module-console-kit.c: dbus: interface=org.freed
[pulseaudio-discuss] [PATCH] client.conf.in: Typo. s/a/are/
This is the same typo as reported in [1] and fixed in [2] in `client.conf.in`. I did `git grep "values a commented"` and did not get anymore hits. [1] http://pulseaudio.org/ticket/783 [2] http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=ea8927f1d47be101a1f64b2f87fd6e527ff76dbe Signed-off-by: Paul Menzel --- src/pulse/client.conf.in |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/pulse/client.conf.in b/src/pulse/client.conf.in index 436747b..17753b0 100644 --- a/src/pulse/client.conf.in +++ b/src/pulse/client.conf.in @@ -16,7 +16,7 @@ # USA. ## Configuration file for PulseAudio clients. See pulse-client.conf(5) for -## more information. Default values a commented out. Use either ; or # for +## more information. Default values are commented out. Use either ; or # for ## commenting. ; default-sink = -- 1.6.6.1 signature.asc Description: Dies ist ein digital signierter Nachrichtenteil ___ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss