From dbd629fd6de40562a66bfda8326efc563a689152 Mon Sep 17 00:00:00 2001 From: Thomas Martitz Date: Thu, 27 Jan 2011 22:52:11 +0100 Subject: [PATCH] Disable buffering codecs (and code generally) on RaaA. It's not useful to do it since you need to write back the code to disk to be able to load it from memory, it also requires writing to an executable directory. Keep it for the simulator for the sake of simulating. --- apps/buffering.c | 6 +++++- apps/buffering.h | 1 + apps/playback.c | 15 +++++++++++---- firmware/load_code.c | 20 +++++++++++--------- 4 files changed, 28 insertions(+), 14 deletions(-) diff --git a/apps/buffering.c b/apps/buffering.c index 46b645517..14859772d 100644 --- a/apps/buffering.c +++ b/apps/buffering.c @@ -1004,7 +1004,11 @@ int bufopen(const char *file, size_t offset, enum data_type type, return h->id; } - +#ifdef APPLICATION + /* loading code from memory is not supported in application builds */ + else if (type == TYPE_CODEC) + return ERR_UNSUPPORTED_TYPE; +#endif /* Other cases: there is a little more work. */ int fd = open(file, O_RDONLY); if (fd < 0) diff --git a/apps/buffering.h b/apps/buffering.h index 6e17b65d8..c3a9f928d 100644 --- a/apps/buffering.h +++ b/apps/buffering.h @@ -44,6 +44,7 @@ enum data_type { #define ERR_INVALID_VALUE -3 #define ERR_FILE_ERROR -4 #define ERR_HANDLE_NOT_DONE -5 +#define ERR_UNSUPPORTED_TYPE -6 /* Initialise the buffering subsystem */ diff --git a/apps/playback.c b/apps/playback.c index e71e06b92..5dda6aa4e 100644 --- a/apps/playback.c +++ b/apps/playback.c @@ -1059,7 +1059,7 @@ static bool audio_release_tracks(void) static bool audio_loadcodec(bool start_play) { - int prev_track; + int prev_track, hid; char codec_path[MAX_PATH]; /* Full path to codec */ const struct mp3entry *id3, *prev_id3; @@ -1114,11 +1114,18 @@ static bool audio_loadcodec(bool start_play) codec_get_full_path(codec_path, codec_fn); - tracks[track_widx].codec_hid = bufopen(codec_path, 0, TYPE_CODEC, NULL); - if (tracks[track_widx].codec_hid < 0) + hid = tracks[track_widx].codec_hid = bufopen(codec_path, 0, TYPE_CODEC, NULL); + + /* not an error if codec load it supported, will load it from disk + * application builds don't support it + */ + if (hid < 0 && hid != ERR_UNSUPPORTED_TYPE) return false; - logf("Loaded codec"); + if (hid > 0) + logf("Loaded codec"); + else + logf("Buffering codec unsupported, load later from disk"); return true; } diff --git a/firmware/load_code.c b/firmware/load_code.c index 2337ee5ca..59eb7ac0f 100644 --- a/firmware/load_code.c +++ b/firmware/load_code.c @@ -134,6 +134,16 @@ void * _lc_open(const _lc_open_char *filename, unsigned char *buf, size_t buf_si void *lc_open_from_mem(void *addr, size_t blob_size) { +#ifndef SIMULATOR + (void)addr; + (void)blob_size; + /* we don't support loading code from memory on application builds, + * it doesn't make sense (since it means writing the blob to disk again and + * then falling back to load from disk) and requires the ability to write + * to an executable directory */ + return NULL; +#else + /* support it in the sim for the sake of simulating */ int fd, i; char temp_filename[MAX_PATH]; @@ -143,17 +153,8 @@ void *lc_open_from_mem(void *addr, size_t blob_size) to find an unused filename */ for (i = 0; i < 10; i++) { -#if (CONFIG_PLATFORM & PLATFORM_ANDROID) - /* we need that path fixed, since _get_user_file_path() - * gives us the folder on the sdcard where we cannot load libraries - * from (no exec permissions) - */ - snprintf(temp_filename, sizeof(temp_filename), - "/data/data/org.rockbox/app_rockbox/libtemp_binary_%d.so", i); -#else snprintf(temp_filename, sizeof(temp_filename), ROCKBOX_DIR "/libtemp_binary_%d.dll", i); -#endif fd = open(temp_filename, O_WRONLY|O_CREAT|O_TRUNC, 0700); if (fd >= 0) break; /* Created a file ok */ @@ -175,6 +176,7 @@ void *lc_open_from_mem(void *addr, size_t blob_size) close(fd); return lc_open(temp_filename, NULL, 0); +#endif } -- 2.11.4.GIT