Correct some RF_ASSERTS() that were missed when fixing memory issues
with this code. Thanks to palle at lyckegaard.dk for pointing them out. Addresses PR#26776 (but doesn't use all the suggested fixes).
This commit is contained in:
parent
484705032c
commit
bcb300782d
|
@ -1,4 +1,4 @@
|
|||
/* $NetBSD: rf_dagdegrd.c,v 1.21 2004/03/19 15:16:18 oster Exp $ */
|
||||
/* $NetBSD: rf_dagdegrd.c,v 1.22 2004/08/27 15:55:50 oster Exp $ */
|
||||
/*
|
||||
* Copyright (c) 1995 Carnegie-Mellon University.
|
||||
* All rights reserved.
|
||||
|
@ -33,7 +33,7 @@
|
|||
*/
|
||||
|
||||
#include <sys/cdefs.h>
|
||||
__KERNEL_RCSID(0, "$NetBSD: rf_dagdegrd.c,v 1.21 2004/03/19 15:16:18 oster Exp $");
|
||||
__KERNEL_RCSID(0, "$NetBSD: rf_dagdegrd.c,v 1.22 2004/08/27 15:55:50 oster Exp $");
|
||||
|
||||
#include <dev/raidframe/raidframevar.h>
|
||||
|
||||
|
@ -525,7 +525,7 @@ rf_CreateDegradedReadDAG(RF_Raid_t *raidPtr, RF_AccessStripeMap_t *asmap,
|
|||
xorNode->antType[0] = rf_trueData;
|
||||
tmprrdNode = rrdNodes;
|
||||
for (i = 0; i < nRrdNodes; i++) {
|
||||
RF_ASSERT(rrdNode->numSuccedents == 1);
|
||||
RF_ASSERT(tmprrdNode->numSuccedents == 1);
|
||||
tmprrdNode->succedents[0] = xorNode;
|
||||
xorNode->antecedents[1 + i] = tmprrdNode;
|
||||
xorNode->antType[1 + i] = rf_trueData;
|
||||
|
|
|
@ -1,4 +1,4 @@
|
|||
/* $NetBSD: rf_dagffwr.c,v 1.27 2004/04/09 23:10:16 oster Exp $ */
|
||||
/* $NetBSD: rf_dagffwr.c,v 1.28 2004/08/27 15:55:50 oster Exp $ */
|
||||
/*
|
||||
* Copyright (c) 1995 Carnegie-Mellon University.
|
||||
* All rights reserved.
|
||||
|
@ -34,7 +34,7 @@
|
|||
*/
|
||||
|
||||
#include <sys/cdefs.h>
|
||||
__KERNEL_RCSID(0, "$NetBSD: rf_dagffwr.c,v 1.27 2004/04/09 23:10:16 oster Exp $");
|
||||
__KERNEL_RCSID(0, "$NetBSD: rf_dagffwr.c,v 1.28 2004/08/27 15:55:50 oster Exp $");
|
||||
|
||||
#include <dev/raidframe/raidframevar.h>
|
||||
|
||||
|
@ -409,13 +409,13 @@ rf_CommonCreateLargeWriteDAG(RF_Raid_t *raidPtr, RF_AccessStripeMap_t *asmap,
|
|||
RF_ASSERT(xorNode->numAntecedents == nRodNodes);
|
||||
tmpNode = rodNodes;
|
||||
for (i = 0; i < nRodNodes; i++) {
|
||||
RF_ASSERT(tmpNode.numAntecedents == 1);
|
||||
RF_ASSERT(tmpNode->numAntecedents == 1);
|
||||
blockNode->succedents[i] = tmpNode;
|
||||
tmpNode->antecedents[0] = blockNode;
|
||||
tmpNode->antType[0] = rf_control;
|
||||
|
||||
/* connect the Rod nodes to the Xor node */
|
||||
RF_ASSERT(tmpNode.numSuccedents == 1);
|
||||
RF_ASSERT(tmpNode->numSuccedents == 1);
|
||||
tmpNode->succedents[0] = xorNode;
|
||||
xorNode->antecedents[i] = tmpNode;
|
||||
xorNode->antType[i] = rf_trueData;
|
||||
|
@ -1110,7 +1110,7 @@ rf_CommonCreateSmallWriteDAG(RF_Raid_t *raidPtr, RF_AccessStripeMap_t *asmap,
|
|||
RF_ASSERT(commitNode->numSuccedents == (numDataNodes + (nfaults * numParityNodes)));
|
||||
tmpwriteDataNode = writeDataNodes;
|
||||
for (i = 0; i < numDataNodes; i++) {
|
||||
RF_ASSERT(tmpwriteDataNodes->numAntecedents == 1);
|
||||
RF_ASSERT(tmpwriteDataNode->numAntecedents == 1);
|
||||
commitNode->succedents[i] = tmpwriteDataNode;
|
||||
tmpwriteDataNode->antecedents[0] = commitNode;
|
||||
tmpwriteDataNode->antType[0] = rf_trueData;
|
||||
|
|
|
@ -1,4 +1,4 @@
|
|||
/* $NetBSD: rf_raid0.c,v 1.10 2004/01/02 21:41:08 oster Exp $ */
|
||||
/* $NetBSD: rf_raid0.c,v 1.11 2004/08/27 15:55:51 oster Exp $ */
|
||||
/*
|
||||
* Copyright (c) 1995 Carnegie-Mellon University.
|
||||
* All rights reserved.
|
||||
|
@ -33,7 +33,7 @@
|
|||
***************************************/
|
||||
|
||||
#include <sys/cdefs.h>
|
||||
__KERNEL_RCSID(0, "$NetBSD: rf_raid0.c,v 1.10 2004/01/02 21:41:08 oster Exp $");
|
||||
__KERNEL_RCSID(0, "$NetBSD: rf_raid0.c,v 1.11 2004/08/27 15:55:51 oster Exp $");
|
||||
|
||||
#include <dev/raidframe/raidframevar.h>
|
||||
|
||||
|
@ -71,7 +71,6 @@ rf_ConfigureRAID0(RF_ShutdownList_t **listp, RF_Raid_t *raidPtr,
|
|||
for (i = 0; i < raidPtr->numCol; i++)
|
||||
info->stripeIdentifier[i] = i;
|
||||
|
||||
RF_ASSERT(raidPtr->numRow == 1);
|
||||
raidPtr->totalSectors = layoutPtr->stripeUnitsPerDisk * raidPtr->numCol * layoutPtr->sectorsPerStripeUnit;
|
||||
layoutPtr->numStripe = layoutPtr->stripeUnitsPerDisk;
|
||||
layoutPtr->dataSectorsPerStripe = raidPtr->numCol * layoutPtr->sectorsPerStripeUnit;
|
||||
|
|
Loading…
Reference in New Issue