[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v2 1/6] avocado: Add a Test.arch property
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [RFC PATCH v2 1/6] avocado: Add a Test.arch property |
Date: |
Fri, 27 Jul 2018 02:35:12 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
Hi Cleber,
On 06/28/2018 07:03 PM, Philippe Mathieu-Daudé wrote:
> On 06/28/2018 06:54 PM, Alex Bennée wrote:
>> Alex Bennée <address@hidden> writes:
>>> Philippe Mathieu-Daudé <address@hidden> writes:
>>>
>>>> Tests can change this property to run tests in other
>>>> architectures than the host one.
>>>>
>>>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>>>> ---
>>>> tests/acceptance/avocado_qemu/__init__.py | 17 +++++++++++++----
>>>> 1 file changed, 13 insertions(+), 4 deletions(-)
>>>>
>>>> diff --git a/tests/acceptance/avocado_qemu/__init__.py
>>>> b/tests/acceptance/avocado_qemu/__init__.py
>>>> index 1e54fd5932..6e9601f5e9 100644
>>>> --- a/tests/acceptance/avocado_qemu/__init__.py
>>>> +++ b/tests/acceptance/avocado_qemu/__init__.py
>>>> @@ -13,6 +13,7 @@ import sys
>>>>
>>>> import avocado
>>>>
>>>> +HOST_ARCH = os.uname()[4]
>>>
>>> The python docs seem to point to platform as a more stable way of
>>> querying this stuff:
>>>
>>> platform.machine() => 'x86_64' or 'aarch64'
>>>
>>>> SRC_ROOT_DIR = os.path.dirname(os.path.dirname(os.path.dirname(__file__)))
>>>> SRC_ROOT_DIR = os.path.abspath(os.path.dirname(SRC_ROOT_DIR))
>>>> sys.path.append(os.path.join(SRC_ROOT_DIR, 'scripts'))
>>>> @@ -23,12 +24,11 @@ def is_readable_executable_file(path):
>>>> return os.path.isfile(path) and os.access(path, os.R_OK | os.X_OK)
>>>>
>>>>
>>>> -def pick_default_qemu_bin():
>>>> +def pick_default_qemu_bin(arch):
>>>> """
>>>> Picks the path of a QEMU binary, starting either in the current
>>>> working
>>>> directory or in the source tree root directory.
>>>> """
>>>> - arch = os.uname()[4]
>>>> qemu_bin_relative_path = os.path.join("%s-softmmu" % arch,
>>>> "qemu-system-%s" % arch)
>>>> if is_readable_executable_file(qemu_bin_relative_path):
>>>> @@ -41,10 +41,19 @@ def pick_default_qemu_bin():
>>>>
>>>>
>>>> class Test(avocado.Test):
>>>> + _arch = HOST_ARCH
>>
>> But actually this is wrong - because the host arch may not boot machines
>> defined in the tests. I suspect each superclass needs to explicitly set
>> it's arch and we should assert it has done so here.
Do you have any suggestion on how we could clear this?
>
> Hmm but arch-specific tests are protected by the 'arch' tag:
>
> class BootLinuxConsoleMips(Test):
> """
> :avocado: enable
> :avocado: tags=endian:big
> :avocado: tags=arch:mips
> :avocado: tags=board:malta
> """
>
> Oh no they aren't, it is just a way to filter which selection of tests
> to run :|
>>>> +
>>>> + @property
>>>> + def arch(self):
>>>> + """
>>>> + Returns the architecture required to run the current test
>>>> + """
>>>> + return self._arch
>>>> +>
>>
>> --
>> Alex Bennée
>>
>>>> def setUp(self):
>>>> self.vm = None
>>>> - self.qemu_bin = self.params.get('qemu_bin',
>>>> - default=pick_default_qemu_bin())
>>>> + qemu_bin = pick_default_qemu_bin(self.arch)
>>>> + self.qemu_bin = self.params.get('qemu_bin', default=qemu_bin)
>>>> if self.qemu_bin is None:
>>>> self.cancel("No QEMU binary defined or found in the source
>>>> tree")
>>>> self.vm = QEMUMachine(self.qemu_bin)
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [RFC PATCH v2 1/6] avocado: Add a Test.arch property,
Philippe Mathieu-Daudé <=