We're currently using the x86_64 qemu for i386 builds. While this is not
incorrect, it's probably more sensible to use the i386 one, which will
at least fail properly if we accidentally were to build a 64-bit kernel.
Signed-off-by: David Gow <[email protected]>
---
tools/testing/kunit/qemu_configs/i386.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/kunit/qemu_configs/i386.py b/tools/testing/kunit/qemu_configs/i386.py
index 52b80be40e4b..4463ebefd567 100644
--- a/tools/testing/kunit/qemu_configs/i386.py
+++ b/tools/testing/kunit/qemu_configs/i386.py
@@ -4,7 +4,7 @@ QEMU_ARCH = QemuArchParams(linux_arch='i386',
kconfig='''
CONFIG_SERIAL_8250=y
CONFIG_SERIAL_8250_CONSOLE=y''',
- qemu_arch='x86_64',
+ qemu_arch='i386',
kernel_path='arch/x86/boot/bzImage',
kernel_command_line='console=ttyS0',
extra_qemu_params=[])
--
2.36.0.550.gb090851708-goog
On Fri, May 13, 2022 at 10:56 AM Daniel Latypov <[email protected]> wrote:
>
> On Fri, May 13, 2022 at 1:51 AM David Gow <[email protected]> wrote:
> >
> > We're currently using the x86_64 qemu for i386 builds. While this is not
> > incorrect, it's probably more sensible to use the i386 one, which will
> > at least fail properly if we accidentally were to build a 64-bit kernel.
> >
> > Signed-off-by: David Gow <[email protected]>
>
> Tested-by: Daniel Latypov <[email protected]>
>
> Works for me:
> [10:55:58] Testing complete. Ran 201 tests: passed: 165, skipped: 36
> [10:55:58] Elapsed time: 21.268s total, 0.003s configuring, 4.009s
> building, 16.967s running
>
> Side note:
> I ran into an unrelated bug where running on QEMU in general seems to
> hang when you try to filter to a test suite that doesn't exist.
> I've confirmed it happens on i386 before this patch, and it at least
> also happens on x86_64.
Ah, this "unrelated bug" is my fault, specifically commit a02353f49162
("kunit: bail out of test filtering logic quicker if OOM").
The use of _any_ filter glob is broken on kunit.py + QEMU.
Fix is https://lore.kernel.org/linux-kselftest/[email protected]/
On Fri, May 13, 2022 at 1:51 AM David Gow <[email protected]> wrote:
>
> We're currently using the x86_64 qemu for i386 builds. While this is not
> incorrect, it's probably more sensible to use the i386 one, which will
> at least fail properly if we accidentally were to build a 64-bit kernel.
>
> Signed-off-by: David Gow <[email protected]>
Tested-by: Daniel Latypov <[email protected]>
Works for me:
[10:55:58] Testing complete. Ran 201 tests: passed: 165, skipped: 36
[10:55:58] Elapsed time: 21.268s total, 0.003s configuring, 4.009s
building, 16.967s running
Side note:
I ran into an unrelated bug where running on QEMU in general seems to
hang when you try to filter to a test suite that doesn't exist.
I've confirmed it happens on i386 before this patch, and it at least
also happens on x86_64.
On Fri, May 13, 2022 at 4:51 AM David Gow <[email protected]> wrote:
>
> We're currently using the x86_64 qemu for i386 builds. While this is not
> incorrect, it's probably more sensible to use the i386 one, which will
> at least fail properly if we accidentally were to build a 64-bit kernel.
>
> Signed-off-by: David Gow <[email protected]>
A very sensible thing to do! When I was writing this, I probably
copied and pasted the x64_64 qemu config and then forgot to change
everything to i386 - whoops.
Reviewed-by: Brendan Higgins <[email protected]>