diff options
author | Jon Medhurst <tixy@linaro.org> | 2014-02-06 14:13:16 +0000 |
---|---|---|
committer | Dan Handley <dan.handley@arm.com> | 2014-03-05 16:21:14 +0000 |
commit | a4d9f26b009647db14fbf24479245227e712d35d (patch) | |
tree | 2f2bfcdc2394046563c86c64de83a2d96132f4a6 /lib | |
parent | b2388490302d41038a5dd9a16d7f6d554c828bd3 (diff) |
Fix assert in bakery_lock_release()
bakery_lock_release() expects an mpidr as the first argument however
bakery_lock_release() is calling it with the 'entry' argument it has
calculated. Rather than fixing this to pass the mpidr value it would be
much more efficient to just replace the call with
assert(bakery->owner == entry)
As this leaves no remaining users of bakery_lock_held(), we might as
well delete it.
Fixes ARM-software/tf-issues#27
Signed-off-by: Jon Medhurst <tixy@linaro.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/sync/locks/bakery/bakery_lock.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/lib/sync/locks/bakery/bakery_lock.c b/lib/sync/locks/bakery/bakery_lock.c index 9641248..444b6a1 100644 --- a/lib/sync/locks/bakery/bakery_lock.c +++ b/lib/sync/locks/bakery/bakery_lock.c @@ -88,17 +88,8 @@ void bakery_lock_release(unsigned long mpidr, bakery_lock * bakery) unsigned int entry = platform_get_core_pos(mpidr); assert_bakery_entry_valid(entry, bakery); - assert(bakery_lock_held(entry, bakery)); + assert(bakery->owner == entry); bakery->owner = NO_OWNER; bakery->number[entry] = 0; } - -int bakery_lock_held(unsigned long mpidr, const bakery_lock * bakery) -{ - unsigned int entry = platform_get_core_pos(mpidr); - - assert_bakery_entry_valid(entry, bakery); - - return bakery->owner == entry; -} |