devs/foxbox
1
0
Fork 0
forked from len0rd/rockbox
Code Pull requests Activity

buflib: Try harder in buflib_alloc_maximum().

Browse source 
This function will now ask shrinkable allocations to give up all of their
memory. With future support of playback.c this can be used as a safe
replacement for audio_get_buffer().

Change-Id: I290a51d2c75254e66baf5698c41dc444dea6247a
This commit is contained in:
Thomas Martitz 2013-05-29 07:12:01 +02:00
parent 9f878b105d
commit d25a512caf
2 changed files with 17 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

13
firmware/buflib.c
View file

@ -724,7 +724,18 @@ buflib_alloc_maximum(struct buflib_context* ctx, const char* name, size_t *size,
/* limit name to 16 since that's what buflib_available() accounts for it */ /* limit name to 16 since that's what buflib_available() accounts for it */
char buf[16]; char buf[16];
*size = buflib_available(ctx); /* ignore ctx->compact because it's true if all movable blocks are contiguous
* even if the buffer has holes due to unmovable allocations */
unsigned hints;
size_t bufsize = ctx->handle_table - ctx->buf_start;
bufsize = MIN(BUFLIB_SHRINK_SIZE_MASK, bufsize*sizeof(union buflib_data)); /* make it bytes */
/* try as hard as possible to free up space. allocations are
* welcome to give up some or all of their memory */
hints = BUFLIB_SHRINK_POS_BACK | BUFLIB_SHRINK_POS_FRONT | bufsize;
/* compact until no space can be gained anymore */
while (buflib_compact_and_shrink(ctx, hints));
*size = buflib_allocatable(ctx);
if (*size <= 0) /* OOM */ if (*size <= 0) /* OOM */
return -1; return -1;

8
firmware/include/buflib.h
View file

@ -111,10 +111,10 @@ struct buflib_callbacks {
void (*sync_callback)(int handle, bool sync_on); void (*sync_callback)(int handle, bool sync_on);
}; };
#define BUFLIB_SHRINK_POS_MASK ((1<<0|1<<1)<<30)
#define BUFLIB_SHRINK_SIZE_MASK (~BUFLIB_SHRINK_POS_MASK) #define BUFLIB_SHRINK_SIZE_MASK (~BUFLIB_SHRINK_POS_MASK)
#define BUFLIB_SHRINK_POS_FRONT (1u<<31) #define BUFLIB_SHRINK_POS_FRONT (1u<<31)
#define BUFLIB_SHRINK_POS_BACK (1u<<30) #define BUFLIB_SHRINK_POS_BACK (1u<<30)
#define BUFLIB_SHRINK_POS_MASK (BUFLIB_SHRINK_POS_FRONT|BUFLIB_SHRINK_POS_BACK)
/** /**
* Possible return values for the callbacks, some of them can cause * Possible return values for the callbacks, some of them can cause
@ -193,8 +193,10 @@ int buflib_alloc_ex(struct buflib_context *ctx, size_t size, const char *name,
* will allow buflib to permit allocations by shrinking the buffer returned by * will allow buflib to permit allocations by shrinking the buffer returned by
* this function. * this function.
* *
* Note that this currently gives whatever buflib_available() returns. However, * Note that this might return many more bytes than buflib_available() or
* do not depend on this behavior, it may change. * buflib_allocatable() return, because it agressively compacts the pool
* and even shrinks other allocations. However, do not depend on this behavior,
* it may change.
* *
* name: A string identifier giving this allocation a name * name: A string identifier giving this allocation a name
* size: The actual size will be returned into size * size: The actual size will be returned into size