Source-Changes-HG archive

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

[src/trunk]: src/sys/dev/raidframe Since rf_mutex_destroy() is effectively a ...



details:   https://anonhg.NetBSD.org/src/rev/3905301b7a16
branches:  trunk
changeset: 556861:3905301b7a16
user:      oster <oster%NetBSD.org@localhost>
date:      Mon Dec 29 05:01:14 2003 +0000

description:
Since rf_mutex_destroy() is effectively a NOP now, nuke it.

diffstat:

 sys/dev/raidframe/rf_acctrace.c      |   6 ++----
 sys/dev/raidframe/rf_paritylogging.c |  18 ++----------------
 sys/dev/raidframe/rf_psstatus.c      |   5 ++---
 sys/dev/raidframe/rf_reconmap.c      |   5 ++---
 sys/dev/raidframe/rf_reconutil.c     |   6 ++----
 sys/dev/raidframe/rf_stripelocks.c   |   8 ++------
 sys/dev/raidframe/rf_threadstuff.c   |   8 ++------
 sys/dev/raidframe/rf_threadstuff.h   |   3 +--
 8 files changed, 15 insertions(+), 44 deletions(-)

diffs (truncated from 331 to 300 lines):

diff -r 24fb3dfcdf2f -r 3905301b7a16 sys/dev/raidframe/rf_acctrace.c
--- a/sys/dev/raidframe/rf_acctrace.c   Mon Dec 29 04:56:26 2003 +0000
+++ b/sys/dev/raidframe/rf_acctrace.c   Mon Dec 29 05:01:14 2003 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: rf_acctrace.c,v 1.11 2003/12/29 04:56:26 oster Exp $   */
+/*     $NetBSD: rf_acctrace.c,v 1.12 2003/12/29 05:01:14 oster Exp $   */
 /*
  * Copyright (c) 1995 Carnegie-Mellon University.
  * All rights reserved.
@@ -34,7 +34,7 @@
 
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_acctrace.c,v 1.11 2003/12/29 04:56:26 oster Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_acctrace.c,v 1.12 2003/12/29 05:01:14 oster Exp $");
 
 #include <sys/stat.h>
 #include <sys/types.h>
@@ -66,7 +66,6 @@
                                accessTraceBufCount = 0;
                RF_Free(access_tracebuf, rf_accessTraceBufSize * sizeof(RF_AccTraceEntry_t));
        }
-       rf_mutex_destroy(&rf_tracing_mutex);
 }
 
 int 
@@ -87,7 +86,6 @@
                rf_print_unable_to_add_shutdown(__FILE__, __LINE__, rc);
                if (rf_accessTraceBufSize) {
                        RF_Free(access_tracebuf, rf_accessTraceBufSize * sizeof(RF_AccTraceEntry_t));
-                       rf_mutex_destroy(&rf_tracing_mutex);
                }
        }
        return (rc);
diff -r 24fb3dfcdf2f -r 3905301b7a16 sys/dev/raidframe/rf_paritylogging.c
--- a/sys/dev/raidframe/rf_paritylogging.c      Mon Dec 29 04:56:26 2003 +0000
+++ b/sys/dev/raidframe/rf_paritylogging.c      Mon Dec 29 05:01:14 2003 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: rf_paritylogging.c,v 1.17 2003/12/29 03:33:48 oster Exp $      */
+/*     $NetBSD: rf_paritylogging.c,v 1.18 2003/12/29 05:01:14 oster Exp $      */
 /*
  * Copyright (c) 1995 Carnegie-Mellon University.
  * All rights reserved.
@@ -32,7 +32,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_paritylogging.c,v 1.17 2003/12/29 03:33:48 oster Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_paritylogging.c,v 1.18 2003/12/29 05:01:14 oster Exp $");
 
 #include "rf_archs.h"
 
@@ -313,7 +313,6 @@
        rc = rf_cond_init(&raidPtr->regionBufferPool.cond);
        if (rc) {
                rf_print_unable_to_init_cond(__FILE__, __LINE__, rc);
-               rf_mutex_destroy(&raidPtr->regionBufferPool.mutex);
                return (ENOMEM);
        }
        raidPtr->regionBufferPool.bufferSize = raidPtr->regionLogCapacity * 
@@ -335,7 +334,6 @@
                  raidPtr->regionBufferPool.totalBuffers * sizeof(caddr_t), 
                  (caddr_t *));
        if (raidPtr->regionBufferPool.buffers == NULL) {
-               rf_mutex_destroy(&raidPtr->regionBufferPool.mutex);
                rf_cond_destroy(&raidPtr->regionBufferPool.cond);
                return (ENOMEM);
        }
@@ -347,7 +345,6 @@
                          raidPtr->regionBufferPool.bufferSize * sizeof(char),
                          (caddr_t));
                if (raidPtr->regionBufferPool.buffers[i] == NULL) {
-                       rf_mutex_destroy(&raidPtr->regionBufferPool.mutex);
                        rf_cond_destroy(&raidPtr->regionBufferPool.cond);
                        for (j = 0; j < i; j++) {
                                RF_Free(raidPtr->regionBufferPool.buffers[i], 
@@ -381,7 +378,6 @@
        rc = rf_cond_init(&raidPtr->parityBufferPool.cond);
        if (rc) {
                rf_print_unable_to_init_cond(__FILE__, __LINE__, rc);
-               rf_mutex_destroy(&raidPtr->parityBufferPool.mutex);
                return (ENOMEM);
        }
        raidPtr->parityBufferPool.bufferSize = parityBufferCapacity * 
@@ -404,7 +400,6 @@
                  raidPtr->parityBufferPool.totalBuffers * sizeof(caddr_t), 
                  (caddr_t *));
        if (raidPtr->parityBufferPool.buffers == NULL) {
-               rf_mutex_destroy(&raidPtr->parityBufferPool.mutex);
                rf_cond_destroy(&raidPtr->parityBufferPool.cond);
                return (ENOMEM);
        }
@@ -416,7 +411,6 @@
                          raidPtr->parityBufferPool.bufferSize * sizeof(char),
                          (caddr_t));
                if (raidPtr->parityBufferPool.buffers == NULL) {
-                       rf_mutex_destroy(&raidPtr->parityBufferPool.mutex);
                        rf_cond_destroy(&raidPtr->parityBufferPool.cond);
                        for (j = 0; j < i; j++) {
                                RF_Free(raidPtr->parityBufferPool.buffers[i], 
@@ -487,7 +481,6 @@
                rc = rf_mutex_init(&raidPtr->regionInfo[i].reintMutex);
                if (rc) {
                        rf_print_unable_to_init_mutex(__FILE__, __LINE__, rc);
-                       rf_mutex_destroy(&raidPtr->regionInfo[i].mutex);
                        for (j = 0; j < i; j++)
                                FreeRegionInfo(raidPtr, j);
                        RF_Free(raidPtr->regionInfo, 
@@ -531,8 +524,6 @@
                           sizeof(RF_DiskMap_t)), 
                          (RF_DiskMap_t *));
                if (raidPtr->regionInfo[i].diskMap == NULL) {
-                       rf_mutex_destroy(&raidPtr->regionInfo[i].mutex);
-                       rf_mutex_destroy(&raidPtr->regionInfo[i].reintMutex);
                        for (j = 0; j < i; j++)
                                FreeRegionInfo(raidPtr, j);
                        RF_Free(raidPtr->regionInfo, 
@@ -609,8 +600,6 @@
                RF_ASSERT(raidPtr->regionInfo[regionID].diskCount == 0);
        }
        RF_UNLOCK_MUTEX(raidPtr->regionInfo[regionID].mutex);
-       rf_mutex_destroy(&raidPtr->regionInfo[regionID].mutex);
-       rf_mutex_destroy(&raidPtr->regionInfo[regionID].reintMutex);
 }
 
 
@@ -631,7 +620,6 @@
                RF_Free(l2, sizeof(RF_ParityLog_t));
        }
        RF_UNLOCK_MUTEX(queue->mutex);
-       rf_mutex_destroy(&queue->mutex);
 }
 
 
@@ -649,7 +637,6 @@
                RF_Free(queue->buffers[i], queue->bufferSize);
        RF_Free(queue->buffers, queue->totalBuffers * sizeof(caddr_t));
        RF_UNLOCK_MUTEX(queue->mutex);
-       rf_mutex_destroy(&queue->mutex);
 }
 
 static void 
@@ -737,7 +724,6 @@
        }
        while (raidPtr->parityLogDiskQueue.freeCommonList) {
                c = raidPtr->parityLogDiskQueue.freeCommonList;
-               rf_mutex_destroy(&c->mutex);
                raidPtr->parityLogDiskQueue.freeCommonList = 
                        raidPtr->parityLogDiskQueue.freeCommonList->next;
                RF_Free(c, sizeof(RF_CommonLogData_t));
diff -r 24fb3dfcdf2f -r 3905301b7a16 sys/dev/raidframe/rf_psstatus.c
--- a/sys/dev/raidframe/rf_psstatus.c   Mon Dec 29 04:56:26 2003 +0000
+++ b/sys/dev/raidframe/rf_psstatus.c   Mon Dec 29 05:01:14 2003 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: rf_psstatus.c,v 1.17 2003/12/29 04:56:26 oster Exp $   */
+/*     $NetBSD: rf_psstatus.c,v 1.18 2003/12/29 05:01:14 oster Exp $   */
 /*
  * Copyright (c) 1995 Carnegie-Mellon University.
  * All rights reserved.
@@ -37,7 +37,7 @@
  *****************************************************************************/
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_psstatus.c,v 1.17 2003/12/29 04:56:26 oster Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_psstatus.c,v 1.18 2003/12/29 05:01:14 oster Exp $");
 
 #include <dev/raidframe/raidframevar.h>
 
