Re: [libvirt] [PATCH 0/6] virsh: Add support for byte granularity and scaled integers for virsh block APIs

On 03/18/2016 04:56 AM, Peter Krempa wrote: > Quite some time ago we've added support for byte granularity for block job > bandwidth. Make it work in virsh and add support for scaled integers too. > > Peter Krempa (6): > vsh: Tweak error message for scaled integers > vsh: Refactor vshCommandOptScaledInt > virsh: blockjob: Support --bytes and scaled integers as bandwidth > virsh: blockcommit: Support --bytes and scaled integers > virsh: blockcopy: Support --bytes and scaled integers > virsh: blockpull: Support --bytes and scaled integers > > tests/virsh-optparse | 6 ++--- > tools/virsh-domain.c | 73 ++++++++++++++++++++++++++++++++++++---------------- > tools/virsh.pod | 37 ++++++++++++++------------ > tools/vsh.c | 66 +++++++++++++++++++++++++++++++++++++++++------ > tools/vsh.h | 4 +++ > 5 files changed, 137 insertions(+), 49 deletions(-) > Note specific nits from patch 3 I think the commit messages for patches 4-6 shouldn't be: "Reuse the approach and helper from the last patch." since "last patch" causes me to go find the "last patch"... cut-copy-paste what they're using vshBlockJobOptionBandwidth and allowing the --bytes on the set. If we really wanted to be picky, patch 3 could introduce the new function with patches 4-7 using it and indicating so in their commit messages. ACK series with at least the typos fixed and commit messages adjusted. Your choice if you want to extract out the function.