From b15aa47c56d4f8c6e4bf83fef48e7a764dd119a2 Mon Sep 17 00:00:00 2001 From: Michael Sevakis Date: Mon, 14 Feb 2011 11:27:45 +0000 Subject: All kernel objects in code shared amongs targets (core, plugins, codecs) should be declared SHAREDBSS_ATTR as any core could potentially touch them even though they seem only to involve threads on one core. The exception is target code for particular CPUs where proper allocation is fixed. playlist.c was a little odd too-- use one mutex for the current playlist and a separate one for created playlists (still pondering the necessity of more than one). git-svn-id: svn://svn.rockbox.org/rockbox/trunk@29305 a1c6a512-1295-4272-9138-f99709370657 --- apps/plugins/battery_bench.c | 2 +- apps/plugins/mikmod/mikmod.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'apps/plugins') diff --git a/apps/plugins/battery_bench.c b/apps/plugins/battery_bench.c index 044db9ff1f..6a0493ceaa 100644 --- a/apps/plugins/battery_bench.c +++ b/apps/plugins/battery_bench.c @@ -266,7 +266,7 @@ static struct batt_info #define BUF_ELEMENTS (sizeof(bat)/sizeof(struct batt_info)) static unsigned int thread_id; -static struct event_queue thread_q; +static struct event_queue thread_q SHAREDBSS_ATTR; static bool in_usb_mode; static unsigned int buf_idx; diff --git a/apps/plugins/mikmod/mikmod.c b/apps/plugins/mikmod/mikmod.c index fe1768376e..c99d11d636 100644 --- a/apps/plugins/mikmod/mikmod.c +++ b/apps/plugins/mikmod/mikmod.c @@ -31,7 +31,7 @@ #define EV_EXIT 9999 #define THREAD_STACK_SIZE DEFAULT_STACK_SIZE + 0x200 static unsigned int thread_id; -static struct event_queue thread_q; +static struct event_queue thread_q SHAREDBSS_ATTR; /* use long for aligning */ unsigned long thread_stack[THREAD_STACK_SIZE/sizeof(long)]; #endif -- cgit v1.2.3