jimzah/openafs
1
0
Fork 0
mirror of https://git.openafs.org/openafs.git synced 2025-02-01 14:07:39 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

volser: Ignore duplicate file tags when restoring

Browse Source 
Currently, if the volserver encounters multiple file tags for a vnode
when restoring a volume dump, it will create extra vnodes which are
not recorded anywhere, and are leaked when the volume is removed.
Instead of doing that, ignore any duplicate file tags that are
encountered (which is what happens with other tags), and log a
warning.

Change-Id: I1ac76b0217096d013db355b899f1a53174d20eb5
Reviewed-on: http://gerrit.openafs.org/2531
Tested-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
This commit is contained in:
Andrew Deason 2010-08-06 14:32:16 -05:00 committed by Derrick Brashear
parent f7d298d367
commit 72da1d2eb2
1 changed files with 22 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/volser/dumpstuff.c
View File

@ -1330,6 +1330,7 @@ ReadVnodes(register struct iod *iodp, Volume * vp, int incremental,
V_pref(vp, nearInode);
while (tag == D_VNODE) {
int haveStuff = 0;
int saw_f = 0;
memset(buf, 0, sizeof(buf));
if (!ReadInt32(iodp, (afs_uint32 *) & vnodeNumber))
break;
@ -1402,6 +1403,15 @@ ReadVnodes(register struct iod *iodp, Volume * vp, int incremental,
Error error;
afs_fsize_t vnodeLength;
if (saw_f) {
Log("Volser: ReadVnodes: warning: ignoring duplicate "
"file entries for vnode %lu in dump\n",
(unsigned long)vnodeNumber);
volser_WriteFile(vnodeNumber, iodp, NULL, tag, &error);
break;
}
saw_f = 1;
ino =
IH_CREATE(V_linkHandle(vp), V_device(vp),
VPartitionPath(V_partition(vp)), nearInode,
@ -1506,6 +1516,9 @@ ReadVnodes(register struct iod *iodp, Volume * vp, int incremental,
/* called with disk file only. Note that we don't have to worry about rx_Read
* needing to read an ungetc'd character, since the ReadInt32 will have read
* it instead.
*
* if handleP == NULL, don't write the file anywhere; just read and discard
* the file contents
*/
static afs_fsize_t
volser_WriteFile(int vn, struct iod *iodp, FdHandle_t * handleP, int tag,
@ -1556,15 +1569,17 @@ volser_WriteFile(int vn, struct iod *iodp, FdHandle_t * handleP, int tag,
*status = 3;
break;
}
if (handleP) {
nBytes = FDH_WRITE(handleP, p, size);
if (nBytes > 0)
written += nBytes;
if (nBytes != size) {
Log("1 Volser: WriteFile: Error writing (%u) bytes to vnode %d: %s; restore aborted\n", (int)(nBytes & 0xffffffff), vn, afs_error_message(errno));
Log("1 Volser: WriteFile: Error writing (%u) bytes to vnode %d; %s; restore aborted\n", (int)(nBytes & 0xffffffff), vn, afs_error_message(errno));
*status = 4;
break;
}
}
}
free(p);
return (written);
}