[Xenomai-git] Philippe Gerum : copperplate: drop useless membars

[git repository hosting]

Module: xenomai-3
Branch: master
Commit: 37b043b12a50ef7980cd8a3afecd853fac8b4148
URL:    
http://git.xenomai.org/?p=xenomai-3.git;a=commit;h=37b043b12a50ef7980cd8a3afecd853fac8b4148

Author: Philippe Gerum <r...@xenomai.org>
Date:   Mon Sep 21 15:59:27 2015 +0200

copperplate: drop useless membars

---

 lib/copperplate/threadobj.c |    7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/lib/copperplate/threadobj.c b/lib/copperplate/threadobj.c
index 2f25992..cc64caa 100644
--- a/lib/copperplate/threadobj.c
+++ b/lib/copperplate/threadobj.c
@@ -359,8 +359,6 @@ static inline int threadobj_unblocked_corespec(struct 
threadobj *current)
 
 int __threadobj_lock_sched(struct threadobj *current)
 {
-       smp_rmb();
-
        if (current->schedlock_depth++ > 0)
                return 0;
 
@@ -388,8 +386,6 @@ int __threadobj_unlock_sched(struct threadobj *current)
         * gracefully handle unbalanced calls here, and let them
         * decide of the outcome in case of error.
         */
-       smp_rmb();
-
        if (current->schedlock_depth == 0)
                return __bt(-EINVAL);
 
@@ -912,7 +908,7 @@ static void __threadobj_boost(void)
                current->core.policy_unboosted = current->policy;
                current->core.schedparam_unboosted = current->schedparam;
        }
-       smp_mb();
+       barrier();
 
        ret = pthread_setschedparam(current->ptid, SCHED_FIFO, &param);
        if (ret) {
@@ -932,7 +928,6 @@ static void __threadobj_unboost(void)
                return;
 
        param.sched_priority = 
current->core.schedparam_unboosted.sched_priority;
-       smp_mb();
 
        ret = pthread_setschedparam(current->ptid,
                                    current->core.policy_unboosted, &param);


_______________________________________________
Xenomai-git mailing list
Xenomai-git@xenomai.org
http://xenomai.org/mailman/listinfo/xenomai-git