On Fri, Sep 14, 2018 at 03:25:03PM +0800, Qu Wenruo wrote: > This patchset can be fetched from github: > https://github.com/adam900710/btrfs-progs/tree/convert_error_messages > As usual, it's based on latest stable tag (v4.17.1). > > There is one error report of btrfs-convert, the error message looks > pretty meaningless: > > create btrfs filesystem: > blocksize: 4096 > nodesize: 16384 > features: extref, skinny-metadata (default) > creating ext2 image file > ERROR: failed to create ext2_saved/image: -1 > WARNING: an error occurred during conversion, filesystem is partially > created but not finalized and not mountable > > After some investigation, the problem turns out to be read failure. > But the error number is intermediate number (-1) returned from > read_disk_extent(). > > This patchset will first fix the intermediate return number of > read_disk_extent(), then add more error messages for btrfs-convert (at > least convert part) to makes it easier to identify the problem. > > In this particular case, it should output things like: > > create btrfs filesystem: > blocksize: 4096 > nodesize: 16384 > features: extref, skinny-metadata (default) > creating ext2 image file > ERROR: failed to calculate csum for bytenr 2732765184 len 4096, > Input/output error > ERROR: failed to create ext2_saved/image: -5 > WARNING: an error occurred during conversion, filesystem is partially > created but not finalized and not mountable > > > Qu Wenruo (2): > btrfs: convert: Make read_disk_extent() return more meaningful -EIO > other -1 > btrfs-progs: convert: Output meaningful error messages for > create_image()
Added to devel, thanks.