On 03/01/2016 08:57 AM, Martin Galvan wrote: Missing a S-o-b line; we cannot accept patches unless you sign them: http://wiki.qemu.org/Contribute/SubmitAPatch https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches?id=f6f94e2a#n297
Subject line is long, and lacking a 'topic: summary' prefix. I'd suggest:
mem: use unsigned 'len' for read/write
Then give more details, if needed, in the body of the commit message.
Also, your threading didn't work right. Patch 1/13 was sent with:
References:
<1456847859-4771-1-git-send-email-martin.gal...@tallertechnologies.com>
but with no In-Reply-To:. Meanwhile, the 00/13 cover letter was sent with:
Message-Id:
<1456847681-4532-1-git-send-email-martin.gal...@tallertechnologies.com>
which is subtly different from the references of all the other patches,
making it appear as separate threads.
> ---
> cpus.c | 2 +-
> exec.c | 29 +++++++++++++++--------------
> include/exec/cpu-all.h | 2 +-
> include/exec/cpu-common.h | 10 +++++-----
> include/exec/memory.h | 10 +++++-----
> kvm-all.c | 2 +-
> 6 files changed, 28 insertions(+), 27 deletions(-)
>
> diff --git a/cpus.c b/cpus.c
> index 9592163..e7aa8cc 100644
> --- a/cpus.c
> +++ b/cpus.c
> @@ -1602,7 +1602,7 @@ void qmp_memsave(int64_t addr, int64_t size, const char
> *filename,
> bool has_cpu, int64_t cpu_index, Error **errp)
> {
> FILE *f;
> - uint32_t l;
> + size_t l;
These are both unsigned types. So based on the subject line alone, this
doesn't fit in the patch, unless the commit message body goes into more
details on why you are changing the size of the type, and not just the
signedness.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
