summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
| * | sbc: ARM NEON optimization for scale factors calculationSiarhei Siamashka2011-03-142-1/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improves SBC encoding performance when joint stereo is not used. Benchmarked on ARM Cortex-A8: == Before: == $ time ./sbcenc -b53 -s8 test.au > /dev/null real 0m4.756s user 0m4.313s sys 0m0.438s samples % image name symbol name 2569 27.6296 sbcenc sbc_pack_frame 1934 20.8002 sbcenc sbc_analyze_4b_8s_neon 1386 14.9064 sbcenc sbc_calculate_bits 1221 13.1319 sbcenc sbc_calc_scalefactors 996 10.7120 sbcenc sbc_enc_process_input_8s_be 878 9.4429 no-vmlinux /no-vmlinux 204 2.1940 sbcenc sbc_encode 56 0.6023 libc-2.10.1.so memcpy == After: == $ time ./sbcenc -b53 -s8 test.au > /dev/null real 0m4.220s user 0m3.797s sys 0m0.422s samples % image name symbol name 2563 31.3249 sbcenc sbc_pack_frame 1892 23.1239 sbcenc sbc_analyze_4b_8s_neon 1368 16.7196 sbcenc sbc_calculate_bits 961 11.7453 sbcenc sbc_enc_process_input_8s_be 836 10.2176 no-vmlinux /no-vmlinux 262 3.2022 sbcenc sbc_calc_scalefactors_neon 199 2.4322 sbcenc sbc_encode 49 0.5989 libc-2.10.1.so memcpy
| * | sbc: MMX optimization for scale factors calculationSiarhei Siamashka2011-03-141-0/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improves SBC encoding performance when joint stereo is not used. Benchmarked on Pentium-M: == Before: == $ time ./sbcenc -b53 -s8 test.au > /dev/null real 0m1.439s user 0m1.336s sys 0m0.104s samples % image name symbol name 8642 33.7473 sbcenc sbc_pack_frame 5873 22.9342 sbcenc sbc_analyze_4b_8s_mmx 4435 17.3188 sbcenc sbc_calc_scalefactors 4285 16.7331 sbcenc sbc_calculate_bits 1942 7.5836 sbcenc sbc_enc_process_input_8s_be 322 1.2574 sbcenc sbc_encode == After: == $ time ./sbcenc -b53 -s8 test.au > /dev/null real 0m1.319s user 0m1.220s sys 0m0.084s samples % image name symbol name 8706 37.9959 sbcenc sbc_pack_frame 5740 25.0513 sbcenc sbc_analyze_4b_8s_mmx 4307 18.7972 sbcenc sbc_calculate_bits 1937 8.4537 sbcenc sbc_enc_process_input_8s_be 1801 7.8602 sbcenc sbc_calc_scalefactors_mmx 307 1.3399 sbcenc sbc_encode
| * | sbc: new 'sbc_calc_scalefactors_j' function added to sbc primitivesSiarhei Siamashka2011-03-143-68/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The code for scale factors calculation with joint stereo support has been moved to a separate function. It can get platform-specific SIMD optimizations later for best possible performance. But even this change in C code improves performance because of the use of __builtin_clz() instead of loops similar to what was done to sbc_calc_scalefactors earlier. Also technically it does loop unrolling by processing two channels at once, which might be either good or bad for performance (if the registers pressure is increased and more data is spilled to memory). But the benchmark from 32-bit x86 system (pentium-m) shows that it got clearly faster: $ time ./sbcenc.old -b53 -s8 -j test.au > /dev/null real 0m1.868s user 0m1.808s sys 0m0.048s $ time ./sbcenc.new -b53 -s8 -j test.au > /dev/null real 0m1.742s user 0m1.668s sys 0m0.064s
| * | sbc: Fix redundant null check on calling free()Gustavo F. Padovan2011-03-141-2/+1
| | | | | | | | | | | | Issues found by smatch static check: http://smatch.sourceforge.net/
| * | sbc: added saturated clipping of decoder output to 16-bitSiarhei Siamashka2011-03-141-5/+15
| | | | | | | | | | | | | | | | | | This prevents overflows and audible artefacts for the audio files which originally had loudness maximized. Music from audio CD disks is an example of such files, see http://en.wikipedia.org/wiki/Loudness_war
| * | sbc: ensure 16-byte buffer position alignment for 4 subbands encodingSiarhei Siamashka2011-03-142-4/+4
| | | | | | | | | | | | | | | | | | | | | Buffer position in X array was not always 16-bytes aligned. Strict 16-byte alignment is strictly required for powerpc altivec simd optimizations because altivec does not have support for unaligned vector loads at all.
| * | build: move sbc related files to its own directoryLuiz Augusto von Dentz2011-03-1412-2/+2
| | | | | | | | | | | | | | | This should make it easier to apply patches from BlueZ which also uses sbc subdir for this files.
| * | bluetooth: add proper handling for bluetooth.nrec propertyLuiz Augusto von Dentz2011-03-143-48/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | NREC stands for Noise Reduction and Echo Cancelation, it can be changed at any point by the headset. When set to "1" indicates that those algorithms shall be enabled by default and "0" means the headset probably have them active so they should be disabled in PA side.
| * | bluetooth: fix a2dp_process_pushLuiz Augusto von Dentz2011-03-141-6/+12
| | | | | | | | | | | | | | | | | | Use minimum bitpool configured to get the maximum block_size possible, also remove checks for how much has been written when decoding sbc frames since the block size may change due to bitpool changes.
| * | bluetooth: reduce bitpool if audio start skippingLuiz Augusto von Dentz2011-03-141-0/+71
| | | | | | | | | | | | | | | | | | | | | When audio skips it could be that there is some bandwidth limitation in the link e.g. headset doesn't support EDR (< 2.0), and by reducing the bitpool it may find a better rate that either prevent the skips completely or at least reduce them.
| * | bluetooth: handle Acquire API changeLuiz Augusto von Dentz2011-03-143-47/+12
| | | | | | | | | | | | | | | | | | Acquire now return input and output MTU of the file descriptor so it is no longer necessary to get those after acquiring the fd, which less round trips and faster response time when switching profiles.
* | | build-sys: No need to create folder for echo-cancel module.Colin Guthrie2011-03-141-1/+1
|/ / | | | | | | | | | | | | The ORCC stage does this anyway (and this was buggy anyway as it had an extra 'src/' prefix so never worked properly when it was introduced in d6cdd80). Thanks to Maarten Bosmans for pointing it out.
* | cork-on-phone: Only cork (and subsequently uncork) streams that are not ↵Colin Guthrie2011-03-121-8/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | already corked. Although by "cork" I really mean "cork+mute" as that's what the module does. If e.g. Rhythmbox is paused when a phone call comes in, the current stream state will be corked and thus we should not track it for future uncorking when the phone call ends. Likewise if the stream is just muted (manually) we will not take any action either when the phone stream is seen first, nor when it disappears. Also add some additional debug messages.
* | build: Use silent rules for generating filesMaarten Bosmans2011-03-111-10/+10
| |
* | Fixup #include directives according to Coding StyleMaarten Bosmans2011-03-1121-41/+32
| | | | | | | | | | Use #include "header.h" if functionality of header.h is implemented and #include <header.h> if functionality of header.h is used.
* | Fix up according to Coding StyleMaarten Bosmans2011-03-1160-424/+303
| | | | | | | | Only whitespace changes in here
* | alsa-mixer: When figuring out the max_dB of a path, use only channels that ↵Tanu Kaskinen2011-03-111-4/+9
| | | | | | | | | | | | | | | | | | | | | | are used by the path elements. Without this, p->max_dB could never be less than 0 dB, because the loop at the end of pa_alsa_path_probe() would reset p->max_dB to 0 as soon as the loop encountered a channel that wasn't touched by any element. There was a similar issue for p->min_dB too (it could never be more than 0 dB), which is also fixed by this patch.
* | alsa-mixer: Implement support for setting element specific upper limits for ↵Tanu Kaskinen2011-03-113-1/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | volume. This feature is mainly useful in embedded systems that have built-in speakers. In such situations the full audio path is known beforehand, so it's possible to know what is the maximum sensible volume, and any higher volume can be disabled. The volume limit is set in path configuration files in the [Element] section, using option "volume-limit". The value is the desired maximum volume step of the volume element.
* | dbus: Always accept mono volumes when setting device or stream volume.Tanu Kaskinen2011-03-112-10/+8
| | | | | | | | | | pa_sink_set_volume() and friends accept mono volumes too, so no need to impose unneeded restrictions in the D-Bus API.
* | dbusiface-stream: Fix crash when there's no resampling used.Tanu Kaskinen2011-03-111-0/+5
| |
* | alsa-card: Print the profile set configuration when loading the card.Tanu Kaskinen2011-03-111-0/+1
| |
* | alsa-mixer: Add a default case for a switch, so that the compiler won't ↵Tanu Kaskinen2011-03-111-15/+17
| | | | | | | | complain about unhandled cases.
* | pacat: Fix memory leak when draining the context.Tanu Kaskinen2011-03-111-1/+3
| |
* | alsa-mixer: Use decibel fixes when getting and setting decibel volumes.Tanu Kaskinen2011-03-112-56/+212
| |
* | alsa-mixer: Add DecibelFix section to the profile set config file format.Tanu Kaskinen2011-03-114-14/+297
| | | | | | | | | | This commit only implements the parser, the decibel fix data is not yet used for anything.
* | volume: Add Orc-based optimised volume scalingArun Raghavan2011-03-056-0/+282
| | | | | | | | | | | | | | | | | | 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.
* | volume: Fix sample array size for testsArun Raghavan2011-03-053-3/+3
| | | | | | | | | | | | Somewhere in the history of the MMX tests, the number of channels was changed from 1 to 2, but the number of samples was not increased to make it even (multiple of the frame size).
* | volume: Make tests use only valid volumesArun Raghavan2011-03-053-3/+3
| |
* | alsa-mixer: Fix a git-am cockup in b0f72311Colin Guthrie2011-03-041-3/+3
| | | | | | | | | | | | | | | | | | | | It seems git managed to mess up a git-am with a patch from David which moved where this function was called element_probe to within itself (recursive which could normally lead to an infinite loop, but as it was now never called from anywhere else, this didn't happen). Thank you to Maarten for spotting and following up the issue.
* | volume: Add a PA_VOLUME_UI_MAX define for the recommended max volume to show ↵Colin Guthrie2011-03-031-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in UIs This value is not a technical upper limit, it's just a 'sensible' value that is not crazy high, but also allows software amplification above 0dB (aka 100%) for very quiet audio sources. We recommend that a comprehensive volume control UI should allow users to set volumes up to this limit, although of course should deal gracefully if the user has set the volume even higher than this without resulting in a feedback loop that effectively limits the upper volume. The value chosen is +11dB. This was selected somewhat subjectively and is very similar to the current 150% that gnome-volume-control uses (which is ~+10.57dB). On the plus side, we now recommend that everyone allows 'Volumes up to 11' which is pretty awesome. http://en.wikipedia.org/wiki/Up_to_eleven https://tango.0pointer.de/pipermail/pulseaudio-discuss/2010-April/006945.html https://tango.0pointer.de/pipermail/pulseaudio-discuss/2010-April/006950.html
* | conf: Make system.pa use udev-detect and not hal-detect.Colin Guthrie2011-03-032-4/+4
| | | | | | | | Also fix a left over reference to HAL in default.pa
* | Various fixes for build warningsMaarten Bosmans2011-03-025-13/+7
| |
* | Get rid of some unused-function compiler warningsMaarten Bosmans2011-03-025-5/+11
| |
* | introspect: Client-side implementation for has_volume/read_only_volumeArun Raghavan2011-03-021-2/+6
| | | | | | | | | | This completes the client-side changes to the protocol extension introduced by commit 99ddca89cdca9b0b92ab9870764f9211e6a82e31
* | x11: Make pax11publish -r remove PULSE_SESSION_IDColin Guthrie2011-03-011-0/+1
| | | | | | | | | | | | | | This is not set by pax11publish, but module-x11-publish does so this tool should tidy that up. It is only removed when passing -r and is ignored when actually setting up it's own properties from the conf files/guesswork.
* | Include <time.h> where necessaryMaarten Bosmans2011-03-014-1/+4
| |
* | Merge remote-tracking branch 'arun/no-ramping'Colin Guthrie2011-02-288-1641/+82
|\ \
| * | Remove remaining ramping/envelope referencesArun Raghavan2011-02-282-2/+0
| | | | | | | | | | | | (part of a patch series removing all ramping code)
| * | Revert "Add new subsystem for applying envelopes (such as volume ramps) to ↵Arun Raghavan2011-02-284-1089/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | audio signals" This reverts commit 95a98fe6f2002c9dd448b70bb6944541b5616df3. Conflicts: src/Makefile.am src/pulsecore/envelope.c src/pulsecore/envelope.h src/tests/envelope-test.c (part of a patch series removing all ramping code)
| * | Revert "Add volume ramping feature - envelop fix"Arun Raghavan2011-02-282-271/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 8eaa40b6f4cae749610770c85ba500f326d59b50. Conflicts: src/pulsecore/envelope.c (part of a patch series removing all ramping code)
| * | Revert "Add volume ramping feature - sink-input modification"Arun Raghavan2011-02-282-330/+81
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 5318eb35ef3f91836084382a4f3d5ef08d322554. Conflicts: src/pulsecore/sink-input.c (part of a patch series removing all ramping code)
| * | Revert "Add volume ramping feature - sink modification"Arun Raghavan2011-02-281-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 897ef86b7fbb87ef17d30c584e6cd93abfc342bc. Conflicts: src/pulsecore/sink.c (part of a patch series removing all ramping code)
| * | Revert "ramping: minor cleanups"Arun Raghavan2011-02-281-10/+2
| | | | | | | | | | | | | | | | | | This reverts commit f202af17b717f5b383ac072f80a6c1327bc3143b. (part of a patch series removing all ramping code)
| * | Revert "core: volume ramping fix"Arun Raghavan2011-02-281-1/+1
| | | | | | | | | | | | | | | | | | This reverts commit aa9348441db34b787784711f19882d6c42fa315d. (part of a patch series removing all ramping code)
* | | alsa-mixer: Add support for "Line Boost" elementDavid Henningsson2011-02-282-0/+11
|/ / | | | | | | Signed-off-by: David Henningsson <david.henningsson@canonical.com>
* | Add src/*-symdef.h to .gitignore.Tanu Kaskinen2011-02-282-1/+1
| | | | | | | | | | Also remove src/module/.gitignore as this is no longer needed as pointed out by Arun Raghavan
* | sink: Add casts to some printf arguments to get rid of compiler warnings.Tanu Kaskinen2011-02-281-5/+6
| |
* | sink: Don't send unnecessary PA_SINK_MESSAGE_SET_SHARED_VOLUME messages.Tanu Kaskinen2011-02-281-2/+0
| | | | | | | | If send_msg is false, the message will be sent by the caller.
* | virtual-sink: Fix a crash when moving the sink to a new master right after ↵Tanu Kaskinen2011-02-263-5/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | setup. If the virtual sink is moved to a new master right after it has been created, then the virtual sink input's memblockq can be rewound to a negative read index. The data written prior to the move starts from index zero, so after the rewind there's a bit of silence. If the memblockq doesn't have a silence memchunk set, then pa_memblockq_peek() will return zero in such case, and the returned memchunk's memblock pointer will be NULL. That scenario wasn't taken into account in the implementation of sink_input_pop_cb. Setting a silence memchunk for the memblockq solves this problem, because pa_memblock_peek() will now return a valid memblock if the read index happens to point to a hole in the memblockq. I believe this isn't the best possible solution, though. It doesn't really make sense to rewind the sink input's memblockq beyond index 0 in the first place, because now when the stream starts to play to the new master sink, there's some unnecessary silence before the actual data starts. This is a small problem, though, and I don't grok the rewinding system well enough to know how to fix this issue properly. I went through all files that call pa_memblockq_peek() to see if there are more similar bugs. play-memblockq.c was the only one that looked to me like it might be broken in the same way. I didn't try reproducing the bug with play-memblockq.c, though, so I just added a FIXME comment there.
* | virtual-sink/source: Remove an unused variable.Tanu Kaskinen2011-02-262-10/+0
| |