Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/rump/librump/rumpkern Remove redundant UNLOCKED and LOCKED



details:   https://anonhg.NetBSD.org/src/rev/4d807a8ad420
branches:  trunk
changeset: 340770:4d807a8ad420
user:      ozaki-r <ozaki-r%NetBSD.org@localhost>
date:      Wed Sep 30 01:31:56 2015 +0000

description:
Remove redundant UNLOCKED and LOCKED

UNLOCKED and LOCKED are done inside mutex_exit and mutex_enter respectively
so we don't need to do them outside mutex_exit and mutex_enter.

Reviewed by pooka@

diffstat:

 sys/rump/librump/rumpkern/locks.c |  6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diffs (34 lines):

diff -r 0a55bb5e8944 -r 4d807a8ad420 sys/rump/librump/rumpkern/locks.c
--- a/sys/rump/librump/rumpkern/locks.c Wed Sep 30 00:35:46 2015 +0000
+++ b/sys/rump/librump/rumpkern/locks.c Wed Sep 30 01:31:56 2015 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: locks.c,v 1.69 2014/04/25 18:13:59 pooka Exp $ */
+/*     $NetBSD: locks.c,v 1.70 2015/09/30 01:31:56 ozaki-r Exp $       */
 
 /*
  * Copyright (c) 2007-2011 Antti Kantee.  All Rights Reserved.
@@ -26,7 +26,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: locks.c,v 1.69 2014/04/25 18:13:59 pooka Exp $");
+__KERNEL_RCSID(0, "$NetBSD: locks.c,v 1.70 2015/09/30 01:31:56 ozaki-r Exp $");
 
 #include <sys/param.h>
 #include <sys/kmem.h>
@@ -368,7 +368,6 @@
        if (__predict_false(l->l_flag & LW_RUMP_QEXIT)) {
                struct proc *p = l->l_proc;
 
-               UNLOCKED(mtx, false);
                mutex_exit(mtx); /* drop and retake later */
 
                mutex_enter(p->p_lock);
@@ -383,7 +382,6 @@
                /* ok, we can exit and remove "reference" to l->private */
 
                mutex_enter(mtx);
-               LOCKED(mtx, false);
                rv = EINTR;
        }
        l->l_private = NULL;



Home | Main Index | Thread Index | Old Index