Re: [Qemu-devel] [PATCH v4 0/3] re-enable balloon stats

[Anthony Liguori]

Hi,

Thank you for submitting your patch series.  checkpatch.pl has
detected that one or more of the patches in this series violate
the QEMU coding style.

If you believe this message was sent in error, please ignore it
or respond here with an explanation.

Otherwise, please correct the coding style issues and resubmit a
new version of the patch.

For more information about QEMU coding style, see:

http://git.qemu.org/?p=qemu.git;a=blob_plain;f=CODING_STYLE;hb=HEAD

Here is the output from checkpatch.pl:

Subject: docs: document virtio-balloon stats
ERROR: trailing whitespace
#27: FILE: docs/virtio-balloon-stats.txt:14:
+  $

ERROR: trailing whitespace
#100: FILE: docs/virtio-balloon-stats.txt:87:
+            "stat-swap-out": 0, $

ERROR: trailing whitespace
#101: FILE: docs/virtio-balloon-stats.txt:88:
+            "stat-free-memory": 844943360, $

ERROR: trailing whitespace
#102: FILE: docs/virtio-balloon-stats.txt:89:
+            "stat-minor-faults": 219028, $

ERROR: trailing whitespace
#103: FILE: docs/virtio-balloon-stats.txt:90:
+            "stat-major-faults": 235, $

ERROR: trailing whitespace
#104: FILE: docs/virtio-balloon-stats.txt:91:
+            "stat-total-memory": 1044406272, $

ERROR: trailing whitespace
#106: FILE: docs/virtio-balloon-stats.txt:93:
+        }, $

total: 7 errors, 0 warnings, 104 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

Subject: balloon: re-enable balloon stats
ERROR: trailing whitespace
#79: FILE: hw/virtio-balloon.c:62:
+   [VIRTIO_BALLOON_S_SWAP_IN] = "stat-swap-in", $

total: 1 errors, 0 warnings, 207 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

Subject: balloon: drop old stats code & API


Regards,

Anthony Liguori