On Wed, Jul 10, 2013 at 5:12 PM, David Sterba <dste...@suse.cz> wrote: > On Tue, Jul 09, 2013 at 07:49:53PM +0100, Filipe David Borba Manana wrote: >> The module cmds-restore.c was defining its own next_leaf() >> function, which did exactly the same as btrfs_next_leaf() >> from ctree.c. > > This has been removed by Eric's patch present in the integration > branches: > Btrfs-progs: remove cut & paste btrfs_next_leaf from restore > http://www.spinics.net/lists/linux-btrfs/msg24477.html
Oh, didn't notice that. > > but now Chris has a fix in the master branch, > btrfs-restore: deal with NULL returns from read_node_slot > https://git.kernel.org/cgit/linux/kernel/git/mason/btrfs-progs.git/commit/?id=194aa4a1bd6447bb545286d0bcb0b0be8204d79f > > the code of updated next_leaf is not identical to btrfs_next_leaf and I > think 'restore' could be more tolerant to partially corrupted > structures, so both functions could make sense in the end. Ok, I understand now why both exist. So please just ignore this patch and the following one (https://patchwork.kernel.org/patch/2825425/). thanks > > david -- Filipe David Manana, "Reasonable men adapt themselves to the world. Unreasonable men adapt the world to themselves. That's why all progress depends on unreasonable men." -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
