From: John Snow <jsnow@redhat.com>
To: Ed Maste <emaste@freebsd.org>
Cc: Peter Maydell <peter.maydell@linaro.org>,
qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH 4/4] ahci-test: improve rw buffer patterns
Date: Tue, 31 Mar 2015 16:58:49 -0400 [thread overview]
Message-ID: <551B0A89.8060507@redhat.com> (raw)
In-Reply-To: <CAPyFy2BuV2EzpqbjiGr8jfeFbKcLPtUj0=S_0Zen3Qx_+d0a9w@mail.gmail.com>
On 03/31/2015 04:55 PM, Ed Maste wrote:
> On 23 March 2015 at 12:56, John Snow <jsnow@redhat.com> wrote:
>> My pattern was cyclical every 256 bytes, so it missed a fairly obvious
>> failure case. Add some rand() pepper into the test pattern, and for large
>> patterns that exceed 256 sectors, start writing an ID per-sector so that
>> we never generate identical sector patterns.
>>
>> Signed-off-by: John Snow <jsnow@redhat.com>
>> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
>> Tested-by: Andreas Färber <afaerber@suse.de>
>> Message-id: 1426811056-2202-5-git-send-email-jsnow@redhat.com
>> ---
>> tests/ahci-test.c | 36 ++++++++++++++++++++++++++++--------
>> 1 file changed, 28 insertions(+), 8 deletions(-)
>
> As of the introduction of this test the test suite fails on FreeBSD,
> with 6 errors of the form:
>
> ERROR:tests/ahci-test.c:790:ahci_test_io_rw_simple: assertion failed
> (memcmp(tx, rx, bufsize) == 0): (0xffffffffffffffff == 0x00000000)
> GTester: last random seed: R02S76f68c4cfb2d54af66de227ed44fd54a
>
> I haven't yet done any investigation beyond bisecting.
>
which test case does it fail under? io_rw_simple is shared by a number
of different tests.
I suspect deeply that this test pattern has just unearthed an existing
problem.
next prev parent reply other threads:[~2015-03-31 20:59 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-23 16:56 [Qemu-devel] [PATCH 0/4] Ide patches for 2.3-rc1 John Snow
2015-03-23 16:56 ` [Qemu-devel] [PATCH 1/4] ide: fix cmd_write_pio when nsectors > 1 John Snow
2015-03-23 16:56 ` [Qemu-devel] [PATCH 2/4] ide: fix cmd_read_pio " John Snow
2015-03-23 16:56 ` [Qemu-devel] [PATCH 3/4] ahci: Fix sglist offset manipulation for BE machines John Snow
2015-03-23 16:56 ` [Qemu-devel] [PATCH 4/4] ahci-test: improve rw buffer patterns John Snow
2015-03-31 20:55 ` Ed Maste
2015-03-31 20:58 ` John Snow [this message]
2015-03-31 21:29 ` Ed Maste
2015-03-31 21:44 ` John Snow
2015-03-31 21:55 ` Ed Maste
2015-03-31 22:01 ` John Snow
2015-04-01 0:29 ` Ed Maste
2015-04-01 0:34 ` John Snow
2015-03-23 16:57 ` [Qemu-devel] [PATCH 0/4] Ide patches for 2.3-rc1 John Snow
2015-03-23 17:56 ` Peter Maydell
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=551B0A89.8060507@redhat.com \
--to=jsnow@redhat.com \
--cc=emaste@freebsd.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/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.