[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/unini
From: |
Greg Kurz |
Subject: |
Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic |
Date: |
Mon, 12 Sep 2016 19:37:23 +0200 |
On Mon, 12 Sep 2016 17:46:35 +0200
Laurent Vivier <address@hidden> wrote:
> On 12/09/2016 03:27, David Gibson wrote:
> > On Fri, Sep 09, 2016 at 02:31:55PM +0200, Laurent Vivier wrote:
> >>
> >>
> >> On 09/09/2016 14:25, Greg Kurz wrote:
> >>> On Thu, 8 Sep 2016 09:50:31 +0200
> >>> Laurent Vivier <address@hidden> wrote:
> >>>
> >>>> On 08/09/2016 04:04, David Gibson wrote:
> >>>>> On Tue, Sep 06, 2016 at 03:17:56PM +0200, Laurent Vivier wrote:
> >>>>>> And add support for ppc64.
> >>>>>>
> >>>>>> Signed-off-by: Laurent Vivier <address@hidden>
> >>>>>
> >>>>> Some of my coments may be obsoleted by the discussion with Greg.
> >>>>>
> >>>>>> ---
> >>>>>> v2:
> >>>>>> - remove useless parenthesis, inline
> >>>> [...]
> >>>>>> +
> >>>>>> +QGuestAllocator *machine_alloc_init(void)
> >>>>>> +{
> >>>>>> + const char *arch = qtest_get_arch();
> >>>>>
> >>>>> Maybe we need to add a qtest_get_machine_type().
> >>>>
> >>>> I'm working on that...
> >>>>
> >>>
> >>> The problem is that qtest only knows about archs, based on $(TARGETS).
> >>> Maybe the machine type could be the default one for a given arch ?
> >>
> >> Once the machine is started we can use QMP[1] to ask the machine type
> >> (for instance, "pseries-2.7-machine").
> >>
> >> So what we could do is a generic qtest_machine_vboot() which ask the
> >> machine type and configure the qtest framework accordingly.
> >>
> >> Laurent
> >> [1] { 'execute': 'qom-get', 'arguments': { 'path': '/machine',
> >> 'property': 'type' } }
> >
> > Ok.. doesn't the qtest framework start the machine though? So it
> > should already know the machine type, shouldn't it?
>
> In fact qtest starts the machine with the content of QTEST_QEMU_BINARY
> and we don't provide "-machine" parameter, so it doesn't know the
> machine type. It can guess it according to the machine arch, and by
> default, ppc64 (arch) is pseries (machine).
>
> Perhaps we can add "-machine pseries" in qtest_spapr_vboot().
>
We may want to test something that is only available for a given
version (like CPU hotplug for pseries >= 2.7 for example)... it
could *work* if we can pass "-machine pseries -machine pseries-2.7"
and the latter prevails, but that looks a bit like a hack.
Cheers.
--
Greg
> Laurent
> Laurent
- Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, (continued)
Re: [Qemu-ppc] [Qemu-devel] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, Thomas Huth, 2016/09/06
Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, David Gibson, 2016/09/07
- Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, Laurent Vivier, 2016/09/08
- Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, Greg Kurz, 2016/09/09
- Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, Laurent Vivier, 2016/09/09
- Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, David Gibson, 2016/09/11
- Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, Greg Kurz, 2016/09/12
- Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic, Laurent Vivier, 2016/09/12
- Re: [Qemu-ppc] [PATCH v4 2/3] tests: make pc_alloc_init/init_flags/uninit generic,
Greg Kurz <=
[Qemu-ppc] [PATCH v4 3/3] tests: add RTAS command in the protocol, Laurent Vivier, 2016/09/06