From: Stefan Weil <weil@mail.berlios.de>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
Anthony Liguori <aliguori@us.ibm.com>,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v4 2/7] qtest: add test framework
Date: Fri, 30 Mar 2012 19:52:00 +0200 [thread overview]
Message-ID: <4F75F2C0.3080403@mail.berlios.de> (raw)
In-Reply-To: <4F75F26A.80801@codemonkey.ws>
Am 30.03.2012 19:50, schrieb Anthony Liguori:
> On 03/30/2012 12:40 PM, Stefan Weil wrote:
>> Hi
>>
>> this commit breaks builds on Linux i386:
>>
>> CC libhw64/qtest.o
>> cc1: warnings being treated as errors
>> /home/stefan/src/qemu/qemu.org/qemu/qtest.c: In function
>> ‘qtest_send_prefix’:
>> /home/stefan/src/qemu/qemu.org/qemu/qtest.c:156: error: format ‘%lld’
>> expects
>> type ‘long long int’, but argument 3 has type ‘__time_t’
>> /home/stefan/src/qemu/qemu.org/qemu/qtest.c:156: error: format
>> ‘%06lld’ expects
>> type ‘long long int’, but argument 4 has type ‘__suseconds_t’
>> (more follow)
>>
>> The size of the timeval elements is only 32 bit on my Debian
>> installation,
>> Therefore this format string in qtest.c does not work:
>>
>> +#define FMT_timeval "%" PRId64 ".%06" PRId64
>
> Does %ld work instead of PRId64?
That works for my 32 bit Debian, but it would fail on 64 bit Linux.
>
> Regards,
>
> Anthony Liguori
>
>>
>> Regards,
>> Stefan W.
>>
>>
>
>
next prev parent reply other threads:[~2012-03-30 17:52 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-28 13:42 [Qemu-devel] [PATCH v4 0/7] new test infrastructure + qtest Paolo Bonzini
2012-03-28 13:42 ` [Qemu-devel] [PATCH v4 1/7] test makefile overhaul Paolo Bonzini
2012-03-28 13:42 ` [Qemu-devel] [PATCH v4 2/7] qtest: add test framework Paolo Bonzini
2012-03-30 17:40 ` Stefan Weil
2012-03-30 17:50 ` Anthony Liguori
2012-03-30 17:52 ` Stefan Weil [this message]
2012-03-30 17:53 ` Anthony Liguori
2012-03-30 18:37 ` Paolo Bonzini
2012-03-28 13:42 ` [Qemu-devel] [PATCH v4 3/7] qtest: IRQ interception infrastructure Paolo Bonzini
2012-03-28 13:42 ` [Qemu-devel] [PATCH v4 4/7] qtest: add clock management Paolo Bonzini
2012-03-28 13:42 ` [Qemu-devel] [PATCH v4 5/7] qtest: add C version of test infrastructure Paolo Bonzini
2012-03-28 13:42 ` [Qemu-devel] [PATCH v4 6/7] rtc: split out macros into a header file and use in test case Paolo Bonzini
2012-03-28 13:42 ` [Qemu-devel] [PATCH v4 7/7] qtest: add rtc-test test-case Paolo Bonzini
2012-03-30 17:15 ` Blue Swirl
2012-03-30 17:22 ` Blue Swirl
2012-03-30 17:49 ` Anthony Liguori
2012-03-30 17:53 ` Blue Swirl
2012-03-30 17:29 ` Anthony Liguori
2012-03-30 17:37 ` Blue Swirl
2012-03-28 17:43 ` [Qemu-devel] [PATCH v4 0/7] new test infrastructure + qtest Anthony Liguori
2012-03-30 14:45 ` Anthony Liguori
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=4F75F2C0.3080403@mail.berlios.de \
--to=weil@mail.berlios.de \
--cc=aliguori@us.ibm.com \
--cc=anthony@codemonkey.ws \
--cc=pbonzini@redhat.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).