From: Eric Blake <eblake@redhat.com>
To: Amit Shah <amit.shah@redhat.com>, qemu list <qemu-devel@nongnu.org>
Cc: "Juan Quintela" <quintela@redhat.com>,
"Alexander Graf" <agraf@suse.de>,
"Markus Armbruster" <armbru@redhat.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Andreas Färber" <afaerber@suse.de>,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v3 00/18] migration: add static analysis tool to check vmstate compat
Date: Tue, 17 Jun 2014 07:34:07 -0600 [thread overview]
Message-ID: <53A043CF.9030800@redhat.com> (raw)
In-Reply-To: <cover.1403011018.git.amit.shah@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 480 bytes --]
On 06/17/2014 07:19 AM, Amit Shah wrote:
> Hello,
>
> v3:
> - Python script returns an error code: 0 if no errors, positive for
> the number of errors identified.
Returning the error count as $? is unreliable - if you have exactly 256
errors, that gives the same status as if you had success. Does your
code guarantee non-zero return in that case?
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]
next prev parent reply other threads:[~2014-06-17 13:34 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-17 13:19 [Qemu-devel] [PATCH v3 00/18] migration: add static analysis tool to check vmstate compat Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 01/18] migration: dump vmstate info as a json file for static analysis Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 02/18] vmstate-static-checker: script to validate vmstate changes Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 03/18] tests: vmstate static checker: add dump1 and dump2 files Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 04/18] tests: vmstate static checker: incompat machine types Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 05/18] tests: vmstate static checker: add version error in main section Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 06/18] tests: vmstate static checker: version mismatch inside a Description Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 07/18] tests: vmstate static checker: minimum_version_id check Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 08/18] tests: vmstate static checker: remove a section Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 09/18] tests: vmstate static checker: remove a field Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 10/18] tests: vmstate static checker: remove last field in a struct Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 11/18] tests: vmstate static checker: change description name Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 12/18] tests: vmstate static checker: remove Fields Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 13/18] tests: vmstate static checker: remove Description Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 14/18] tests: vmstate static checker: remove Description inside Fields Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 15/18] tests: vmstate static checker: remove a subsection Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 16/18] tests: vmstate static checker: remove Subsections Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 17/18] tests: vmstate static checker: add substructure for usb-kbd for hid section Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 18/18] tests: vmstate static checker: add size mismatch inside substructure Amit Shah
2014-06-17 13:34 ` Eric Blake [this message]
2014-06-17 13:59 ` [Qemu-devel] [PATCH v3 00/18] migration: add static analysis tool to check vmstate compat Amit Shah
2014-06-17 15:25 ` Eric Blake
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53A043CF.9030800@redhat.com \
--to=eblake@redhat.com \
--cc=afaerber@suse.de \
--cc=agraf@suse.de \
--cc=amit.shah@redhat.com \
--cc=armbru@redhat.com \
--cc=dgilbert@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.