forked from len0rd/rockbox
puzzles: enable fallback to audiobuf when smalloc() fails
- upon a failed smalloc(), the audio buffer will be used for further allocations - should fix things on low-memory targets (c100 and c200v2), but breaks playback - playback should still be intact on other targets Change-Id: Ic239f1316efadc957050afacf5c614dbbca3f805
This commit is contained in:
parent
954d934ad2
commit
637c7414a9
2 changed files with 55 additions and 7 deletions
|
@ -14,12 +14,40 @@
|
|||
int allocs = 0;
|
||||
int frees = 0;
|
||||
|
||||
bool audiobuf_available =
|
||||
#ifndef COMBINED
|
||||
true;
|
||||
#else
|
||||
false;
|
||||
#endif
|
||||
|
||||
static bool grab_audiobuf(void)
|
||||
{
|
||||
if(!audiobuf_available)
|
||||
return false;
|
||||
|
||||
if(rb->audio_status())
|
||||
rb->audio_stop();
|
||||
|
||||
size_t sz, junk;
|
||||
void *audiobuf = rb->plugin_get_audio_buffer(&sz);
|
||||
extern char *giant_buffer;
|
||||
|
||||
add_new_area(audiobuf, sz, giant_buffer);
|
||||
audiobuf_available = false;
|
||||
return true;
|
||||
}
|
||||
|
||||
void *smalloc(size_t size) {
|
||||
void *p;
|
||||
p = malloc(size);
|
||||
LOGF("allocs: %d", ++allocs);
|
||||
if (!p)
|
||||
{
|
||||
if(grab_audiobuf())
|
||||
return smalloc(size);
|
||||
fatal("out of memory");
|
||||
}
|
||||
return p;
|
||||
}
|
||||
|
||||
|
@ -46,7 +74,11 @@ void *srealloc(void *p, size_t size) {
|
|||
q = malloc(size);
|
||||
}
|
||||
if (!q)
|
||||
{
|
||||
if(grab_audiobuf())
|
||||
return srealloc(p, size);
|
||||
fatal("out of memory");
|
||||
}
|
||||
return q;
|
||||
}
|
||||
|
||||
|
|
|
@ -71,6 +71,7 @@ static void fix_size(void);
|
|||
|
||||
static struct viewport clip_rect;
|
||||
static bool clipped = false;
|
||||
extern bool audiobuf_available;
|
||||
|
||||
static struct settings_t {
|
||||
int slowmo_factor;
|
||||
|
@ -1026,7 +1027,10 @@ static int pausemenu_cb(int action, const struct menu_item_ex *this_item)
|
|||
* care, I bet */
|
||||
return ACTION_EXIT_MENUITEM;
|
||||
#else
|
||||
if(audiobuf_available)
|
||||
break;
|
||||
else
|
||||
return ACTION_EXIT_MENUITEM;
|
||||
#endif
|
||||
case 9:
|
||||
if(!midend_num_presets(me))
|
||||
|
@ -1355,10 +1359,10 @@ void deactivate_timer(frontend *fe)
|
|||
|
||||
#ifdef COMBINED
|
||||
/* can't use audio buffer */
|
||||
static char giant_buffer[1024*1024*4];
|
||||
char giant_buffer[1024*1024*4];
|
||||
#else
|
||||
/* points to audiobuf */
|
||||
static char *giant_buffer = NULL;
|
||||
/* points to pluginbuf */
|
||||
char *giant_buffer = NULL;
|
||||
#endif
|
||||
static size_t giant_buffer_len = 0; /* set on start */
|
||||
|
||||
|
@ -1565,6 +1569,18 @@ static int mainmenu_cb(int action, const struct menu_item_ex *this_item)
|
|||
return ACTION_EXIT_MENUITEM;
|
||||
#else
|
||||
break;
|
||||
#endif
|
||||
case 4:
|
||||
#ifdef COMBINED
|
||||
/* audio buf is used, so no playback */
|
||||
/* TODO: neglects app builds, but not many people will
|
||||
* care, I bet */
|
||||
return ACTION_EXIT_MENUITEM;
|
||||
#else
|
||||
if(audiobuf_available)
|
||||
break;
|
||||
else
|
||||
return ACTION_EXIT_MENUITEM;
|
||||
#endif
|
||||
case 5:
|
||||
if(!midend_num_presets(me))
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue