On 05/28/2013 09:27 AM, Kevin Wolf wrote: > This is the implementation of all qemu-io commands that make sense to be > called from the qemu monitor, i.e. everything except open, close and > quit. > > Signed-off-by: Kevin Wolf <kw...@redhat.com> > --- > Makefile | 2 +- > qemu-io-cmds.c | 1835 > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > qemu-io.c | 1817 +------------------------------------------------------ > 3 files changed, 1838 insertions(+), 1816 deletions(-) > create mode 100644 qemu-io-cmds.c
I checked the bulk of this patch via: $ diff -u <(sed -n '/^-/ s///p' file) <(sed -n '/^\+/ s///p' file) The bulk of the patch is blind code motion plus a bit of touchup; I did however spot one case where you were too eager in your search-and-replace: * Memory allocation helpers. * - * Make sure memory is aligned by default, or purposefully misaligned if + * Make sure memory is aligned by default, or purposefully qemuio_misaligned if * that is specified on the command line. > + > +int qemuio_misalign; Should this variable be typed 'bool'? Other than that, Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature