Rockbox

Tasklist

FS#12159 - GSoC/Buflib: Remove direct audiobuf accesses

Attached to Project: Rockbox
Opened by Thomas Martitz (kugel.) - Tuesday, 14 June 2011, 16:06 GMT
Last edited by Thomas Martitz (kugel.) - Monday, 15 August 2011, 12:52 GMT
Task Type Patches
Category Operating System/Drivers
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.8.1
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Replace all direct accesses to audiobuf with buffer API functions.

Namely, introduce buffer_get_buffer() and buffer_release_buffer().
buffer_get_buffer() aquires all available and grabs a lock, attempting to
call buffer_alloc() or buffer_get_buffer() while this lock is locked will cause a panicf() (doesn't actually happen, but is for debugging purpose).
buffer_release_buffer() unlocks that lock and can additionally increment the audiobuf buffer to make an allocation. Pass 0 to only unlock if buffer was used temporarily only.
buffer_available() is a replacement function to query audiobuflen, i.e. what's left in the buffer.

Further changes happened to mp3data.c and talk.c as to not call the above API functions, but get the buffer from callers. The caller is the audio system which has the buffer lock while mp3dat and talk mess with the buffer.
mpeg.c now implements some buffer related functions of playback.h, that no #ifdefs are needed anymore around audio_get_buffer().

audiobuf and audiobufend are local to buffer.c now.
This task depends upon

Closed by  Thomas Martitz (kugel.)
Monday, 15 August 2011, 12:52 GMT
Reason for closing:  Accepted
Additional comments about closing:  r30308
Comment by Thomas Martitz (kugel.) - Monday, 20 June 2011, 21:48 GMT
Updated patch.
*) Rebased to current svn
*) Few bug fixes.
Comment by Thomas Martitz (kugel.) - Saturday, 09 July 2011, 23:48 GMT
Sync to latest SVN.
Comment by Frank Gevaerts (fg) - Friday, 29 July 2011, 21:35 GMT
This looks sane to me, with of course the caveat that I'm not an expert for most of the apps/ code
Comment by Thomas Martitz (kugel.) - Monday, 01 August 2011, 15:02 GMT
Updated version with a few bug fixes. Intent to commit this.
Comment by Thomas Martitz (kugel.) - Wednesday, 03 August 2011, 20:14 GMT
(Probably) final update of this patch. Just a resync to SVN.
Comment by Thomas Martitz (kugel.) - Tuesday, 09 August 2011, 18:48 GMT
Another updated patch with minor fixes.

Loading...