[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 3/6] tests/acceptance: Test OpenBI
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 3/6] tests/acceptance: Test OpenBIOS on the PReP/40p |
Date: |
Tue, 17 Sep 2019 11:44:50 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 9/17/19 11:31 AM, Alex Bennée wrote:
> Cleber Rosa <address@hidden> writes:
>> On Sun, Sep 15, 2019 at 11:19:37PM +0200, Philippe Mathieu-Daudé wrote:
>>> User case from:
>>> https://mail.coreboot.org/pipermail/openbios/2018-May/010360.html
>>>
>>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>>> ---
>>> tests/acceptance/ppc_prep_40p.py | 32 ++++++++++++++++++++++++++++++++
>>> 1 file changed, 32 insertions(+)
>>>
>>> diff --git a/tests/acceptance/ppc_prep_40p.py
>>> b/tests/acceptance/ppc_prep_40p.py
>>> index a0eac40d9f..87b5311b89 100644
>>> --- a/tests/acceptance/ppc_prep_40p.py
>>> +++ b/tests/acceptance/ppc_prep_40p.py
>>> @@ -82,3 +82,35 @@ class IbmPrep40pMachine(Test):
>>> self.wait_for_console_pattern(fw_banner)
>>> prompt_msg = 'Type any key to interrupt automatic startup'
>>> self.wait_for_console_pattern(prompt_msg)
>>> +
>>> + def test_openbios_192m(self):
>>> + """
>>> + :avocado: tags=arch:ppc
>>> + :avocado: tags=machine:40p
>>> + """
>>> + self.vm.set_machine('40p')
>>> + self.vm.set_console()
>>> + self.vm.add_args('-m', '192')
>>
>> Is 192 a magic number (some kind of limit)? Or just a value to check
>> against later?
>
> Just the memory configured for the machine, checked bellow. Or are you
> wondering why 192m and not any other amount?
The machine default is 128MiB. The firmware asks the memory controller
how many SIMMs of 8MiB are inserted. We can run OpenBIOS with 128MiB,
but by specifying a different default in this test, we also test the
memory controller device :)
Since this is not obvious I'll add a comment.
>>> +
>>> + self.vm.launch()
>>> + self.wait_for_console_pattern('>> OpenBIOS')
>>> + self.wait_for_console_pattern('>> Memory: 192M')
>>> + self.wait_for_console_pattern('>> CPU type PowerPC,604')
>>> +
>>
>> On my testing, this is a very stable test, I'm only getting PASSes.
> <snip>
>
> --
> Alex Bennée
>
- Re: [Qemu-ppc] [PATCH v2 2/6] tests/acceptance: Test Open Firmware on the PReP/40p, (continued)
[Qemu-ppc] [PATCH v2 1/6] tests/acceptance: Add test that runs NetBSD 4.0 installer on PRep/40p, Philippe Mathieu-Daudé, 2019/09/15
[Qemu-ppc] [PATCH v2 3/6] tests/acceptance: Test OpenBIOS on the PReP/40p, Philippe Mathieu-Daudé, 2019/09/15
[Qemu-ppc] [PATCH v2 4/6] tests/acceptance: Test Sandalfoot initrd on the PReP/40p, Philippe Mathieu-Daudé, 2019/09/15
[Qemu-ppc] [PATCH v2 5/6] .travis.yml: Let the avocado job run the 40p tests, Philippe Mathieu-Daudé, 2019/09/15
[Qemu-ppc] [PATCH v2 6/6] .travis.yml: Split enterprise vs. hobbyist acceptance test job, Philippe Mathieu-Daudé, 2019/09/15
Re: [Qemu-ppc] [PATCH v2 0/6] tests/acceptance: Add tests for the PReP/40p machine, David Gibson, 2019/09/16