Re: Next ZFSv28 patchset ready for testing.

Anonymous <swell.k_at_gmail.com> writes:

> Pawel Jakub Dawidek <pjd_at_FreeBSD.org> writes:
>
>> The new patchset is ready for testing:
>>
>> 	http://people.freebsd.org/~pjd/patches/zfs_20101212.patch.bz2
>>
>
> `-e' option in zdb(8) now looks under /dev/dsk by default
>
>   $ zdb -ec blah
>
>   Configuration for import:
>           vdev_children: 1
>           version: 6
>           vdev_tree:
>               children[0]:
>                   phys_path: '/dev/gptid/A-B-C-D-E'
>                   path: '/dev/dsk/gptid/A-B-C-D-E'
>   zdb: can't open 'blah': No such file or directory
>   Exit 1

How about below diff then?

%%
--- cddl/contrib/opensolaris/lib/libzfs/common/libzfs_import.c~
+++ cddl/contrib/opensolaris/lib/libzfs/common/libzfs_import.c
_at__at_ -1129,7 +1129,11 _at__at_ zpool_find_import_impl(libzfs_handle_t *
 	char *end, **dir = iarg->path;
 	size_t pathleft;
 	nvlist_t *ret = NULL;
+#ifdef sun
 	static char *default_dir = "/dev/dsk";
+#else
+	static char *default_dir = "/dev";
+#endif
 	pool_list_t pools = { 0 };
 	pool_entry_t *pe, *penext;
 	vdev_entry_t *ve, *venext;
%%

>
>   $ zdb -p /dev -ec blah
>
>   Traversing all blocks to verify metadata checksums and verify nothing leaked ...
>   Assertion failed: (mp->initialized == B_TRUE), file /usr/src/cddl/lib/libzpool/../../../cddl/contrib/opensolaris/lib/libzpool/common/kernel.c, line 130.
>   Exit 134

I can't reproduce anymore, at least as of ch188544.