aboutsummaryrefslogtreecommitdiffstats
path: root/Alc/ALu.c
Commit message (Collapse)AuthorAgeFilesLines
* Replace some ALvoid with voidChris Robinson2017-01-181-2/+2
|
* Use ALsizei in more placesChris Robinson2017-01-181-1/+1
|
* Pass the left and right buffers to the hrtf mixers directlyChris Robinson2017-01-171-14/+16
|
* Use ALsizei for sizes and offsets with the mixerChris Robinson2017-01-161-1/+1
| | | | | | Unsigned 32-bit offsets actually have some potential overhead on 64-bit targets for pointer/array accesses due to rules on integer wrapping. No idea how much impact it has in practice, but it's nice to be correct about it.
* Use second-order ambisonics for basic HRTF renderingChris Robinson2017-01-151-0/+7
| | | | | | This should improve positional quality for relatively low cost. Full HRTF rendering still only uses first-order since the only use of the dry buffer there is for first-order content (B-Format buffers, effects).
* Avoid duplicating code using a macroChris Robinson2016-12-211-38/+3
|
* Use separate macros for atomics that don't take a memory orderChris Robinson2016-12-201-7/+7
|
* Update some atomic memory orderingChris Robinson2016-11-211-6/+9
|
* Don't interpolate between nearest HRIRsChris Robinson2016-10-091-4/+4
| | | | | | | | It still fades between HRIRs when it changes, but now it selects the nearest one instead of blending the nearest four. Due to the minimum-phase nature of the HRIRs, interpolating between delays lead to some oddities which are exasperated by the fading (and the fading is needed to avoid clicks and pops, and smooth out changes).
* Make some pointer-to-array parameters constChris Robinson2016-10-041-7/+6
|
* Add a volume-adjust config option to adjust the source output volumeChris Robinson2016-09-241-1/+1
| | | | | | | | | Designed for apps that either don't change the listener's AL_GAIN, or don't allow the listener's AL_GAIN to go above 1. This allows the volume to still be increased further than such apps may allow, if users find it too quiet. Be aware that increasing this can easily cause clipping. The gain limit reported by AL_GAIN_LIMIT_SOFT is also affected by this.
* Use a predefined identity matrixChris Robinson2016-09-051-0/+7
|
* Correct a comment about B-Format conversionChris Robinson2016-09-051-1/+1
|
* Clamp the maximum mixing gain boost to 16Chris Robinson2016-08-271-10/+11
| | | | | | The combined source and listener gains now can't exceed a multiplier of 16 (~24dB). This is to avoid mixes getting out of control with large volume boosts, which reduces the effective precision given by floating-point.
* Properly defer effect slot changesChris Robinson2016-08-251-3/+5
| | | | | | | | Note that this now also causes all playing sources to update when an effect slot is updated. This is a bit wasteful, as it should only need to re-update sources that are using the effect slot (and only when a relevant property is changed), but it's good enough. Especially with deferring since all playing sources are going to get updated on the process call anyway.
* Track all references for effect statesChris Robinson2016-08-251-9/+6
| | | | | | | | This allows us to not have to play around with trying to avoid duplicate state pointers, since the reference count will ensure they're deleted as appropriate. The only caveat is that the mixer is not allowed to decrement references, since that can cause the object to be freed (which the mixer code is not allowed to do).
* Consolidate duplicate codeChris Robinson2016-08-241-39/+23
|
* Combine related members into a structChris Robinson2016-08-241-7/+7
|
* Don't pass the context's distance model as the source'sChris Robinson2016-08-231-1/+6
|
* Avoid resupplying unneeded source updatesChris Robinson2016-08-231-28/+51
| | | | | The source's voice holds a copy of the last properties it received, so listener updates can make sources recalculate internal properties from that stored copy.
* Use a more specialized mixer function for B-Format to HRTFChris Robinson2016-08-121-11/+8
|
* Decode directly from B-Format to HRTF instead of a cubeChris Robinson2016-08-111-1/+1
| | | | | | | | | | | | | | | | | | | | | Last time this attempted to average the HRIRs according to their contribution to a given B-Format channel as if they were loudspeakers, as well as averaging the HRIR delays. The latter part resulted in the loss of the ITD (inter-aural time delay), a key component of HRTF. This time, the HRIRs are averaged similar to above, except instead of averaging the delays, they're applied to the resulting coefficients (for example, a delay of 8 would apply the HRIR starting at the 8th sample of the target HRIR). This does roughly double the IR length, as the largest delay is about 35 samples while the filter is normally 32 samples. However, this is still smaller the original data set IR (which was 256 samples), it also only needs to be applied to 4 channels for first-order ambisonics, rather than the 8-channel cube. So it's doing twice as much work per sample, but only working on half the number of samples. Additionally, since the resulting HRIRs no longer rely on an extra delay line, a more efficient HRTF mixing function can be made that doesn't use one. Such a function can also avoid the per-sample stepping parameters the original uses.
* Constify some variablesChris Robinson2016-08-041-2/+2
|
* Don't store the looping state in the voiceChris Robinson2016-07-311-2/+0
| | | | | Certain operations on the buffer queue depend on the loop state to behave properly, so it should not be deferred until the async voice update occurs.
* Add a stand-alone upsampler for higher-order ambisonic oputputChris Robinson2016-07-301-0/+7
|
* Modify bs2b_cross_feed to do multiple samples at onceChris Robinson2016-07-131-22/+17
|
* Move the input channel array out of the DirectParams and SendParamsChris Robinson2016-07-131-55/+56
|
* Store the voice output buffers separate from the paramsChris Robinson2016-07-111-20/+20
|
* Avoid function calls to get the HRTF sample rate and IR sizeChris Robinson2016-07-071-1/+1
|
* Remove the VirtOut buffer aliasChris Robinson2016-07-051-10/+11
|
* Ensure voices has been updated once before mixing themChris Robinson2016-06-161-1/+2
| | | | | | | | Sometimes the mixer is temporarily prevented from applying updates, when multiple sources need to be updated simultaneously for example, but does not prevent mixing. If the mixer runs during that time and a voice was just started, it would've mixed the voice without any internal properties being set for it.
* Use a linked list for active effect slotsChris Robinson2016-05-291-16/+24
|
* Increment the device's mix count closer to the mixing loopsChris Robinson2016-05-231-3/+2
|
* Make the source position calues atomicChris Robinson2016-05-191-2/+2
|
* Copy the source's Looping property into the voiceChris Robinson2016-05-181-0/+2
|
* Avoid redundantly storing distance model settingsChris Robinson2016-05-171-7/+1
|
* Ignore the listening angle for the wet path sound conesChris Robinson2016-05-161-32/+51
| | | | | | | Since the wet path is essentially the room response to a sound, the direction of the sound to the listener doesn't change the amount of energy the room receives. Instead, the surface area defined by the cones dictate the volume the room gets for the sound.
* Use floats for the listener transformsChris Robinson2016-05-161-61/+23
|
* Don't store the source's update method with the voiceChris Robinson2016-05-161-38/+42
|
* Avoid separate updates to sources that should apply togetherChris Robinson2016-05-151-12/+17
|
* Avoid using a flag to specify if the effect state needs to be updatedChris Robinson2016-05-151-4/+8
| | | | | This fixes a potential missed state change if an update with a new state got replaced with one that doesn't.
* Provide asynchronous property updates for sourcesChris Robinson2016-05-141-132/+135
| | | | | | | | | | | | | | | | | | | | | | | | | This necessitates a change in how source updates are handled. Rather than just being able to update sources when a dependent object state is changed (e.g. a listener gain change), now all source updates must be proactively provided. Consequently, apps that do not utilize any deferring (AL_SOFT_defer_updates or alcSuspendContext/alcProcessContext) may utilize more CPU since it'll be filling out more update containers for the mixer thread to use. The upside is that there's less blocking between the app's calling thread and the mixer thread, particularly for vectors and other multi-value properties (filters and sends). Deferring behavior when used is also improved, since updates that shouldn't be applied yet are simply not provided. And when they are provided, the mixer doesn't have to ignore them, meaning the actual deferring of a context doesn't have to synchrnously force an update -- the process call will send any pending updates, which the mixer will apply even if another deferral occurs before the mixer runs, because it'll still be there waiting on the next mixer invocation. There is one slight bug introduced by this commit. When a listener change is made, or changes to multiple sources while updates are being deferred, it is possible for the mixer to run while the sources are prepping their updates, causing some of the source updates to be seen before the other. This will be fixed in short order.
* Store the remaining context properties with the listener propertiesChris Robinson2016-05-131-2/+6
|
* Get rid of an unnecessary copy of ALeffectPropsChris Robinson2016-05-131-6/+5
|
* Call the effect state update method after "returning" the container object.Chris Robinson2016-05-121-2/+2
|
* Avoid updating the effect state object if it's not changedChris Robinson2016-05-121-8/+7
|
* Provide (mostly) lockless updates for effect slotsChris Robinson2016-05-121-34/+69
| | | | | | | | | | | | | | | | | Similar to the listener, separate containers are provided atomically for the mixer thread to apply updates without needing to block, and a free-list is used to reuse container objects. A couple things to note. First, the lock is still used when the effect state's deviceUpdate method is called to prevent asynchronous calls to reset the device from interfering. This can be fixed by using the list lock in ALc.c instead. Secondly, old effect states aren't immediately deleted when the effect type changes (the actual type, not just its properties). This is because the mixer thread is intended to be real-time safe, and so can't be freeing anything. They are cleared away when updates reuse the container they were kept in, and they don't incur any extra processing cost, but there may be cases where the memory is kept around until the effect slot is deleted.
* Use a lockless method for updating listener and context propertiesChris Robinson2016-05-111-18/+40
| | | | | | | | | | | This uses a separate container to provide the relevant properties to the internal update method, using atomic pointer swaps. A free-list is used to avoid having too many individual containers. This allows the mixer to update the internal listener properties without requiring the lock to protect against async updates. It also allows concurrent read access to the user-facing property values, even the multi-value ones (e.g. the vectors).
* Find a valid source buffer before updating the voiceChris Robinson2016-05-091-47/+45
|
* Store more "active" listener and context properties separatelyChris Robinson2016-05-091-11/+20
| | | | | This helps ensure async listener/context property changes affect all playing sources at the same time.