Remove the wfe() for bounded wait in bakery_lock
This patch is an optimization in the bakery_lock_get() function
which removes the wfe() when waiting for other contenders to choose
their ticket i.e when their `entering` flag is set. Since the time
taken to execute bakery_get_ticket() by other contenders is bounded,
this wait is a bounded time wait. Hence the removal of wfe() and the
corresponding sev() and dsb() in bakery_get_ticket() may result
in better time performance during lock acquisition.
Change-Id: I141bb21294226b54cb6e89e7cac0175c553afd8d
diff --git a/lib/locks/bakery/bakery_lock.c b/lib/locks/bakery/bakery_lock.c
index 7e71dec..6bdc35a 100644
--- a/lib/locks/bakery/bakery_lock.c
+++ b/lib/locks/bakery/bakery_lock.c
@@ -107,8 +107,6 @@
++my_ticket;
bakery->number[me] = my_ticket;
bakery->entering[me] = 0;
- dsb();
- sev();
return my_ticket;
}
@@ -151,7 +149,7 @@
/* Wait for the contender to get their ticket */
while (bakery->entering[they])
- wfe();
+ ;
/*
* If the other party is a contender, they'll have non-zero