Displaying 3 results from an estimated 3 matches for "guestfd".
Did you mean:
guestfs
2015 Oct 19
5
Re: [PATCH 1/2] New API: vfs_min_size
..." SCNd64, &volume_size) != 1) {
> + reply_with_error("Cannot parse volume size");
> + return -1;
> + }
It sounds like these scans of the lines could be done using the
analyze_line helper in btrfs.c (which would need to be moved as common
code in guestfd.c). This way, the parsing of numbers could use
xstrtoul/xstrtoull (see btrfs.c:do_btrfs_subvolume_list) instead of
sscanf.
> + }
> + }
> + if (is_full) {
> + /* Ceil to cluster size */
> + if (cluster_size == 0) {
> + reply_with_error("Bad cluster...
2015 Oct 20
0
Re: [PATCH 1/2] New API: vfs_min_size
..._size) != 1) {
>> + reply_with_error("Cannot parse volume size");
>> + return -1;
>> + }
> It sounds like these scans of the lines could be done using the
> analyze_line helper in btrfs.c (which would need to be moved as common
> code in guestfd.c). This way, the parsing of numbers could use
> xstrtoul/xstrtoull (see btrfs.c:do_btrfs_subvolume_list) instead of
> sscanf.
This is the only place where it could be used for now. Does it worth
moving the code? I guess this code won't be shorter or simpler when
using analyze_line.
>...
2015 Oct 19
5
[PATCHv2 0/2] Introduce vfs_min_size API to get minimum filesystem size.
Tried to make it in accordance with your comments.
Difference to v1:
Added reply_with_error where necessary.
Changed name get_min_size -> vfs_min_size.
Maxim Perevedentsev (2):
New API: vfs_min_size
Include resize2fs_P into vfs_min_size.
daemon/Makefile.am | 1 +
daemon/daemon.h | 2 ++
daemon/ext2.c | 45 ++++++++++++++++++++++++++-----
daemon/fs-min-size.c | 49