[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/5] fw_cfg_test refactor and add two test ca
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/5] fw_cfg_test refactor and add two test cases |
Date: |
Thu, 25 Apr 2019 11:57:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 24/04/2019 16.06, Li Qiang wrote:
> In the disscuss of adding reboot timeout test case:
> https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg03304.html
>
> Philippe suggested we should uses the only related option for one
> specific test. However currently we uses one QTestState for all the
> test cases. In order to achieve Philippe's idea, I split the test case
> for its own QTestState. As this patchset has changed a lot, I don't bump
> the version.
>
> Change since v1:
> Add a patch to store the reboot_timeout as little endian
> Fix the endian issue per Thomas's review
The test still aborts on a big endian host:
$ QTEST_QEMU_BINARY=x86_64-softmmu/qemu-system-x86_64 tests/fw_cfg-test
/x86_64/fw_cfg/signature: OK
/x86_64/fw_cfg/id: OK
/x86_64/fw_cfg/uuid: OK
/x86_64/fw_cfg/ram_size: OK
/x86_64/fw_cfg/nographic: OK
/x86_64/fw_cfg/nb_cpus: OK
/x86_64/fw_cfg/max_cpus: OK
/x86_64/fw_cfg/numa: OK
/x86_64/fw_cfg/boot_menu: OK
/x86_64/fw_cfg/reboot_timeout: **
ERROR:/home/thuth/devel/qemu/tests/fw_cfg-test.c:190:test_fw_cfg_reboot_timeout:
assertion failed (reboot_timeout == 15): (251658240 == 15)
Aborted
251658240 is 0x0F000000, i.e. a byte-swapped 0xf = 15 ... i.e. you still
got an endianess issue somewhere in the code.
Thomas
- [Qemu-devel] [PATCH v2 0/5] fw_cfg_test refactor and add two test cases, Li Qiang, 2019/04/24
- [Qemu-devel] [PATCH v2 4/5] tests: fw_cfg: add reboot_timeout test case, Li Qiang, 2019/04/24
- [Qemu-devel] [PATCH v2 5/5] tests: fw_cfg: add splash time test case, Li Qiang, 2019/04/24
- [Qemu-devel] [PATCH v2 3/5] fw_cfg: reboot: store reboot-timeout as little endian, Li Qiang, 2019/04/24
- [Qemu-devel] [PATCH v2 2/5] tests: fw_cfg: add a function to get the fw_cfg file, Li Qiang, 2019/04/24
- [Qemu-devel] [PATCH v2 1/5] tests: refactor fw_cfg_test, Li Qiang, 2019/04/24
- Re: [Qemu-devel] [PATCH v2 0/5] fw_cfg_test refactor and add two test cases,
Thomas Huth <=