Subject: Re: dump bug?
To: Brook Milligan <brook@trillium.NMSU.Edu>
From: Paul Goyette <paul@pgoyette.bdt.com>
List: current-users
Date: 01/14/1997 16:39:48
Check out PR#bin/2977 that I submitted a while ago. Basically, my
suggested patch modifies dump to only recognize ufs/ffs file system
entries in /etc/fstab. IMHO, do recognize any others is ridiculous,
since dump(8) couldn't dump them anyway.
On Tue, 14 Jan 1997, Brook Milligan wrote:
> QUESTION: Is there a reason for dump to construct its data structure
> representing /etc/fstab in reverse order?
>
> Have I missed something obvious here?
>
> I should have realized what I was missing (but a question still
> remains). Of course, the last /etc/fstab entry is the one most
> recently mounted and therefore the one in use and to be backed up.
> Hence the reverse order linked list. OK.
>
> But, shouldn't union mounts be disregarded altogether since they are
> only namespace manipulations and not actual device mounts? I.e.,
> shouldn't dump skip over union mounts in constructing its internal
> version of /etc/fstab so that it only matches "real" filesystem
> entries?
>
> Now what am I missing?
>
> One patch to skip union mounts is the following:
>
> *** ./src/sbin/dump/optr.c.orig Sat May 18 10:16:17 1996
> --- ./src/sbin/dump/optr.c Tue Jan 14 14:29:46 1997
> ***************
> *** 444,449 ****
> --- 444,451 ----
> strcmp(fs->fs_type, FSTAB_RO) &&
> strcmp(fs->fs_type, FSTAB_RQ))
> continue;
> + if (strcmp(fs->fs_vfstype, "union") == 0)
> + continue;
> fs = allocfsent(fs);
> if ((pf = (struct pfstab *)malloc(sizeof (*pf))) == NULL)
> quit("%s\n", strerror(errno));
>
> Thanks again for your help.
>
> Cheers,
> Brook
>