Re: [Patch] Robustly search for ZFS labels & uberblocks

["Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>]

On 31.01.2012 09:45, Richard Laager wrote:
> Attached is a stack of trivial patches that apply*on top of*  your
> zfs.diff from 2012-01-29. They each deal with one logical change and
> should be very easy to review.
>
> After your original patch and this stack have been dealt with, I'll
> submit an updated patch for native ZFS on Linux support, which is much
> shorter than before.
>
>                                                                       --
>                                                                   Richard
>
> zfs-on-linux-rlaager0.patch
>
>
> Eliminate stray trailing spaces.
We don't do it manually, just from time to time run it automatically on 
whole codebase.

>
> zfs-on-linux-rlaager1.patch
>
>
> Change device to devices in find_root_devices_from_libzfs
>
>
>
committed
> zfs-on-linux-rlaager2.patch
>
>
> Change strlen to sizeof
It doesn't matter since GCC changes strlen of const string to just a 
const. But applied for conformity with the rest of codebase
>
> zfs-on-linux-rlaager3.patch
>
>
> Avoid crashing when canonicalize_file_name() fails
>
> I could reproduce this with: grub-probe /dev/sda1
>
>
>
Committed
> zfs-on-linux-rlaager4.patch
>
>
> Drop an unused variable.
>
>
>
committed.
> zfs-on-linux-rlaager5.patch
>
>
> Add braces around and indent the `zpool status` parsing loop
Not needed. And we don't put a braces which are not needed.

>
> zfs-on-linux-rlaager6.patch
>
>
> Handle pool names with trailing spaces
What about the ones with spaces in the middle? It feels like the logic 
is broken elsewhere and using strncmp is just a workaround which works 
only for one particular case
> zfs-on-linux-rlaager7.patch
>
>
> Handle all raidz types in `zpool status` output
>
>
Committed.

-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko