Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 23 Feb 2011 23:28:32 +0100
From:      Olivier Smedts <olivier@gid0.org>
To:        Jason Garrett <kingedgar@gmail.com>
Cc:        freebsd-current <freebsd-current@freebsd.org>
Subject:   Re: Next ZFSv28 patchset ready for testing.
Message-ID:  <AANLkTi=LMVwq7U1n8E3ZKLWpgZdB0LOf=FpCFCB9w_9e@mail.gmail.com>
In-Reply-To: <AANLkTi=WKdK2w7feTVXX28me6mmzFn0J1s8WWkRjA9Mi@mail.gmail.com>
References:  <20101213214556.GC2038@garage.freebsd.pl> <8662upxg76.fsf@gmail.com> <86lj1s3pv0.fsf@gmail.com> <AANLkTi=WKdK2w7feTVXX28me6mmzFn0J1s8WWkRjA9Mi@mail.gmail.com>

index | next in thread | previous in thread | raw e-mail

2011/2/23 Jason Garrett <kingedgar@gmail.com>:
> On Mon, Feb 7, 2011 at 08:01, Anonymous <swell.k@gmail.com> wrote:
>
>> Anonymous <swell.k@gmail.com> writes:
>>
>> > Pawel Jakub Dawidek <pjd@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
>> @@ -1129,7 +1129,11 @@ 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@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."


help

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTi=LMVwq7U1n8E3ZKLWpgZdB0LOf=FpCFCB9w_9e>