[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerat
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator |
Date: |
Tue, 22 Aug 2017 21:20:46 +1000 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Tue, Aug 22, 2017 at 10:35:17AM +0100, Peter Maydell wrote:
> On 22 August 2017 at 09:49, Laurent Vivier <address@hidden> wrote:
> > On 22/08/2017 10:47, Peter Maydell wrote:
> >> On 22 August 2017 at 02:09, David Gibson <address@hidden> wrote:
> >>> On Wed, Aug 16, 2017 at 12:18:07PM +0100, Peter Maydell wrote:
> >>>> On 16 August 2017 at 11:51, Paolo Bonzini <address@hidden> wrote:
> >>>>> On 16/08/2017 10:26, Cornelia Huck wrote:
> >>>>>> Prefer to use the tcg accelarator if it is available: This is our only
> >>>>>> real smoke test for tcg, and fast enough to use it for that.
> >>>>>
> >>>>> I'm not sure this is required for 2.10. Yes, it means the coverage from
> >>>>> "make check" is worse, but that's it.
> >>>>
> >>>> Yes, I'd put it under "if we need to roll an rc4 anyway for
> >>>> some more significant bug we might as well put this in too,
> >>>> but it doesn't merit cutting rc4 by itself."
> >>>
> >>> It does entirely break "make check" on a ppc host. And that in turn
> >>> has held up my testing cycle for a couple of ppc regressions from 2.9
> >>> that I was hoping to squeeze in. Does that change your calculations?
> >>
> >> I have a PPC64 box in my standard set of build tests, and it
> >> runs 'make check' without problems...
> >
> > You need to use KVM HV to have the problem, not KVM PR.
> > Is that the case?
>
> I don't have access to KVM at all on that box, so it will be
> using TCG only.
That would explain it, it's the attempt to use KVM that triggers the
problem.
Obviously it's not a thing to fix right now, but I've really been
thinking that none of the tests should use this "TCG or KVM" stuff.
They should instead be run with *both* options - or at least the ones
that are available on the host.
That would have caught the bug in the pull request I sent you, and
would at least have given you a chance at seeing the problem with
boot-serial-test.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, (continued)
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Laurent Vivier, 2017/08/16
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Thomas Huth, 2017/08/16
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Paolo Bonzini, 2017/08/16
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Peter Maydell, 2017/08/16
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Cornelia Huck, 2017/08/16
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, David Gibson, 2017/08/21
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Thomas Huth, 2017/08/22
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Peter Maydell, 2017/08/22
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Laurent Vivier, 2017/08/22
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Peter Maydell, 2017/08/22
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator,
David Gibson <=
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Cornelia Huck, 2017/08/22
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, David Gibson, 2017/08/22
- Re: [Qemu-devel] [PATCH for-2.10] boot-serial-test: prefer tcg accelerator, Cornelia Huck, 2017/08/23
- [Qemu-devel] make check speed (was: Re: [PATCH for-2.10] boot-serial-test: prefer tcg accelerator), Thomas Huth, 2017/08/23
- Re: [Qemu-devel] make check speed, Paolo Bonzini, 2017/08/23
- Re: [Qemu-devel] make check speed, Thomas Huth, 2017/08/23
- Re: [Qemu-devel] make check speed, Fam Zheng, 2017/08/23
- Re: [Qemu-devel] make check speed, Fam Zheng, 2017/08/23
- Re: [Qemu-devel] make check speed, Paolo Bonzini, 2017/08/23
- Re: [Qemu-devel] make check speed, David Gibson, 2017/08/23