@@ -139,7 +139,6 @@
                if (pssTable[i].chain) {
                        printf("ERROR: pss hash chain not null at recon shutdown\n");
                }
-               rf_mutex_destroy(&pssTable[i].mutex);
        }
        RF_Free(pssTable, raidPtr->pssTableSize * sizeof(RF_PSStatusHeader_t));
 }
diff -r 24fb3dfcdf2f -r 3905301b7a16 sys/dev/raidframe/rf_reconmap.c
--- a/sys/dev/raidframe/rf_reconmap.c   Mon Dec 29 04:56:26 2003 +0000
+++ b/sys/dev/raidframe/rf_reconmap.c   Mon Dec 29 05:01:14 2003 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: rf_reconmap.c,v 1.20 2003/12/29 04:56:26 oster Exp $   */
+/*     $NetBSD: rf_reconmap.c,v 1.21 2003/12/29 05:01:14 oster Exp $   */
 /*
  * Copyright (c) 1995 Carnegie-Mellon University.
  * All rights reserved.
@@ -34,7 +34,7 @@
  *************************************************************************/
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_reconmap.c,v 1.20 2003/12/29 04:56:26 oster Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_reconmap.c,v 1.21 2003/12/29 05:01:14 oster Exp $");
 
 #include "rf_raid.h"
 #include <sys/time.h>
