Re: Next ZFSv28 patchset ready for testing.

From: Olivier Smedts <olivier_at_gid0.org>
Date: Wed, 23 Feb 2011 23:28:32 +0100
2011/2/23 Jason Garrett <kingedgar_at_gmail.com>:
> On Mon, Feb 7, 2011 at 08:01, Anonymous <swell.k_at_gmail.com> wrote:
>
>> 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.
>>
>
> Has there been any recent developments with these patches (eg. Will they
> apply and build on latest checkouts?) ?

I'm successfuly using the following on latest 9-CURRENT :
http://people.freebsd.org/~mm/patches/zfs/v28/head-zfsv28-20110219-nopython.patch.xz

If you want patches for 8-STABLE, look at the directory contents.

Cheers

-- 
Olivier Smedts                                                 _
                                        ASCII ribbon campaign ( )
e-mail: olivier_at_gid0.org        - against HTML email & vCards  X
www: http://www.gid0.org    - against proprietary attachments / \

  "Il y a seulement 10 sortes de gens dans le monde :
  ceux qui comprennent le binaire,
  et ceux qui ne le comprennent pas."
Received on Wed Feb 23 2011 - 21:28:33 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:11 UTC