[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 
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, );
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, );


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

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

[git repository hosting]

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

Author: Philippe Gerum 
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, );
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, );


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

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

[git repository hosting]

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

Author: Philippe Gerum 
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, );
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, );


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