@@ -303,7 +303,6 @@
                }
        }
        pool_destroy(&mapPtr->elem_pool);
-       rf_mutex_destroy(&mapPtr->mutex);
        RF_Free(mapPtr->status, mapPtr->totalRUs * 
                sizeof(RF_ReconMapListElem_t *));
        RF_Free(mapPtr, sizeof(RF_ReconMap_t));
diff -r 24fb3dfcdf2f -r 3905301b7a16 sys/dev/raidframe/rf_reconutil.c
--- a/sys/dev/raidframe/rf_reconutil.c  Mon Dec 29 04:56:26 2003 +0000
+++ b/sys/dev/raidframe/rf_reconutil.c  Mon Dec 29 05:01:14 2003 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: rf_reconutil.c,v 1.17 2003/12/29 04:39:29 oster Exp $  */
+/*     $NetBSD: rf_reconutil.c,v 1.18 2003/12/29 05:01:14 oster Exp $  */
 /*
  * Copyright (c) 1995 Carnegie-Mellon University.
  * All rights reserved.
@@ -31,7 +31,7 @@
  ********************************************/
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_reconutil.c,v 1.17 2003/12/29 04:39:29 oster Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_reconutil.c,v 1.18 2003/12/29 05:01:14 oster Exp $");
 
 #include <dev/raidframe/raidframevar.h>
 
