cm_volume_t * volp;
for (volp = cm_data.allVolumesp; volp; volp=volp->allNextp) {
-
- if (volp->rw.ID)
- cm_VolumeStatusNotification(volp, volp->rw.ID, volp->rw.state, vl_alldown);
- if (volp->ro.ID)
- cm_VolumeStatusNotification(volp, volp->ro.ID, volp->ro.state, vl_alldown);
- if (volp->bk.ID)
- cm_VolumeStatusNotification(volp, volp->bk.ID, volp->bk.state, vl_alldown);
+ afs_uint32 volType;
+ for ( volType = RWVOL; volType < NUM_VOL_TYPES; volType++) {
+ if (volp->vol[volType].ID)
+ cm_VolumeStatusNotification(volp, volp->vol[volType].ID, volp->vol[volType].state, vl_alldown);
+ }
volp->cbExpiresRO = 0;
- lock_FinalizeMutex(&volp->mx);
+ lock_FinalizeRWLock(&volp->rw);
}
return 0;
cm_volume_t * volp;
for (volp = cm_data.allVolumesp; volp; volp=volp->allNextp) {
- lock_InitializeMutex(&volp->mx, "cm_volume_t mutex");
+ afs_uint32 volType;
+
+ lock_InitializeRWLock(&volp->rw, "cm_volume_t rwlock");
volp->flags |= CM_VOLUMEFLAG_RESET;
- volp->rw.state = vl_unknown;
- volp->rw.serversp = NULL;
- volp->ro.state = vl_unknown;
- volp->ro.serversp = NULL;
- volp->bk.state = vl_unknown;
- volp->bk.serversp = NULL;
- if (volp->rw.ID)
- cm_VolumeStatusNotification(volp, volp->rw.ID, vl_alldown, volp->rw.state);
- if (volp->ro.ID)
- cm_VolumeStatusNotification(volp, volp->ro.ID, vl_alldown, volp->ro.state);
- if (volp->bk.ID)
- cm_VolumeStatusNotification(volp, volp->bk.ID, vl_alldown, volp->bk.state);
+ volp->flags &= ~CM_VOLUMEFLAG_UPDATING_VL;
+ for (volType = RWVOL; volType < NUM_VOL_TYPES; volType++) {
+ volp->vol[volType].state = vl_unknown;
+ volp->vol[volType].serversp = NULL;
+ if (volp->vol[volType].ID)
+ cm_VolumeStatusNotification(volp, volp->vol[volType].ID, vl_alldown, volp->vol[volType].state);
+ }
volp->cbExpiresRO = 0;
}
}
/*
- * Update a volume. Caller holds volume's lock (volp->mx).
+ * Update a volume. Caller holds a write lock on the volume (volp->rw).
*
*
* shadow / openafs / jhutz@CS.CMU.EDU {ANDREW.CMU.EDU} 01:38 (JHutz)
* RXGEN_OPCODE.
*/
#define MULTIHOMED 1
-long cm_UpdateVolume(struct cm_cell *cellp, cm_user_t *userp, cm_req_t *reqp,
+long cm_UpdateVolumeLocation(struct cm_cell *cellp, cm_user_t *userp, cm_req_t *reqp,
cm_volume_t *volp)
{
cm_conn_t *connp;
#ifdef AFS_FREELANCE_CLIENT
int freelance = 0;
#endif
-
- /* clear out old bindings */
- if (volp->rw.serversp)
- cm_FreeServerList(&volp->rw.serversp, CM_FREESERVERLIST_DELETE);
- if (volp->ro.serversp)
- cm_FreeServerList(&volp->ro.serversp, CM_FREESERVERLIST_DELETE);
- if (volp->bk.serversp)
- cm_FreeServerList(&volp->bk.serversp, CM_FREESERVERLIST_DELETE);
+ afs_uint32 volType;
#ifdef AFS_FREELANCE_CLIENT
- if ( cellp->cellID == AFS_FAKE_ROOT_CELL_ID && volp->rw.ID == AFS_FAKE_ROOT_VOL_ID )
+ if ( cellp->cellID == AFS_FAKE_ROOT_CELL_ID && volp->vol[RWVOL].ID == AFS_FAKE_ROOT_VOL_ID )
{
freelance = 1;
memset(&vldbEntry, 0, sizeof(vldbEntry));
} else
#endif
{
+ while (volp->flags & CM_VOLUMEFLAG_UPDATING_VL) {
+ osi_Log3(afsd_logp, "cm_UpdateVolumeLocation sleeping name %s:%s flags 0x%x",
+ volp->cellp->name, volp->namep, volp->flags);
+ osi_SleepW((LONG_PTR) &volp->flags, &volp->rw);
+ lock_ObtainWrite(&volp->rw);
+ osi_Log3(afsd_logp, "cm_UpdateVolumeLocation awake name %s:%s flags 0x%x",
+ volp->cellp->name, volp->namep, volp->flags);
+ if (!(volp->flags & CM_VOLUMEFLAG_RESET)) {
+ osi_Log3(afsd_logp, "cm_UpdateVolumeLocation nothing to do, waking others name %s:%s flags 0x%x",
+ volp->cellp->name, volp->namep, volp->flags);
+ osi_Wakeup((LONG_PTR) &volp->flags);
+ return 0;
+ }
+ }
+
+ /* clear out old bindings */
+ for ( volType = RWVOL; volType < NUM_VOL_TYPES; volType++) {
+ if (volp->vol[volType].serversp)
+ cm_FreeServerList(&volp->vol[volType].serversp, CM_FREESERVERLIST_DELETE);
+ }
+
+ volp->flags |= CM_VOLUMEFLAG_UPDATING_VL;
+ lock_ReleaseWrite(&volp->rw);
+
+ if (cellp->flags & CM_CELLFLAG_VLSERVER_INVALID)
+ cm_UpdateCell(cellp, 0);
+
/* now we have volume structure locked and held; make RPC to fill it */
osi_Log2(afsd_logp, "CALL VL_GetEntryByName{UNO} name %s:%s", volp->cellp->name, volp->namep);
do {
+ struct rx_connection * rxconnp;
+
code = cm_ConnByMServers(cellp->vlServersp, userp, reqp, &connp);
if (code)
continue;
+
+ rxconnp = cm_GetRxConn(connp);
#ifdef MULTIHOMED
- code = VL_GetEntryByNameU(connp->callp, volp->namep, &uvldbEntry);
+ code = VL_GetEntryByNameU(rxconnp, volp->namep, &uvldbEntry);
method = 2;
if ( code == RXGEN_OPCODE )
#endif
{
- code = VL_GetEntryByNameN(connp->callp, volp->namep, &nvldbEntry);
+ code = VL_GetEntryByNameN(rxconnp, volp->namep, &nvldbEntry);
method = 1;
}
if ( code == RXGEN_OPCODE ) {
- code = VL_GetEntryByNameO(connp->callp, volp->namep, &vldbEntry);
+ code = VL_GetEntryByNameO(rxconnp, volp->namep, &vldbEntry);
method = 0;
}
+ rx_PutConnection(rxconnp);
} while (cm_Analyze(connp, userp, reqp, NULL, NULL, cellp->vlServersp, NULL, code));
code = cm_MapVLRPCError(code, reqp);
if ( code )
* doesn't exist we will not care about the .backup that might be left behind
* since there should be no method to access it.
*/
- if (code == CM_ERROR_NOSUCHVOLUME && volp->rw.ID == 0 && strlen(volp->namep) < (VL_MAXNAMELEN - 9)) {
+ if (code == CM_ERROR_NOSUCHVOLUME && volp->vol[RWVOL].ID == 0 && strlen(volp->namep) < (VL_MAXNAMELEN - 9)) {
char name[VL_MAXNAMELEN];
snprintf(name, VL_MAXNAMELEN, "%s.readonly", volp->namep);
/* now we have volume structure locked and held; make RPC to fill it */
- osi_Log2(afsd_logp, "CALL VL_GetEntryByName{UNO} name %s:%s", volp->cellp->name, name);
+ osi_Log2(afsd_logp, "CALL VL_GetEntryByName{UNO} name %s:%s", volp->cellp->name,
+ osi_LogSaveString(afsd_logp,name));
do {
+ struct rx_connection * rxconnp;
+
code = cm_ConnByMServers(cellp->vlServersp, userp, reqp, &connp);
if (code)
continue;
+
+ rxconnp = cm_GetRxConn(connp);
#ifdef MULTIHOMED
- code = VL_GetEntryByNameU(connp->callp, name, &uvldbEntry);
+ code = VL_GetEntryByNameU(connp->rxconnp, name, &uvldbEntry);
method = 2;
if ( code == RXGEN_OPCODE )
#endif
{
- code = VL_GetEntryByNameN(connp->callp, name, &nvldbEntry);
+ code = VL_GetEntryByNameN(connp->rxconnp, name, &nvldbEntry);
method = 1;
}
if ( code == RXGEN_OPCODE ) {
- code = VL_GetEntryByNameO(connp->callp, name, &vldbEntry);
+ code = VL_GetEntryByNameO(connp->rxconnp, name, &vldbEntry);
method = 0;
}
+ rx_PutConnection(rxconnp);
} while (cm_Analyze(connp, userp, reqp, NULL, NULL, cellp->vlServersp, NULL, code));
code = cm_MapVLRPCError(code, reqp);
if ( code )
osi_Log3(afsd_logp, "CALL VL_GetEntryByName{UNO} name %s:%s FAILURE, code 0x%x",
- volp->cellp->name, name, code);
+ volp->cellp->name, osi_LogSaveString(afsd_logp,name), code);
else
osi_Log2(afsd_logp, "CALL VL_GetEntryByName{UNO} name %s:%s SUCCESS",
- volp->cellp->name, name);
+ volp->cellp->name, osi_LogSaveString(afsd_logp,name));
}
-
+
+ lock_ObtainWrite(&volp->rw);
if (code == 0) {
afs_int32 flags;
afs_int32 nServers;
rwID = uvldbEntry.volumeId[0];
roID = uvldbEntry.volumeId[1];
bkID = uvldbEntry.volumeId[2];
- for ( i=0, j=0; i<nServers && j<NMAXNSERVERS; i++ ) {
+ for ( i=0, j=0; code == 0 && i<nServers && j<NMAXNSERVERS; i++ ) {
if ( !(uvldbEntry.serverFlags[i] & VLSERVER_FLAG_UUID) ) {
serverFlags[j] = uvldbEntry.serverFlags[i];
serverNumber[j] = uvldbEntry.serverNumber[i].time_low;
memset((char *)&addrs, 0, sizeof(addrs));
do {
+ struct rx_connection *rxconnp;
+
code = cm_ConnByMServers(cellp->vlServersp, userp, reqp, &connp);
if (code)
continue;
- code = VL_GetAddrsU(connp->callp, &attrs, &uuid, &unique, &nentries, &addrs);
-
- if (code == 0 && nentries == 0)
- code = VL_NOENT;
+ rxconnp = cm_GetRxConn(connp);
+ code = VL_GetAddrsU(rxconnp, &attrs, &uuid, &unique, &nentries, &addrs);
+ rx_PutConnection(rxconnp);
} while (cm_Analyze(connp, userp, reqp, NULL, NULL, cellp->vlServersp, NULL, code));
- code = cm_MapVLRPCError(code, reqp);
- if (code)
- return code;
+
+ if ( code ) {
+ code = cm_MapVLRPCError(code, reqp);
+ osi_Log2(afsd_logp, "CALL VL_GetAddrsU serverNumber %u FAILURE, code 0x%x",
+ i, code);
+ continue;
+ }
+ osi_Log1(afsd_logp, "CALL VL_GetAddrsU serverNumber %u SUCCESS", i);
addrp = addrs.bulkaddrs_val;
for (k = 0; k < nentries && j < NMAXNSERVERS; j++, k++) {
}
free(addrs.bulkaddrs_val); /* This is wrong */
+
+ if (nentries == 0)
+ code = CM_ERROR_INVAL;
}
}
nServers = j; /* update the server count */
}
if (flags & VLF_RWEXISTS) {
- if (volp->rw.ID != rwID) {
- if (volp->rw.flags & CM_VOLUMEFLAG_IN_HASH)
+ if (volp->vol[RWVOL].ID != rwID) {
+ if (volp->vol[RWVOL].flags & CM_VOLUMEFLAG_IN_HASH)
cm_RemoveVolumeFromIDHashTable(volp, RWVOL);
- volp->rw.ID = rwID;
+ volp->vol[RWVOL].ID = rwID;
cm_AddVolumeToIDHashTable(volp, RWVOL);
}
} else {
- if (volp->rw.flags & CM_VOLUMEFLAG_IN_HASH)
+ if (volp->vol[RWVOL].flags & CM_VOLUMEFLAG_IN_HASH)
cm_RemoveVolumeFromIDHashTable(volp, RWVOL);
- volp->rw.ID = 0;
+ volp->vol[RWVOL].ID = 0;
}
if (flags & VLF_ROEXISTS) {
- if (volp->ro.ID != roID) {
- if (volp->ro.flags & CM_VOLUMEFLAG_IN_HASH)
+ if (volp->vol[ROVOL].ID != roID) {
+ if (volp->vol[ROVOL].flags & CM_VOLUMEFLAG_IN_HASH)
cm_RemoveVolumeFromIDHashTable(volp, ROVOL);
- volp->ro.ID = roID;
+ volp->vol[ROVOL].ID = roID;
cm_AddVolumeToIDHashTable(volp, ROVOL);
}
} else {
- if (volp->ro.flags & CM_VOLUMEFLAG_IN_HASH)
+ if (volp->vol[ROVOL].flags & CM_VOLUMEFLAG_IN_HASH)
cm_RemoveVolumeFromIDHashTable(volp, ROVOL);
- volp->ro.ID = 0;
+ volp->vol[ROVOL].ID = 0;
}
if (flags & VLF_BACKEXISTS) {
- if (volp->bk.ID != bkID) {
- if (volp->bk.flags & CM_VOLUMEFLAG_IN_HASH)
+ if (volp->vol[BACKVOL].ID != bkID) {
+ if (volp->vol[BACKVOL].flags & CM_VOLUMEFLAG_IN_HASH)
cm_RemoveVolumeFromIDHashTable(volp, BACKVOL);
- volp->bk.ID = bkID;
+ volp->vol[BACKVOL].ID = bkID;
cm_AddVolumeToIDHashTable(volp, BACKVOL);
}
} else {
- if (volp->bk.flags & CM_VOLUMEFLAG_IN_HASH)
+ if (volp->vol[BACKVOL].flags & CM_VOLUMEFLAG_IN_HASH)
cm_RemoveVolumeFromIDHashTable(volp, BACKVOL);
- volp->bk.ID = 0;
+ volp->vol[BACKVOL].ID = 0;
}
lock_ReleaseWrite(&cm_volumeLock);
for (i=0; i<nServers; i++) {
tempAddr = htonl(serverNumber[i]);
tsockAddr.sin_addr.s_addr = tempAddr;
tsp = cm_FindServer(&tsockAddr, CM_SERVER_FILE);
- if (!tsp)
- tsp = cm_NewServer(&tsockAddr, CM_SERVER_FILE,
- cellp, 0);
-
+ if (!tsp) {
+ /* cm_NewServer will probe the server which in turn will
+ * update the state on the volume group object */
+ lock_ReleaseWrite(&volp->rw);
+ tsp = cm_NewServer(&tsockAddr, CM_SERVER_FILE, cellp, 0);
+ lock_ObtainWrite(&volp->rw);
+ }
/* if this server was created by fs setserverprefs */
if ( !tsp->cellp )
tsp->cellp = cellp;
*/
if ((tflags & VLSF_RWVOL) && (flags & VLF_RWEXISTS)) {
tsrp = cm_NewServerRef(tsp, rwID);
- cm_InsertServerList(&volp->rw.serversp, tsrp);
+ cm_InsertServerList(&volp->vol[RWVOL].serversp, tsrp);
lock_ObtainWrite(&cm_serverLock);
tsrp->refCount--; /* drop allocation reference */
}
if ((tflags & VLSF_ROVOL) && (flags & VLF_ROEXISTS)) {
tsrp = cm_NewServerRef(tsp, roID);
- cm_InsertServerList(&volp->ro.serversp, tsrp);
+ cm_InsertServerList(&volp->vol[ROVOL].serversp, tsrp);
lock_ObtainWrite(&cm_serverLock);
tsrp->refCount--; /* drop allocation reference */
lock_ReleaseWrite(&cm_serverLock);
*/
if ((tflags & VLSF_RWVOL) && (flags & VLF_BACKEXISTS)) {
tsrp = cm_NewServerRef(tsp, bkID);
- cm_InsertServerList(&volp->bk.serversp, tsrp);
+ cm_InsertServerList(&volp->vol[BACKVOL].serversp, tsrp);
lock_ObtainWrite(&cm_serverLock);
tsrp->refCount--; /* drop allocation reference */
lock_ReleaseWrite(&cm_serverLock);
* lists are length 1.
*/
if (ROcount > 1) {
- cm_RandomizeServer(&volp->ro.serversp);
+ cm_RandomizeServer(&volp->vol[ROVOL].serversp);
}
rwNewstate = rwServers_alldown ? vl_alldown : vl_online;
roNewstate = roServers_alldown ? vl_alldown : vl_online;
bkNewstate = bkServers_alldown ? vl_alldown : vl_online;
+ } else if (code == CM_ERROR_NOSUCHVOLUME || code == VL_NOENT) {
+ /* this volume does not exist - we should discard it */
+ if (volp->flags & CM_VOLUMEFLAG_IN_HASH)
+ cm_RemoveVolumeFromNameHashTable(volp);
+ for ( volType = RWVOL; volType < NUM_VOL_TYPES; volType++) {
+ if (volp->vol[volType].flags & CM_VOLUMEFLAG_IN_HASH)
+ cm_RemoveVolumeFromIDHashTable(volp, volType);
+ if (volp->vol[volType].ID) {
+ cm_VolumeStatusNotification(volp, volp->vol[volType].ID, volp->vol[volType].state, vl_alldown);
+ volp->vol[volType].ID = 0;
+ }
+ cm_SetFid(&volp->vol[volType].dotdotFid, 0, 0, 0, 0);
+ }
+
+ /* Move to the end so it will be recycled first */
+ cm_MoveVolumeToLRULast(volp);
+
+ volp->namep[0] ='\0';
} else {
rwNewstate = roNewstate = bkNewstate = vl_alldown;
}
- if (volp->rw.state != rwNewstate) {
- if (volp->rw.ID)
- cm_VolumeStatusNotification(volp, volp->rw.ID, volp->rw.state, rwNewstate);
- volp->rw.state = rwNewstate;
+ if (volp->vol[RWVOL].state != rwNewstate) {
+ if (volp->vol[RWVOL].ID)
+ cm_VolumeStatusNotification(volp, volp->vol[RWVOL].ID, volp->vol[RWVOL].state, rwNewstate);
+ volp->vol[RWVOL].state = rwNewstate;
}
- if (volp->ro.state != roNewstate) {
- if (volp->ro.ID)
- cm_VolumeStatusNotification(volp, volp->ro.ID, volp->ro.state, roNewstate);
- volp->ro.state = roNewstate;
+ if (volp->vol[ROVOL].state != roNewstate) {
+ if (volp->vol[ROVOL].ID)
+ cm_VolumeStatusNotification(volp, volp->vol[ROVOL].ID, volp->vol[ROVOL].state, roNewstate);
+ volp->vol[ROVOL].state = roNewstate;
}
- if (volp->bk.state != bkNewstate) {
- if (volp->bk.ID)
- cm_VolumeStatusNotification(volp, volp->bk.ID, volp->bk.state, bkNewstate);
- volp->bk.state = bkNewstate;
+ if (volp->vol[BACKVOL].state != bkNewstate) {
+ if (volp->vol[BACKVOL].ID)
+ cm_VolumeStatusNotification(volp, volp->vol[BACKVOL].ID, volp->vol[BACKVOL].state, bkNewstate);
+ volp->vol[BACKVOL].state = bkNewstate;
}
+ if (code == 0)
+ volp->flags &= ~CM_VOLUMEFLAG_RESET;
+
+ volp->flags &= ~CM_VOLUMEFLAG_UPDATING_VL;
+ osi_Log4(afsd_logp, "cm_UpdateVolumeLocation done, waking others name %s:%s flags 0x%x code 0x%x",
+ volp->cellp->name, volp->namep, volp->flags, code);
+ osi_Wakeup((LONG_PTR) &volp->flags);
+
return code;
}
+/* Requires read or write lock on cm_volumeLock */
void cm_GetVolume(cm_volume_t *volp)
{
- if (volp) {
- lock_ObtainWrite(&cm_volumeLock);
- volp->refCount++;
- lock_ReleaseWrite(&cm_volumeLock);
- }
+ InterlockedIncrement(&volp->refCount);
}
+cm_volume_t *cm_GetVolumeByFID(cm_fid_t *fidp)
+{
+ cm_volume_t *volp;
+ afs_uint32 hash;
-long cm_GetVolumeByID(cm_cell_t *cellp, afs_uint32 volumeID, cm_user_t *userp,
+ lock_ObtainRead(&cm_volumeLock);
+ hash = CM_VOLUME_ID_HASH(fidp->volume);
+ /* The volumeID can be any one of the three types. So we must
+ * search the hash table for all three types until we find it.
+ * We will search in the order of RO, RW, BK.
+ */
+ for ( volp = cm_data.volumeROIDHashTablep[hash]; volp; volp = volp->vol[ROVOL].nextp) {
+ if ( fidp->cell == volp->cellp->cellID && fidp->volume == volp->vol[ROVOL].ID )
+ break;
+ }
+ if (!volp) {
+ /* try RW volumes */
+ for ( volp = cm_data.volumeRWIDHashTablep[hash]; volp; volp = volp->vol[RWVOL].nextp) {
+ if ( fidp->cell == volp->cellp->cellID && fidp->volume == volp->vol[RWVOL].ID )
+ break;
+ }
+ }
+ if (!volp) {
+ /* try BK volumes */
+ for ( volp = cm_data.volumeBKIDHashTablep[hash]; volp; volp = volp->vol[BACKVOL].nextp) {
+ if ( fidp->cell == volp->cellp->cellID && fidp->volume == volp->vol[BACKVOL].ID )
+ break;
+ }
+ }
+
+ /* hold the volume if we found it */
+ if (volp)
+ cm_GetVolume(volp);
+
+ lock_ReleaseRead(&cm_volumeLock);
+ return volp;
+}
+
+long cm_FindVolumeByID(cm_cell_t *cellp, afs_uint32 volumeID, cm_user_t *userp,
cm_req_t *reqp, afs_uint32 flags, cm_volume_t **outVolpp)
{
cm_volume_t *volp;
#ifdef SEARCH_ALL_VOLUMES
for(volp = cm_data.allVolumesp; volp; volp=volp->allNextp) {
if (cellp == volp->cellp &&
- ((unsigned) volumeID == volp->rw.ID ||
- (unsigned) volumeID == volp->ro.ID ||
- (unsigned) volumeID == volp->bk.ID))
+ ((unsigned) volumeID == volp->vol[RWVOL].ID ||
+ (unsigned) volumeID == volp->vol[ROVOL].ID ||
+ (unsigned) volumeID == volp->vol[BACKVOL].ID))
break;
}
* search the hash table for all three types until we find it.
* We will search in the order of RO, RW, BK.
*/
- for ( volp = cm_data.volumeROIDHashTablep[hash]; volp; volp = volp->ro.nextp) {
- if ( cellp == volp->cellp && volumeID == volp->ro.ID )
+ for ( volp = cm_data.volumeROIDHashTablep[hash]; volp; volp = volp->vol[ROVOL].nextp) {
+ if ( cellp == volp->cellp && volumeID == volp->vol[ROVOL].ID )
break;
}
if (!volp) {
/* try RW volumes */
- for ( volp = cm_data.volumeRWIDHashTablep[hash]; volp; volp = volp->rw.nextp) {
- if ( cellp == volp->cellp && volumeID == volp->rw.ID )
+ for ( volp = cm_data.volumeRWIDHashTablep[hash]; volp; volp = volp->vol[RWVOL].nextp) {
+ if ( cellp == volp->cellp && volumeID == volp->vol[RWVOL].ID )
break;
}
}
if (!volp) {
/* try BK volumes */
- for ( volp = cm_data.volumeBKIDHashTablep[hash]; volp; volp = volp->bk.nextp) {
- if ( cellp == volp->cellp && volumeID == volp->bk.ID )
+ for ( volp = cm_data.volumeBKIDHashTablep[hash]; volp; volp = volp->vol[BACKVOL].nextp) {
+ if ( cellp == volp->cellp && volumeID == volp->vol[BACKVOL].ID )
break;
}
}
osi_assertx(volp == volp2, "unexpected cm_vol_t");
#endif
- lock_ReleaseRead(&cm_volumeLock);
-
/* hold the volume if we found it */
if (volp)
cm_GetVolume(volp);
+ lock_ReleaseRead(&cm_volumeLock);
+
/* return it held */
if (volp) {
- lock_ObtainMutex(&volp->mx);
+ lock_ObtainWrite(&volp->rw);
- code = 0;
- if (volp->flags & CM_VOLUMEFLAG_RESET) {
- code = cm_UpdateVolume(cellp, userp, reqp, volp);
- if (code == 0)
- volp->flags &= ~CM_VOLUMEFLAG_RESET;
- }
- lock_ReleaseMutex(&volp->mx);
- if (code == 0) {
- *outVolpp = volp;
-
- lock_ObtainWrite(&cm_volumeLock);
- cm_AdjustVolumeLRU(volp);
- lock_ReleaseWrite(&cm_volumeLock);
- } else
- cm_PutVolume(volp);
-
- return code;
+ code = 0;
+ if ((volp->flags & CM_VOLUMEFLAG_RESET) && !(flags & CM_GETVOL_FLAG_NO_RESET)) {
+ code = cm_UpdateVolumeLocation(cellp, userp, reqp, volp);
+ }
+ lock_ReleaseWrite(&volp->rw);
+ if (code == 0) {
+ *outVolpp = volp;
+
+ if (!(flags & CM_GETVOL_FLAG_NO_LRU_UPDATE)) {
+ lock_ObtainWrite(&cm_volumeLock);
+ cm_AdjustVolumeLRU(volp);
+ lock_ReleaseWrite(&cm_volumeLock);
+ }
+ } else {
+ lock_ObtainRead(&cm_volumeLock);
+ cm_PutVolume(volp);
+ lock_ReleaseRead(&cm_volumeLock);
+ }
+ return code;
}
/* otherwise, we didn't find it so consult the VLDB */
sprintf(volNameString, "%u", volumeID);
- code = cm_GetVolumeByName(cellp, volNameString, userp, reqp,
+ code = cm_FindVolumeByName(cellp, volNameString, userp, reqp,
flags, outVolpp);
return code;
}
-long cm_GetVolumeByName(struct cm_cell *cellp, char *volumeNamep,
+long cm_FindVolumeByName(struct cm_cell *cellp, char *volumeNamep,
struct cm_user *userp, struct cm_req *reqp,
afs_uint32 flags, cm_volume_t **outVolpp)
{
#endif
if (!volp && (flags & CM_GETVOL_FLAG_CREATE)) {
+ afs_uint32 volType;
/* otherwise, get from VLDB */
- if ( cm_data.currentVolumes >= cm_data.maxVolumes ) {
+ /*
+ * Change to a write lock so that we have exclusive use of
+ * the first cm_volume_t with a refCount of 0 so that we
+ * have time to increment it.
+ */
+ lock_ConvertRToW(&cm_volumeLock);
+ if ( cm_data.currentVolumes >= cm_data.maxVolumes ) {
#ifdef RECYCLE_FROM_ALL_VOLUMES_LIST
for (volp = cm_data.allVolumesp; volp; volp=volp->allNextp) {
if ( volp->refCount == 0 ) {
if (!volp)
osi_panic("Exceeded Max Volumes", __FILE__, __LINE__);
- lock_ReleaseRead(&cm_volumeLock);
- lock_ObtainMutex(&volp->mx);
+ InterlockedIncrement(&volp->refCount);
+ lock_ReleaseWrite(&cm_volumeLock);
+ lock_ObtainWrite(&volp->rw);
lock_ObtainWrite(&cm_volumeLock);
osi_Log2(afsd_logp, "Recycling Volume %s:%s",
cm_RemoveVolumeFromLRU(volp);
if (volp->flags & CM_VOLUMEFLAG_IN_HASH)
cm_RemoveVolumeFromNameHashTable(volp);
- if (volp->rw.flags & CM_VOLUMEFLAG_IN_HASH)
- cm_RemoveVolumeFromIDHashTable(volp, RWVOL);
- if (volp->ro.flags & CM_VOLUMEFLAG_IN_HASH)
- cm_RemoveVolumeFromIDHashTable(volp, ROVOL);
- if (volp->bk.flags & CM_VOLUMEFLAG_IN_HASH)
- cm_RemoveVolumeFromIDHashTable(volp, BACKVOL);
- if (volp->rw.ID)
- cm_VolumeStatusNotification(volp, volp->rw.ID, volp->rw.state, vl_unknown);
- if (volp->ro.ID)
- cm_VolumeStatusNotification(volp, volp->ro.ID, volp->ro.state, vl_unknown);
- if (volp->bk.ID)
- cm_VolumeStatusNotification(volp, volp->bk.ID, volp->bk.state, vl_unknown);
-
- volp->rw.ID = volp->ro.ID = volp->bk.ID = 0;
- volp->dotdotFid.cell = 0;
- volp->dotdotFid.volume = 0;
- volp->dotdotFid.unique = 0;
- volp->dotdotFid.vnode = 0;
+ for ( volType = RWVOL; volType < NUM_VOL_TYPES; volType++) {
+ if (volp->vol[volType].flags & CM_VOLUMEFLAG_IN_HASH)
+ cm_RemoveVolumeFromIDHashTable(volp, volType);
+ if (volp->vol[volType].ID)
+ cm_VolumeStatusNotification(volp, volp->vol[volType].ID, volp->vol[volType].state, vl_unknown);
+ volp->vol[volType].ID = 0;
+ cm_SetFid(&volp->vol[volType].dotdotFid, 0, 0, 0, 0);
+ }
} else {
volp = &cm_data.volumeBaseAddress[cm_data.currentVolumes++];
memset(volp, 0, sizeof(cm_volume_t));
volp->magic = CM_VOLUME_MAGIC;
volp->allNextp = cm_data.allVolumesp;
cm_data.allVolumesp = volp;
- lock_InitializeMutex(&volp->mx, "cm_volume_t mutex");
- lock_ReleaseRead(&cm_volumeLock);
- lock_ObtainMutex(&volp->mx);
+ lock_InitializeRWLock(&volp->rw, "cm_volume_t rwlock");
+ lock_ReleaseWrite(&cm_volumeLock);
+ lock_ObtainWrite(&volp->rw);
lock_ObtainWrite(&cm_volumeLock);
+ volp->refCount = 1; /* starts off held */
}
volp->cellp = cellp;
strncpy(volp->namep, name, VL_MAXNAMELEN);
volp->namep[VL_MAXNAMELEN-1] = '\0';
- volp->refCount = 1; /* starts off held */
volp->flags = CM_VOLUMEFLAG_RESET;
- volp->rw.state = volp->ro.state = volp->bk.state = vl_unknown;
- volp->rw.nextp = volp->ro.nextp = volp->bk.nextp = NULL;
- volp->rw.flags = volp->ro.flags = volp->bk.flags = 0;
+
+ for ( volType = RWVOL; volType < NUM_VOL_TYPES; volType++) {
+ volp->vol[volType].state = vl_unknown;
+ volp->vol[volType].nextp = NULL;
+ volp->vol[volType].flags = 0;
+ }
volp->cbExpiresRO = 0;
cm_AddVolumeToNameHashTable(volp);
lock_ReleaseWrite(&cm_volumeLock);
}
else {
- lock_ReleaseRead(&cm_volumeLock);
- if (volp) {
+ if (volp)
cm_GetVolume(volp);
- lock_ObtainMutex(&volp->mx);
- } else {
+ lock_ReleaseRead(&cm_volumeLock);
+
+ if (!volp)
return CM_ERROR_NOSUCHVOLUME;
- }
+
+ lock_ObtainWrite(&volp->rw);
}
/* if we get here we are holding the mutex */
- if (volp->flags & CM_VOLUMEFLAG_RESET) {
- code = cm_UpdateVolume(cellp, userp, reqp, volp);
- if (code == 0)
- volp->flags &= ~CM_VOLUMEFLAG_RESET;
+ if ((volp->flags & CM_VOLUMEFLAG_RESET) && !(flags & CM_GETVOL_FLAG_NO_RESET)) {
+ code = cm_UpdateVolumeLocation(cellp, userp, reqp, volp);
}
- lock_ReleaseMutex(&volp->mx);
+ lock_ReleaseWrite(&volp->rw);
- if (code == 0 && (type == BACKVOL && volp->bk.ID == 0 ||
- type == ROVOL && volp->ro.ID == 0))
+ if (code == 0 && (type == BACKVOL && volp->vol[BACKVOL].ID == 0 ||
+ type == ROVOL && volp->vol[ROVOL].ID == 0))
code = CM_ERROR_NOSUCHVOLUME;
if (code == 0) {
- *outVolpp = volp;
-
- lock_ObtainWrite(&cm_volumeLock);
- cm_AdjustVolumeLRU(volp);
- lock_ReleaseWrite(&cm_volumeLock);
- } else
- cm_PutVolume(volp);
-
+ *outVolpp = volp;
+
+ if (!(flags & CM_GETVOL_FLAG_NO_LRU_UPDATE)) {
+ lock_ObtainWrite(&cm_volumeLock);
+ cm_AdjustVolumeLRU(volp);
+ lock_ReleaseWrite(&cm_volumeLock);
+ }
+ } else {
+ lock_ObtainRead(&cm_volumeLock);
+ cm_PutVolume(volp);
+ lock_ReleaseRead(&cm_volumeLock);
+ }
return code;
}
#ifdef SEARCH_ALL_VOLUMES
for(volp = cm_data.allVolumesp; volp; volp=volp->allNextp) {
if (cellp == volp->cellp &&
- (fidp->volume == volp->rw.ID ||
- fidp->volume == volp->ro.ID ||
- fidp->volume == volp->bk.ID))
+ (fidp->volume == volp->vol[RWVOL].ID ||
+ fidp->volume == volp->vol[ROVOL].ID ||
+ fidp->volume == volp->vol[BACKVOL].ID))
break;
}
#endif /* SEARCH_ALL_VOLUMES */
* search the hash table for all three types until we find it.
* We will search in the order of RO, RW, BK.
*/
- for ( volp = cm_data.volumeROIDHashTablep[hash]; volp; volp = volp->ro.nextp) {
- if ( cellp == volp->cellp && fidp->volume == volp->ro.ID )
+ for ( volp = cm_data.volumeROIDHashTablep[hash]; volp; volp = volp->vol[ROVOL].nextp) {
+ if ( cellp == volp->cellp && fidp->volume == volp->vol[ROVOL].ID )
break;
}
if (!volp) {
/* try RW volumes */
- for ( volp = cm_data.volumeRWIDHashTablep[hash]; volp; volp = volp->rw.nextp) {
- if ( cellp == volp->cellp && fidp->volume == volp->rw.ID )
+ for ( volp = cm_data.volumeRWIDHashTablep[hash]; volp; volp = volp->vol[RWVOL].nextp) {
+ if ( cellp == volp->cellp && fidp->volume == volp->vol[RWVOL].ID )
break;
}
}
if (!volp) {
/* try BK volumes */
- for ( volp = cm_data.volumeBKIDHashTablep[hash]; volp; volp = volp->bk.nextp) {
- if ( cellp == volp->cellp && fidp->volume == volp->bk.ID )
+ for ( volp = cm_data.volumeBKIDHashTablep[hash]; volp; volp = volp->vol[BACKVOL].nextp) {
+ if ( cellp == volp->cellp && fidp->volume == volp->vol[BACKVOL].ID )
break;
}
}
#ifdef SEARCH_ALL_VOLUMES
osi_assertx(volp == volp2, "unexpected cm_vol_t");
#endif
-
- lock_ReleaseRead(&cm_volumeLock);
-
/* hold the volume if we found it */
if (volp)
cm_GetVolume(volp);
+ lock_ReleaseRead(&cm_volumeLock);
+
+ if (!volp)
+ return;
+
/* update it */
cm_data.mountRootGen = time(NULL);
- lock_ObtainMutex(&volp->mx);
+ lock_ObtainWrite(&volp->rw);
volp->flags |= CM_VOLUMEFLAG_RESET;
#ifdef COMMENT
/* Mark the volume to be updated but don't update it now.
* accessed by Name or ID the UpdateVolume call will
* occur.
*/
- code = cm_UpdateVolume(cellp, userp, reqp, volp);
- if (code == 0)
- volp->flags &= ~CM_VOLUMEFLAG_RESET;
+ code = cm_UpdateVolumeLocation(cellp, userp, reqp, volp);
#endif
- lock_ReleaseMutex(&volp->mx);
+ lock_ReleaseWrite(&volp->rw);
+ lock_ObtainRead(&cm_volumeLock);
cm_PutVolume(volp);
+ lock_ReleaseRead(&cm_volumeLock);
}
/* find the appropriate servers from a volume */
lock_ObtainWrite(&cm_serverLock);
- if (volume == volp->rw.ID)
- serverspp = &volp->rw.serversp;
- else if (volume == volp->ro.ID)
- serverspp = &volp->ro.serversp;
- else if (volume == volp->bk.ID)
- serverspp = &volp->bk.serversp;
+ if (volume == volp->vol[RWVOL].ID)
+ serverspp = &volp->vol[RWVOL].serversp;
+ else if (volume == volp->vol[ROVOL].ID)
+ serverspp = &volp->vol[ROVOL].serversp;
+ else if (volume == volp->vol[BACKVOL].ID)
+ serverspp = &volp->vol[BACKVOL].serversp;
else
osi_panic("bad volume ID in cm_GetVolServers", __FILE__, __LINE__);
void cm_PutVolume(cm_volume_t *volp)
{
- lock_ObtainWrite(&cm_volumeLock);
- osi_assertx(volp->refCount-- > 0, "cm_volume_t refCount 0");
- lock_ReleaseWrite(&cm_volumeLock);
+ afs_int32 refCount = InterlockedDecrement(&volp->refCount);
+ osi_assertx(refCount >= 0, "cm_volume_t refCount underflow has occurred");
}
/* return the read-only volume, if there is one, or the read-write volume if
{
long id;
- lock_ObtainMutex(&volp->mx);
- if (volp->ro.ID && volp->ro.serversp)
- id = volp->ro.ID;
+ lock_ObtainRead(&volp->rw);
+ if (volp->vol[ROVOL].ID && volp->vol[ROVOL].serversp)
+ id = volp->vol[ROVOL].ID;
else
- id = volp->rw.ID;
- lock_ReleaseMutex(&volp->mx);
+ id = volp->vol[RWVOL].ID;
+ lock_ReleaseRead(&volp->rw);
return id;
}
{
cm_volume_t *volp;
cm_scache_t *scp;
+ afs_int32 refCount;
cm_data.mountRootGen = time(NULL);
/* force a re-loading of volume data from the vldb */
- lock_ObtainWrite(&cm_volumeLock);
+ lock_ObtainRead(&cm_volumeLock);
for (volp = cm_data.allVolumesp; volp; volp=volp->allNextp) {
- volp->refCount++;
- lock_ReleaseWrite(&cm_volumeLock);
+ InterlockedIncrement(&volp->refCount);
+ lock_ReleaseRead(&cm_volumeLock);
- lock_ObtainMutex(&volp->mx);
+ lock_ObtainWrite(&volp->rw);
volp->flags |= CM_VOLUMEFLAG_RESET;
- lock_ReleaseMutex(&volp->mx);
+ lock_ReleaseWrite(&volp->rw);
- lock_ObtainWrite(&cm_volumeLock);
- osi_assertx(volp->refCount-- > 0, "cm_volume_t refCount 0");
+ lock_ObtainRead(&cm_volumeLock);
+ refCount = InterlockedDecrement(&volp->refCount);
+ osi_assertx(refCount >= 0, "cm_volume_t refCount underflow");
}
- lock_ReleaseWrite(&cm_volumeLock);
+ lock_ReleaseRead(&cm_volumeLock);
/* force mount points to be re-evaluated so that
* if the volume location has changed we will pick
&& !(scp->fid.cell == AFS_FAKE_ROOT_CELL_ID && scp->fid.volume == AFS_FAKE_ROOT_VOL_ID)
#endif
) {
- lock_ObtainMutex(&scp->mx);
+ lock_ObtainWrite(&scp->rw);
scp->mountPointStringp[0] = '\0';
- lock_ReleaseMutex(&scp->mx);
+ lock_ReleaseWrite(&scp->rw);
}
}
char *OfflineMsg;
char *MOTD;
cm_req_t req;
- struct rx_connection * callp;
+ struct rx_connection * rxconnp;
char volName[32];
char offLineMsg[256];
char motd[256];
OfflineMsg = offLineMsg;
MOTD = motd;
- lock_ObtainMutex(&volp->mx);
+ lock_ObtainWrite(&volp->rw);
- if (volp->rw.ID != 0 && (!volID || volID == volp->rw.ID) &&
- (volp->rw.state == vl_busy || volp->rw.state == vl_offline)) {
+ if (volp->flags & CM_VOLUMEFLAG_RESET) {
cm_InitReq(&req);
+ code = cm_UpdateVolumeLocation(volp->cellp, cm_rootUserp, &req, volp);
+ }
- for (serversp = volp->rw.serversp; serversp; serversp = serversp->next) {
- if (serversp->status == srv_busy || serversp->status == srv_offline)
+ if (volp->vol[RWVOL].ID != 0 && (!volID || volID == volp->vol[RWVOL].ID) &&
+ volp->vol[RWVOL].serversp) {
+
+ for (serversp = volp->vol[RWVOL].serversp; serversp; serversp = serversp->next) {
+ if (serversp->status == srv_busy || serversp->status == srv_offline) {
serversp->status = srv_not_busy;
+ online = 1;
+ }
}
- do {
- code = cm_ConnFromVolume(volp, volp->rw.ID, cm_rootUserp, &req, &connp);
- if (code)
- continue;
+ if (volp->vol[RWVOL].state == vl_busy || volp->vol[RWVOL].state == vl_offline || volp->vol[RWVOL].state == vl_unknown) {
+ cm_InitReq(&req);
- callp = cm_GetRxConn(connp);
- code = RXAFS_GetVolumeStatus(callp, volp->rw.ID,
- &volStat, &Name, &OfflineMsg, &MOTD);
- rx_PutConnection(callp);
-
- } while (cm_Analyze(connp, cm_rootUserp, &req, NULL, NULL, NULL, NULL, code));
- code = cm_MapRPCError(code, &req);
-
- if (code == 0 && volStat.Online) {
- cm_VolumeStatusNotification(volp, volp->rw.ID, volp->rw.state, vl_online);
- volp->rw.state = vl_online;
- online = 1;
- } else if (code == CM_ERROR_NOACCESS) {
- cm_VolumeStatusNotification(volp, volp->rw.ID, volp->rw.state, vl_unknown);
- volp->rw.state = vl_unknown;
- online = 1;
+ lock_ReleaseWrite(&volp->rw);
+ do {
+ code = cm_ConnFromVolume(volp, volp->vol[RWVOL].ID, cm_rootUserp, &req, &connp);
+ if (code)
+ continue;
+
+ rxconnp = cm_GetRxConn(connp);
+ code = RXAFS_GetVolumeStatus(rxconnp, volp->vol[RWVOL].ID,
+ &volStat, &Name, &OfflineMsg, &MOTD);
+ rx_PutConnection(rxconnp);
+
+ } while (cm_Analyze(connp, cm_rootUserp, &req, NULL, NULL, NULL, NULL, code));
+ code = cm_MapRPCError(code, &req);
+
+ lock_ObtainWrite(&volp->rw);
+ if (code == 0 && volStat.Online) {
+ cm_VolumeStatusNotification(volp, volp->vol[RWVOL].ID, volp->vol[RWVOL].state, vl_online);
+ volp->vol[RWVOL].state = vl_online;
+ online = 1;
+ } else if (code == CM_ERROR_NOACCESS) {
+ cm_VolumeStatusNotification(volp, volp->vol[RWVOL].ID, volp->vol[RWVOL].state, vl_unknown);
+ volp->vol[RWVOL].state = vl_unknown;
+ online = 1;
+ }
}
}
- if (volp->ro.ID != 0 && (!volID || volID == volp->ro.ID) &&
- (volp->ro.state == vl_busy || volp->ro.state == vl_offline)) {
- cm_InitReq(&req);
+ if (volp->vol[ROVOL].ID != 0 && (!volID || volID == volp->vol[ROVOL].ID) &&
+ volp->vol[ROVOL].serversp) {
- for (serversp = volp->ro.serversp; serversp; serversp = serversp->next) {
- if (serversp->status == srv_busy || serversp->status == srv_offline)
+ for (serversp = volp->vol[ROVOL].serversp; serversp; serversp = serversp->next) {
+ if (serversp->status == srv_busy || serversp->status == srv_offline) {
serversp->status = srv_not_busy;
+ online = 1;
+ }
}
- do {
- code = cm_ConnFromVolume(volp, volp->ro.ID, cm_rootUserp, &req, &connp);
- if (code)
- continue;
+ if (volp->vol[ROVOL].state == vl_busy || volp->vol[ROVOL].state == vl_offline || volp->vol[ROVOL].state == vl_unknown) {
+ cm_InitReq(&req);
- callp = cm_GetRxConn(connp);
- code = RXAFS_GetVolumeStatus(callp, volp->ro.ID,
- &volStat, &Name, &OfflineMsg, &MOTD);
- rx_PutConnection(callp);
-
- } while (cm_Analyze(connp, cm_rootUserp, &req, NULL, NULL, NULL, NULL, code));
- code = cm_MapRPCError(code, &req);
-
- if (code == 0 && volStat.Online) {
- cm_VolumeStatusNotification(volp, volp->ro.ID, volp->ro.state, vl_online);
- volp->ro.state = vl_online;
- online = 1;
- } else if (code == CM_ERROR_NOACCESS) {
- cm_VolumeStatusNotification(volp, volp->ro.ID, volp->ro.state, vl_unknown);
- volp->ro.state = vl_unknown;
- online = 1;
+ lock_ReleaseWrite(&volp->rw);
+ do {
+ code = cm_ConnFromVolume(volp, volp->vol[ROVOL].ID, cm_rootUserp, &req, &connp);
+ if (code)
+ continue;
+
+ rxconnp = cm_GetRxConn(connp);
+ code = RXAFS_GetVolumeStatus(rxconnp, volp->vol[ROVOL].ID,
+ &volStat, &Name, &OfflineMsg, &MOTD);
+ rx_PutConnection(rxconnp);
+
+ } while (cm_Analyze(connp, cm_rootUserp, &req, NULL, NULL, NULL, NULL, code));
+ code = cm_MapRPCError(code, &req);
+
+ lock_ObtainWrite(&volp->rw);
+ if (code == 0 && volStat.Online) {
+ cm_VolumeStatusNotification(volp, volp->vol[ROVOL].ID, volp->vol[ROVOL].state, vl_online);
+ volp->vol[ROVOL].state = vl_online;
+ online = 1;
+ } else if (code == CM_ERROR_NOACCESS) {
+ cm_VolumeStatusNotification(volp, volp->vol[ROVOL].ID, volp->vol[ROVOL].state, vl_unknown);
+ volp->vol[ROVOL].state = vl_unknown;
+ online = 1;
+ }
}
}
- if (volp->bk.ID != 0 && (!volID || volID == volp->bk.ID) &&
- (volp->bk.state == vl_busy || volp->bk.state == vl_offline)) {
- cm_InitReq(&req);
-
- for (serversp = volp->bk.serversp; serversp; serversp = serversp->next) {
- if (serversp->status == srv_busy || serversp->status == srv_offline)
+ if (volp->vol[BACKVOL].ID != 0 && (!volID || volID == volp->vol[BACKVOL].ID) &&
+ volp->vol[BACKVOL].serversp) {
+
+ for (serversp = volp->vol[BACKVOL].serversp; serversp; serversp = serversp->next) {
+ if (serversp->status == srv_busy || serversp->status == srv_offline) {
serversp->status = srv_not_busy;
+ online = 1;
+ }
}
- do {
- code = cm_ConnFromVolume(volp, volp->bk.ID, cm_rootUserp, &req, &connp);
- if (code)
- continue;
+ if (volp->vol[BACKVOL].state == vl_busy || volp->vol[BACKVOL].state == vl_offline || volp->vol[BACKVOL].state == vl_unknown) {
+ cm_InitReq(&req);
- callp = cm_GetRxConn(connp);
- code = RXAFS_GetVolumeStatus(callp, volp->bk.ID,
- &volStat, &Name, &OfflineMsg, &MOTD);
- rx_PutConnection(callp);
-
- } while (cm_Analyze(connp, cm_rootUserp, &req, NULL, NULL, NULL, NULL, code));
- code = cm_MapRPCError(code, &req);
-
- if (code == 0 && volStat.Online) {
- cm_VolumeStatusNotification(volp, volp->bk.ID, volp->bk.state, vl_online);
- volp->bk.state = vl_online;
- online = 1;
- } else if (code == CM_ERROR_NOACCESS) {
- cm_VolumeStatusNotification(volp, volp->bk.ID, volp->bk.state, vl_unknown);
- volp->bk.state = vl_unknown;
- online = 1;
+ lock_ReleaseWrite(&volp->rw);
+ do {
+ code = cm_ConnFromVolume(volp, volp->vol[BACKVOL].ID, cm_rootUserp, &req, &connp);
+ if (code)
+ continue;
+
+ rxconnp = cm_GetRxConn(connp);
+ code = RXAFS_GetVolumeStatus(rxconnp, volp->vol[BACKVOL].ID,
+ &volStat, &Name, &OfflineMsg, &MOTD);
+ rx_PutConnection(rxconnp);
+
+ } while (cm_Analyze(connp, cm_rootUserp, &req, NULL, NULL, NULL, NULL, code));
+ code = cm_MapRPCError(code, &req);
+
+ lock_ObtainWrite(&volp->rw);
+ if (code == 0 && volStat.Online) {
+ cm_VolumeStatusNotification(volp, volp->vol[BACKVOL].ID, volp->vol[BACKVOL].state, vl_online);
+ volp->vol[BACKVOL].state = vl_online;
+ online = 1;
+ } else if (code == CM_ERROR_NOACCESS) {
+ cm_VolumeStatusNotification(volp, volp->vol[BACKVOL].ID, volp->vol[BACKVOL].state, vl_unknown);
+ volp->vol[BACKVOL].state = vl_unknown;
+ online = 1;
+ }
}
}
- lock_ReleaseMutex(&volp->mx);
+ lock_ReleaseWrite(&volp->rw);
return online;
}
void cm_CheckOfflineVolumes(void)
{
cm_volume_t *volp;
+ afs_int32 refCount;
- lock_ObtainWrite(&cm_volumeLock);
+ lock_ObtainRead(&cm_volumeLock);
for (volp = cm_data.allVolumesp; volp; volp=volp->allNextp) {
- volp->refCount++;
- lock_ReleaseWrite(&cm_volumeLock);
-
- cm_CheckOfflineVolume(volp, 0);
-
- lock_ObtainWrite(&cm_volumeLock);
- osi_assertx(volp->refCount-- > 0, "cm_volume_t refCount 0");
+ if (volp->flags & CM_VOLUMEFLAG_IN_HASH) {
+ InterlockedIncrement(&volp->refCount);
+ lock_ReleaseRead(&cm_volumeLock);
+ cm_CheckOfflineVolume(volp, 0);
+ lock_ObtainRead(&cm_volumeLock);
+ refCount = InterlockedDecrement(&volp->refCount);
+ osi_assertx(refCount >= 0, "cm_volume_t refCount underflow");
+ }
}
- lock_ReleaseWrite(&cm_volumeLock);
+ lock_ReleaseRead(&cm_volumeLock);
}
cm_UpdateVolumeStatus(cm_volume_t *volp, afs_uint32 volID)
{
- if (volp->rw.ID == volID) {
- cm_UpdateVolumeStatusInt(volp, &volp->rw);
- } else if (volp->ro.ID == volID) {
- cm_UpdateVolumeStatusInt(volp, &volp->ro);
- } else if (volp->bk.ID == volID) {
- cm_UpdateVolumeStatusInt(volp, &volp->bk);
+ if (volp->vol[RWVOL].ID == volID) {
+ cm_UpdateVolumeStatusInt(volp, &volp->vol[RWVOL]);
+ } else if (volp->vol[ROVOL].ID == volID) {
+ cm_UpdateVolumeStatusInt(volp, &volp->vol[ROVOL]);
+ } else if (volp->vol[BACKVOL].ID == volID) {
+ cm_UpdateVolumeStatusInt(volp, &volp->vol[BACKVOL]);
} else {
/*
* If we are called with volID == 0 then something has gone wrong.
* just update all of them that are known to exist. Better to be
* correct than fast.
*/
- if (volp->rw.ID != 0)
- cm_UpdateVolumeStatusInt(volp, &volp->rw);
- if (volp->ro.ID != 0)
- cm_UpdateVolumeStatusInt(volp, &volp->ro);
- if (volp->bk.ID != 0)
- cm_UpdateVolumeStatusInt(volp, &volp->bk);
+ afs_uint32 volType;
+ for ( volType = RWVOL; volType < NUM_VOL_TYPES; volType++) {
+ if (volp->vol[volType].ID != 0)
+ cm_UpdateVolumeStatusInt(volp, &volp->vol[volType]);
+ }
}
}
{
int code;
cm_volume_t* volp;
+ afs_int32 refCount;
/* find volumes which might have RO copy on server*/
- lock_ObtainWrite(&cm_volumeLock);
+ lock_ObtainRead(&cm_volumeLock);
for(volp = cm_data.allVolumesp; volp; volp=volp->allNextp)
{
code = 1 ; /* assume that list is unchanged */
- volp->refCount++;
- lock_ReleaseWrite(&cm_volumeLock);
- lock_ObtainMutex(&volp->mx);
+ InterlockedIncrement(&volp->refCount);
+ lock_ReleaseRead(&cm_volumeLock);
+ lock_ObtainWrite(&volp->rw);
- if ((tsp->cellp==volp->cellp) && (volp->ro.serversp))
- code =cm_ChangeRankServer(&volp->ro.serversp, tsp);
+ if ((tsp->cellp==volp->cellp) && (volp->vol[ROVOL].serversp))
+ code =cm_ChangeRankServer(&volp->vol[ROVOL].serversp, tsp);
/* this volume list was changed */
if ( !code )
- cm_RandomizeServer(&volp->ro.serversp);
+ cm_RandomizeServer(&volp->vol[ROVOL].serversp);
- lock_ReleaseMutex(&volp->mx);
- lock_ObtainWrite(&cm_volumeLock);
- osi_assertx(volp->refCount-- > 0, "cm_volume_t refCount 0");
+ lock_ReleaseWrite(&volp->rw);
+ lock_ObtainRead(&cm_volumeLock);
+ refCount = InterlockedDecrement(&volp->refCount);
+ osi_assertx(refCount >= 0, "cm_volume_t refCount underflow");
}
- lock_ReleaseWrite(&cm_volumeLock);
+ lock_ReleaseRead(&cm_volumeLock);
}
/* dump all volumes that have reference count > 0 to a file.
for (volp = cm_data.allVolumesp; volp; volp=volp->allNextp)
{
- cm_scache_t *scp;
- int scprefs = 0;
-
- for (scp = cm_data.allSCachesp; scp; scp = scp->allNextp)
- {
- if (scp->volp == volp)
- scprefs++;
- }
-
- sprintf(output, "%s - volp=0x%p cell=%s name=%s rwID=%u roID=%u bkID=%u flags=0x%x fid (cell=%d, volume=%d, vnode=%d, unique=%d) refCount=%u scpRefs=%u\r\n",
- cookie, volp, volp->cellp->name, volp->namep, volp->rw.ID, volp->ro.ID, volp->bk.ID, volp->flags,
- volp->dotdotFid.cell, volp->dotdotFid.volume, volp->dotdotFid.vnode, volp->dotdotFid.unique,
- volp->refCount, scprefs);
+ sprintf(output, "%s - volp=0x%p cell=%s name=%s rwID=%u roID=%u bkID=%u flags=0x%x refCount=%u\r\n",
+ cookie, volp, volp->cellp->name, volp->namep, volp->vol[RWVOL].ID, volp->vol[ROVOL].ID, volp->vol[BACKVOL].ID, volp->flags,
+ volp->refCount);
WriteFile(outputFile, output, (DWORD)strlen(output), &zilch, NULL);
}
sprintf(output, "%s - Done dumping volumes.\r\n", cookie);
int i;
struct cm_vol_state * statep;
- switch (volType) {
- case RWVOL:
- statep = &volp->rw;
- break;
- case ROVOL:
- statep = &volp->ro;
- break;
- case BACKVOL:
- statep = &volp->bk;
- break;
- default:
- return;
- }
+ statep = cm_VolumeStateByType(volp, volType);
if (statep->flags & CM_VOLUMEFLAG_IN_HASH)
return;
struct cm_vol_state * statep;
int i;
- switch (volType) {
- case RWVOL:
- statep = &volp->rw;
- break;
- case ROVOL:
- statep = &volp->ro;
- break;
- case BACKVOL:
- statep = &volp->bk;
- break;
- default:
- return;
- }
+ statep = cm_VolumeStateByType(volp, volType);
if (statep->flags & CM_VOLUMEFLAG_IN_HASH) {
/* hash it out first */
lvolpp = &cm_data.volumeBKIDHashTablep[i];
tvolp = cm_data.volumeBKIDHashTablep[i];
break;
+ default:
+ osi_assertx(0, "invalid volume type");
}
do {
if (tvolp == volp) {
break;
}
- switch (volType) {
- case RWVOL:
- lvolpp = &tvolp->rw.nextp;
- tvolp = tvolp->rw.nextp;
- break;
- case ROVOL:
- lvolpp = &tvolp->ro.nextp;
- tvolp = tvolp->ro.nextp;
- break;
- case BACKVOL:
- lvolpp = &tvolp->bk.nextp;
- tvolp = tvolp->bk.nextp;
- break;
- }
+ lvolpp = &tvolp->vol[volType].nextp;
+ tvolp = tvolp->vol[volType].nextp;
} while(tvolp);
}
}
}
/* must be called with cm_volumeLock write-locked! */
+void cm_MoveVolumeToLRULast(cm_volume_t *volp)
+{
+ if (volp == cm_data.volumeLRULastp)
+ return;
+
+ if (volp == cm_data.volumeLRUFirstp)
+ cm_data.volumeLRUFirstp = (cm_volume_t *) osi_QNext(&volp->q);
+ if (volp->flags & CM_VOLUMEFLAG_IN_LRU_QUEUE)
+ osi_QRemoveHT((osi_queue_t **) &cm_data.volumeLRUFirstp, (osi_queue_t **) &cm_data.volumeLRULastp, &volp->q);
+ osi_QAddT((osi_queue_t **) &cm_data.volumeLRUFirstp, (osi_queue_t **) &cm_data.volumeLRULastp, &volp->q);
+ volp->flags |= CM_VOLUMEFLAG_IN_LRU_QUEUE;
+ if (!cm_data.volumeLRULastp)
+ cm_data.volumeLRULastp = volp;
+}
+
+/* must be called with cm_volumeLock write-locked! */
void cm_RemoveVolumeFromLRU(cm_volume_t *volp)
{
if (volp->flags & CM_VOLUMEFLAG_IN_LRU_QUEUE) {
char volstr[CELL_MAXNAMELEN + VL_MAXNAMELEN]="";
char *ext = "";
- if (volID == volp->rw.ID)
+ if (volID == volp->vol[RWVOL].ID)
ext = "";
- else if (volID == volp->ro.ID)
+ else if (volID == volp->vol[ROVOL].ID)
ext = ".readonly";
- else if (volID == volp->bk.ID)
+ else if (volID == volp->vol[BACKVOL].ID)
ext = ".backup";
else
ext = ".nomatch";
enum volstatus cm_GetVolumeStatus(cm_volume_t *volp, afs_uint32 volID)
{
- if (volp->rw.ID == volID) {
- return volp->rw.state;
- } else if (volp->ro.ID == volID) {
- return volp->ro.state;
- } else if (volp->bk.ID == volID) {
- return volp->bk.state;
- } else {
+ cm_vol_state_t * statep = cm_VolumeStateByID(volp, volID);
+ if (statep)
+ return statep->state;
+ else
return vl_unknown;
- }
}
/* Renew .readonly volume callbacks that are more than
cm_fid_t fid;
cm_scache_t * scp;
- fid.cell = volp->cellp->cellID;
- fid.volume = volp->ro.ID;
- fid.vnode = 1;
- fid.unique = 1;
+ cm_SetFid(&fid, volp->cellp->cellID, volp->vol[ROVOL].ID, 1, 1);
cm_InitReq(&req);
lock_ReleaseRead(&cm_volumeLock);
if (cm_GetSCache(&fid, &scp, cm_rootUserp, &req) == 0) {
- lock_ObtainMutex(&scp->mx);
+ lock_ObtainWrite(&scp->rw);
cm_GetCallback(scp, cm_rootUserp, &req, 1);
- lock_ReleaseMutex(&scp->mx);
+ lock_ReleaseWrite(&scp->rw);
cm_ReleaseSCache(scp);
}
lock_ObtainRead(&cm_volumeLock);
}
lock_ReleaseRead(&cm_volumeLock);
}
+
+cm_vol_state_t *
+cm_VolumeStateByType(cm_volume_t *volp, afs_uint32 volType)
+{
+ return &volp->vol[volType];
+}
+
+cm_vol_state_t *
+cm_VolumeStateByID(cm_volume_t *volp, afs_uint32 id)
+{
+ cm_vol_state_t * statep = NULL;
+
+ if (id == volp->vol[RWVOL].ID)
+ statep = &volp->vol[RWVOL];
+ else if (id == volp->vol[ROVOL].ID)
+ statep = &volp->vol[ROVOL];
+ else if (id == volp->vol[BACKVOL].ID)
+ statep = &volp->vol[BACKVOL];
+
+ return(statep);
+}
+
+cm_vol_state_t *
+cm_VolumeStateByName(cm_volume_t *volp, char *volname)
+{
+ size_t len = strlen(volname);
+ cm_vol_state_t *statep;
+
+ if (cm_stricmp_utf8N(".readonly", &volname[len-9]) == 0)
+ statep = &volp->vol[ROVOL];
+ else if (cm_stricmp_utf8N(".backup", &volname[len-7]) == 0)
+ statep = &volp->vol[BACKVOL];
+ else
+ statep = &volp->vol[RWVOL];
+
+ return statep;
+}
+