Jes Sorensen <Jes.Sorensen@xxxxxxxxxx> writes:
> Gioh Kim <gi-oh.kim@xxxxxxxxxxxxxxxx> writes:
> This is definitly not the right way to solve this problem. Error codes
> are negative, and zero should _always_ mean success. Here you suddenly
> introduced a new meaning to positive values of rv.
>
> I agree handling the error case needs to be fixed, so a better way to
> solve this would be to bail out when the load_super() call fails and
> stop there, the same way it does if add_internal_bitmap() fails.
>
> Ie. make it do something like this:
>
> rv = st->ss->load_super(st, fd2, NULL)==0) {
> if (!rv) {
> if (st->ss->add_internal_bitmap(
> ....
> } else {
> pr_err("failed to load super-block.\n");
> close(fd2);
> return 1;
> }
>
> Actually looking at that code, there's a couple of things to do to clean
> it up and make it more readable.
So I started looking at this, and ended up making a bunch of changes
which should both resolve the issue you encountered and also cleans up
this part of the code. I had to change add_internal_bitmap() to return 0
on success, in order to get it cleaned up. Looks like we have a pile of
inconsistencies on the 0 vs 1 as success returns ... guess we won't get
bored.
I just pushed this into git - let me know if it doesn't work for you.
Cheers,
Jes
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