@@ -166,8 +166,6 @@
                reconCtrlPtr->floatingRbufs = t->next;
                rf_FreeReconBuffer(t);
        }
-       rf_mutex_destroy(&reconCtrlPtr->rb_mutex);
-       rf_mutex_destroy(&reconCtrlPtr->eq_mutex);
        rf_FreeReconMap(reconCtrlPtr->reconMap);
        rf_FreeParityStripeStatusTable(raidPtr, reconCtrlPtr->pssTable);
        RF_Free(reconCtrlPtr->perDiskInfo, 
diff -r 24fb3dfcdf2f -r 3905301b7a16 sys/dev/raidframe/rf_stripelocks.c
--- a/sys/dev/raidframe/rf_stripelocks.c        Mon Dec 29 04:56:26 2003 +0000
+++ b/sys/dev/raidframe/rf_stripelocks.c        Mon Dec 29 05:01:14 2003 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: rf_stripelocks.c,v 1.18 2003/12/29 04:56:26 oster Exp $        */
+/*     $NetBSD: rf_stripelocks.c,v 1.19 2003/12/29 05:01:14 oster Exp $        */
 /*
  * Copyright (c) 1995 Carnegie-Mellon University.
  * All rights reserved.
@@ -57,7 +57,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_stripelocks.c,v 1.18 2003/12/29 04:56:26 oster Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_stripelocks.c,v 1.19 2003/12/29 05:01:14 oster Exp $");
 
 #include <dev/raidframe/raidframevar.h>
 
@@ -196,16 +196,12 @@
 static void 
 rf_ShutdownStripeLocks(RF_LockTableEntry_t * lockTable)
 {
-       int     i;
 
 #if RF_DEBUG_STRIPELOCK
        if (rf_stripeLockDebug) {
                PrintLockedStripes(lockTable);
        }
 #endif
-       for (i = 0; i < rf_lockTableSize; i++) {
-               rf_mutex_destroy(&lockTable[i].mutex);
-       }
        RF_Free(lockTable, rf_lockTableSize * sizeof(RF_LockTableEntry_t));
 }
 
diff -r 24fb3dfcdf2f -r 3905301b7a16 sys/dev/raidframe/rf_threadstuff.c
--- a/sys/dev/raidframe/rf_threadstuff.c        Mon Dec 29 04:56:26 2003 +0000
+++ b/sys/dev/raidframe/rf_threadstuff.c        Mon Dec 29 05:01:14 2003 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: rf_threadstuff.c,v 1.13 2003/12/29 04:56:26 oster Exp $        */
+/*     $NetBSD: rf_threadstuff.c,v 1.14 2003/12/29 05:01:14 oster Exp $        */
 /*
  * rf_threadstuff.c
  */
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_threadstuff.c,v 1.13 2003/12/29 04:56:26 oster Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_threadstuff.c,v 1.14 2003/12/29 05:01:14 oster Exp $");
 
 #include <dev/raidframe/raidframevar.h>
 
@@ -50,7 +50,6 @@
        void   *arg;
 {
 
-       rf_mutex_destroy(arg);
 }
 
 static void 
@@ -79,7 +78,6 @@
        rc = _rf_ShutdownCreate(listp, mutex_destroyer, (void *) m, file, line);
        if (rc) {
                RF_ERRORMSG1("RAIDFRAME: Error %d adding shutdown entry\n", rc);
-               rf_mutex_destroy(m);
        }
        return (rc);
 }
@@ -158,7 +156,6 @@
 {



Home | Main Index | Thread Index | Old Index