From: Markus Armbruster <armbru@redhat.com>
To: Cleber Rosa <crosa@redhat.com>
Cc: Stefan Hajnoczi <stefanha@gmail.com>,
Kevin Wolf <kwolf@redhat.com>,
qemu-block@nongnu.org, Jing Liu <liujbjl@linux.vnet.ibm.com>,
qemu-devel@nongnu.org, Max Reitz <mreitz@redhat.com>,
John Snow <jsnow@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 0/3] build configuration query tool and conditional (qemu-io)test skip
Date: Tue, 08 Aug 2017 10:01:55 +0200 [thread overview]
Message-ID: <87r2wm4he4.fsf@dusky.pond.sub.org> (raw)
In-Reply-To: <f263a454-9dca-ecba-89b8-d09bee6b79ec@redhat.com> (Cleber Rosa's message of "Fri, 21 Jul 2017 09:49:11 -0400")
Cleber Rosa <crosa@redhat.com> writes:
> On 07/21/2017 08:33 AM, Stefan Hajnoczi wrote:
>> On Thu, Jul 20, 2017 at 11:47:27PM -0400, Cleber Rosa wrote:
>>> This is a follow up to a previous discussion about reported failures when
>>> running some qemu-iotests. Turns out the failures were due to missing
>>> libraries, which in turn, reflected on the host build configuration.
>>>
>>> This series introduces a tool that can check both host and target level
>>> build configurations. On top of that, it adds a function to to be used
>>> on qemu-iotests. Finally, as an example, it sets a test to be skipped
>>> if the required feature is not enable on the host build configuration.
>>>
>>> Cleber Rosa (3):
>>> scripts: introduce buildconf.py
>>> qemu-iotests: add _require_feature() function
>>> qemu-iotests: require CONFIG_LINUX_AIO for test 087
>>>
>>> scripts/buildconf.py | 278 +++++++++++++++++++++++++++++++++++++++++++
>>> tests/qemu-iotests/087 | 1 +
>>> tests/qemu-iotests/check | 2 +
>>> tests/qemu-iotests/common.rc | 7 ++
>>> 4 files changed, 288 insertions(+)
>>>
>>
>> It should be possible to run iotests against any
>> qemu/qemu-img/qemu-io/qemu-nbd binaries - even if no build root is
>> available.
>>
>
> Yes, I actually overlooked that point.
>
>> How about invoking qemu-img and tools to determine their capabilities?
>>
>
> Can capabilities be consistently queried? I would love to not count on
> a build root if the same information can be consistently queried from
> the binaries themselves.
>
>> At the beginning of ./check, query the qemu/qemu-img/qemu-io/qemu-nbd
>> binaries for specific features. This produces a set of available
>> features and tests can say:
>>
>
> Which would be another ad-hoc thing, limited to qemu-iotests. From a
> test writer perspective, QEMU lacks is a uniform way to introspect its
> capabilities.
The closest we have is query-qmp-schema. It's uniform, but limited to
the qapified part of QMP (see my KVM Forum 2015 talk[*] for details).
Something similar for the command line would be nice, and I hope to get
there some day. Until then, you can often reason like "if QMP supports
X, then surely the command line supports X'".
We commonly reason "if INTERFACE supports FEATURE-API, then QEMU surely
supports FEATURE".
[...]
[*] https://events.linuxfoundation.org/sites/events/files/slides/armbru-qemu-introspection.pdf
next prev parent reply other threads:[~2017-08-08 8:02 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-21 3:47 [Qemu-devel] [PATCH 0/3] build configuration query tool and conditional (qemu-io)test skip Cleber Rosa
2017-07-21 3:47 ` [Qemu-devel] [PATCH 1/3] scripts: introduce buildconf.py Cleber Rosa
2017-07-21 14:00 ` Eric Blake
2017-07-21 14:07 ` Cleber Rosa
2017-07-21 3:47 ` [Qemu-devel] [PATCH 2/3] qemu-iotests: add _require_feature() function Cleber Rosa
2017-07-21 3:47 ` [Qemu-devel] [PATCH 3/3] qemu-iotests: require CONFIG_LINUX_AIO for test 087 Cleber Rosa
2017-07-24 6:44 ` Jing Liu
2017-07-25 15:45 ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2017-07-25 15:48 ` Daniel P. Berrange
2017-07-26 8:55 ` Jing Liu
2017-07-26 9:49 ` Daniel P. Berrange
2017-07-21 4:39 ` [Qemu-devel] [PATCH 0/3] build configuration query tool and conditional (qemu-io)test skip no-reply
2017-07-21 12:33 ` Stefan Hajnoczi
2017-07-21 13:49 ` Cleber Rosa
2017-08-08 8:01 ` Markus Armbruster [this message]
2017-07-21 14:01 ` Daniel P. Berrange
2017-07-21 14:21 ` Cleber Rosa
2017-07-25 15:49 ` Stefan Hajnoczi
2017-07-25 16:16 ` Cleber Rosa
2017-07-25 16:24 ` Daniel P. Berrange
2017-07-25 16:47 ` Cleber Rosa
2017-07-26 17:58 ` Stefan Hajnoczi
2017-07-26 18:24 ` Cleber Rosa
2017-07-27 13:41 ` Stefan Hajnoczi
2017-08-08 8:06 ` Markus Armbruster
2017-08-08 12:44 ` Stefan Hajnoczi
2017-08-08 14:52 ` Markus Armbruster
2017-08-09 10:30 ` Stefan Hajnoczi
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=87r2wm4he4.fsf@dusky.pond.sub.org \
--to=armbru@redhat.com \
--cc=crosa@redhat.com \
--cc=jsnow@redhat.com \
--cc=kwolf@redhat.com \
--cc=liujbjl@linux.vnet.ibm.com \
--cc=mreitz@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.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.