Not sure if this has been reported or not.

This is fairly minor but slightly annoying.

After fresh install of snv_64a I run zpool import to find this :

# zpool import
  pool: zfs0
    id: 13628474126490956011
 state: ONLINE
status: The pool is formatted using an older on-disk version.
action: The pool can be imported using its name or numeric identifier, though
        some features will not be available without an explicit 'zpool
upgrade'.
config:

        zfs0         ONLINE
          mirror     ONLINE
            c1t9d0   ONLINE
            c0t9d0   ONLINE
          mirror     ONLINE
            c1t10d0  ONLINE
            c0t10d0  ONLINE
          mirror     ONLINE
            c1t11d0  ONLINE
            c0t11d0  ONLINE
          mirror     ONLINE
            c1t12d0  ONLINE
            c0t12d0  ONLINE
          mirror     ONLINE
            c1t13d0  ONLINE
            c0t13d0  ONLINE
          mirror     ONLINE
            c1t14d0  ONLINE
            c0t14d0  ONLINE

So I then run a zpool import but I add in the -R option and specify root thus :

# zpool import -f -R / 13628474126490956011

One would think that the "-R /" would not result in any damage but this si
the result :

# zfs list
NAME               USED  AVAIL  REFER  MOUNTPOINT
zfs0               191G  8.23G  24.5K  legacy
zfs0/SUNWspro      567M   201M   567M  //opt/SUNWspro
zfs0/backup        190G  8.23G   189G  //export/zfs/backup
zfs0/backup/qemu  1.09G   934M  1.09G  //export/zfs/qemu
zfs0/csw           124M  3.88G   124M  //opt/csw
zfs0/home          239M  7.77G   239M  //export/home
zfs0/titan        24.5K  8.23G  24.5K  //export/zfs/titan

Note the extra / there that should not be there.

Not a simple thing to fix either :

# zfs set mountpoint=/opt/SUNWspro zfs0/SUNWspro
# zfs list
NAME               USED  AVAIL  REFER  MOUNTPOINT
zfs0               191G  8.23G  24.5K  legacy
zfs0/SUNWspro      567M   201M   567M  //opt/SUNWspro
zfs0/backup        190G  8.23G   189G  //export/zfs/backup
zfs0/backup/qemu  1.09G   934M  1.09G  //export/zfs/qemu
zfs0/csw           124M  3.88G   124M  //opt/csw
zfs0/home          239M  7.77G   239M  //export/home
zfs0/titan        24.5K  8.23G  24.5K  //export/zfs/titan

relatively harmless.

Looks like altroot should be assumed to be / unless otherwise specified and
if it is specified to be / then the altroot can be ignored.  I don't know if
that is clear but I think you know what I mean :

in /usr/src/cmd/zpool/zpool_main.c :

static int do_import(nvlist_t *config, const char *newname, const char
*mntopts, const char *altroot, int force, int argc, char **argv)


if that const char *altroot  happens to be nothing more than a forward slash
char ( nul terminated ) then I think it should be ignored.

What say you ?

Dennis

_______________________________________________
zfs-discuss mailing list
zfs-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/zfs-discuss

Reply via email to