diff options
| author | Michael Sevakis <jethead71@rockbox.org> | 2014-04-28 09:15:06 -0400 |
|---|---|---|
| committer | Michael Sevakis <jethead71@rockbox.org> | 2014-05-09 23:14:58 +0200 |
| commit | 355b5406fafe0d45285e70a0b01d5e674d6a2c16 (patch) | |
| tree | b17e42d8e36182280cf7399c9063a6db8b2f4fc9 /apps | |
| parent | f444ae70ebe9c978818bf473d176538244b4559e (diff) | |
| download | rockbox-355b5406fafe0d45285e70a0b01d5e674d6a2c16.zip rockbox-355b5406fafe0d45285e70a0b01d5e674d6a2c16.tar.gz rockbox-355b5406fafe0d45285e70a0b01d5e674d6a2c16.tar.bz2 rockbox-355b5406fafe0d45285e70a0b01d5e674d6a2c16.tar.xz | |
Make errno a thread-local variable.
errno is supposed to be thread local and we do that here in a rather
makeshift way by implementing a function that returns a pointer to
the __errno variable in the thread block.
If more serious TLS is required, perhaps it would be worthwhile
implementing it using the linker and the "__thread" storage attribute.
That's a bit overkill just for this. I'm also not liking what I saw
the compiler producing.
Change-Id: I03bc0bd6a89f6e3d6bae7653284ee01054614f9a
Reviewed-on: http://gerrit.rockbox.org/803
Reviewed-by: Michael Sevakis <jethead71@rockbox.org>
Tested: Michael Sevakis <jethead71@rockbox.org>
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/plugin.c | 2 | ||||
| -rw-r--r-- | apps/plugin.h | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/apps/plugin.c b/apps/plugin.c index 93779d7..4bed707 100644 --- a/apps/plugin.c +++ b/apps/plugin.c @@ -684,7 +684,7 @@ static const struct plugin_api rockbox_api = { /* misc */ #if (CONFIG_PLATFORM & PLATFORM_NATIVE) - &errno, + __errno, #endif srand, rand, diff --git a/apps/plugin.h b/apps/plugin.h index 6a15f86..fd4e468 100644 --- a/apps/plugin.h +++ b/apps/plugin.h @@ -160,12 +160,12 @@ void* plugin_get_buffer(size_t *buffer_size); #define PLUGIN_MAGIC 0x526F634B /* RocK */ /* increase this every time the api struct changes */ -#define PLUGIN_API_VERSION 229 +#define PLUGIN_API_VERSION 230 /* update this to latest version if a change to the api struct breaks backwards compatibility (and please take the opportunity to sort in any new function which are "waiting" at the end of the function table) */ -#define PLUGIN_MIN_API_VERSION 228 +#define PLUGIN_MIN_API_VERSION 230 /* plugin return codes */ /* internal returns start at 0x100 to make exit(1..255) work */ @@ -829,7 +829,7 @@ struct plugin_api { /* misc */ #if (CONFIG_PLATFORM & PLATFORM_NATIVE) - int* __errno; + int * (*__errno)(void); #endif void (*srand)(unsigned int seed); int (*rand)(void); |