[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/2] virtio-9p: qmp interface for set/query io t
From: |
xiezhide |
Subject: |
Re: [Qemu-devel] [PATCH 0/2] virtio-9p: qmp interface for set/query io throttle for fsdev devices |
Date: |
Tue, 13 Nov 2018 10:17:06 +0000 |
> These patches provide the qmp interface, to set/query the io throttle
> status of the all fsdev devices that are present in a vm.
> Some of the patches also remove the
> duplicate code that was present in block and fsdev files.
>
> Zhide Xie (2):
> fsdev-qmp: qmp interface for set/query io throttle for fsdev devices.
> fsdev-qmp: fix coding style issue
>
Hi Zhide Xie,
This series seems to have some formatting issues, like the broken indention in
the diffstat below for example. Also, no message threading, weird Sob lines:
Signed-off-by: x00390961 <address@hidden<mailto:address@hidden>>
and finally I couldn't even apply patch 1:
$ pwclient git-am 996225
Applying patch #996225 using 'git am'
Description: [1/2] virtio-9p: qmp interface to set/query io throttle for fsdev
devices
Applying: virtio-9p: qmp interface to set/query io throttle for fsdev devices
error: corrupt patch at line 36 <== this confirms that the patch has formatting
issues.
Patch failed at 0001 virtio-9p: qmp interface to set/query io throttle for
fsdev devices Use 'git am --show-current-patch' to see the failed patch When
you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
'git am' failed with exit status 128
Not sure what mail client was used to send this (I don't see any indication in
the mail headers), but if it's not the case already, I suggest you use git
format-patch and git send-email.
I don't have much time, so I'll wait for the well-formatted v2 before starting
review :-)
Email client is outlook, should be copy and paste issue, will fix in v2,
thanks
Thanks
Kidd