Message ID | CAO3qy=EeD4y6cX3JaA4ZKptoL9JH0Kc9zYqPWj-UZ+ythC6LXw@mail.gmail.com |
---|---|
State | Accepted, archived |
Headers | show |
On Tue, Jan 17, 2017 at 06:14:19PM +0300, Artem Blagodarenko wrote: > Hello, > > Darrick J. Wong, thank you for inspection. Here is second version of the patch. > > Changes since v1: > * errno is returned in case failed ioctl > * loop is terminated any time fm_mapped_extents == 0 > > During filesystem creation with -d option files copied from given > directory. If supported, FS_IOC_FIEMAP ioctl is used to get file > extents. For empty file fm_mapped_extents is 0 and no error code > is returned, because this is not fail situation. > > Without this patch errno from previous operation is returned and > successful FS_IOC_FIEMAP ioctl call for empty file is interpreted > as error. > > Signed-off-by: Artem Blagodarenko <artem.blagodarenko@seagate.com> Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> --D > --- > misc/create_inode.c | 6 ++++-- > 1 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/misc/create_inode.c b/misc/create_inode.c > index 5122e56..d9e9b9b 100644 > --- a/misc/create_inode.c > +++ b/misc/create_inode.c > @@ -510,10 +510,12 @@ static errcode_t try_fiemap_copy(ext2_filsys fs, > int fd, ext2_file_t e2_file, > if (err < 0 && (errno == EOPNOTSUPP || errno == ENOTTY)) { > err = EXT2_ET_UNIMPLEMENTED; > goto out; > - } else if (err < 0 || fiemap_buf->fm_mapped_extents == 0) { > + } else if (err < 0) { > err = errno; > goto out; > - } > + } else if (fiemap_buf->fm_mapped_extents == 0) > + goto out; > + > for (i = 0, ext = ext_buf; i < fiemap_buf->fm_mapped_extents; > i++, ext++) { > err = copy_file_range(fs, fd, e2_file, ext->fe_logical, > -- > -- > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Jan 17, 2017 at 06:14:19PM +0300, Artem Blagodarenko wrote: > Hello, > > Darrick J. Wong, thank you for inspection. Here is second version of the patch. > > Changes since v1: > * errno is returned in case failed ioctl > * loop is terminated any time fm_mapped_extents == 0 > > During filesystem creation with -d option files copied from given > directory. If supported, FS_IOC_FIEMAP ioctl is used to get file > extents. For empty file fm_mapped_extents is 0 and no error code > is returned, because this is not fail situation. > > Without this patch errno from previous operation is returned and > successful FS_IOC_FIEMAP ioctl call for empty file is interpreted > as error. > > Signed-off-by: Artem Blagodarenko <artem.blagodarenko@seagate.com> > Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> Thanks, applied. - Ted
diff --git a/misc/create_inode.c b/misc/create_inode.c index 5122e56..d9e9b9b 100644 --- a/misc/create_inode.c +++ b/misc/create_inode.c @@ -510,10 +510,12 @@ static errcode_t try_fiemap_copy(ext2_filsys fs, int fd, ext2_file_t e2_file, if (err < 0 && (errno == EOPNOTSUPP || errno == ENOTTY)) { err = EXT2_ET_UNIMPLEMENTED; goto out; - } else if (err < 0 || fiemap_buf->fm_mapped_extents == 0) { + } else if (err < 0) { err = errno; goto out; - } + } else if (fiemap_buf->fm_mapped_extents == 0) + goto out; + for (i = 0, ext = ext_buf; i < fiemap_buf->fm_mapped_extents; i++, ext++) {
Hello, Darrick J. Wong, thank you for inspection. Here is second version of the patch. Changes since v1: * errno is returned in case failed ioctl * loop is terminated any time fm_mapped_extents == 0 During filesystem creation with -d option files copied from given directory. If supported, FS_IOC_FIEMAP ioctl is used to get file extents. For empty file fm_mapped_extents is 0 and no error code is returned, because this is not fail situation. Without this patch errno from previous operation is returned and successful FS_IOC_FIEMAP ioctl call for empty file is interpreted as error. Signed-off-by: Artem Blagodarenko <artem.blagodarenko@seagate.com> --- misc/create_inode.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) err = copy_file_range(fs, fd, e2_file, ext->fe_logical, -- -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html