feat: Add EB Linux support to QEMU ITF plugin by lurtz · Pull Request #123 · eclipse-score/itf

lurtz · 2026-07-15T15:39:55Z

So far the QEMU plugin code has only been able to run QNX images, which are booted via the -kernel QEMU parameter. This is not enough for Linux images, where the file system is typically not part of the kernel image. In addition to that the EB Linux image is only compiled for AARCH64 for which some architecture abstraction is also needed.

The changes aim to be backwards compatible so that existing code using the current interface of the QEMU ITF plugin stay working without changes.

As drive-by fixes Docker tests have been excluded from QNX test runs. Now bazel test //test/... --config=qemu-integration works as well.

Next steps

This is only sufficient to run the image, but uploading binaries and test data is still missing. I can either do this in this PR or to keep it small in a future PR. I also plan to add Ubuntu support, which typically seems to come in conjunction with cloud-init.

PR elektrobit-contrib#1 is a preview of the next changes.

Copilot

Pull request overview

This PR extends the QEMU ITF plugin beyond the existing QNX -kernel flow to support EB Linux images by adding (a) separate disk image handling and (b) architecture selection (x86_64 vs aarch64). It also updates CI/test configuration to make QEMU-based integration tests easier to run on Linux.

Changes:

Add architecture abstraction and Linux disk-image support (with ephemeral qcow2 overlay) to the QEMU plugin.
Add EBclfsa aarch64 QEMU integration-test resources + a new ping integration test target.
Adjust Bazel/CI config to run the relevant tests/builds on Linux hosts and install QEMU in GitHub Actions.

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
test/unit/BUILD	Restrict selected unit tests to Linux platforms.
test/resources/ebclfsa_aarch64/config-overlay/etc/config/network/network	Add static network configuration overlay for EBclfsa image customization.
test/resources/ebclfsa_aarch64/build_image.sh	New script to boot EBclfsa image in QEMU and deploy tar payloads into the image.
test/resources/ebclfsa_aarch64/BUILD	Add rules to extract fastdev archive and build an ITF-ready EBclfsa image.
test/resources/ebclfsa_aarch64_qemu/qemu_config.json	Provide QEMU network/SSH/core/RAM settings for EBclfsa integration tests.
test/resources/ebclfsa_aarch64_qemu/kernel_cmdline.txt	Provide Linux kernel cmdline for EBclfsa QEMU boot.
test/resources/ebclfsa_aarch64_qemu/BUILD	Export EBclfsa QEMU config artifacts and alias image/kernel targets.
test/resources/BUILD	Mark OCI image/load targets as Linux-compatible only.
test/integration/test_ebclfsa_ping.py	New integration test validating host↔target connectivity.
test/integration/BUILD	Add EBclfsa ping test and include it in the manual QEMU test suite; restrict selected tests to Linux.
score/itf/plugins/qemu/qemu.py	Add architecture-aware QEMU command construction; support optional kernel cmdline and disk image.
score/itf/plugins/qemu/qemu_target.py	Extend target setup to pass architecture/disk/cmdline into QEMU process creation.
score/itf/plugins/qemu/qemu_process.py	Wire new QEMU parameters through the process wrapper.
score/itf/plugins/qemu/init.py	Add CLI options for disk image/arch/kernel cmdline and create qcow2 overlays for disk images.
MODULE.bazel	Add http_file repo for EBclfsa fastdev archive used by integration resources.
.github/workflows/itf.yml	Install QEMU + enable KVM perms in CI for general build/test job.
.bazelrc	Add `--build_tests_only` to the qemu-integration test config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

clanghans

all points addressed. LGTM

revert unneeded changes

This requires using a different machine and architecture, as well as specifying a kernel and rootfs image with kernel command line parameters.

PiotrKorkus · 2026-08-10T16:37:09Z

-        ram="1G",
-        cores="2",
-        cpu="Cascadelake-Server-v5",
-        network_adapters=[],
-        port_forwarding=[],


why drop default params?

QemuProcess always sets all of them:

itf/score/itf/plugins/qemu/qemu_process.py

Lines 29 to 35 in 63a3f10

self._qemu = Qemu(

self._path_to_qemu_image,

self._available_ram,

self._available_cores,

network_adapters=self._network_adapters,

port_forwarding=self._port_forwarding,

)

Thus it looked unneeded.

PiotrKorkus · 2026-08-10T16:40:59Z

+            logger.fatal(f"Qemu is not installed under {qemu_path}")
            sys.exit(-1)

-    def __find_available_kvm_support(self):


whats wrong with detecting accel and setting accordingly?

QEMU can do it for us. There is no need to replicate that

PiotrKorkus · 2026-08-10T16:42:06Z

+                "-accel",
+                "kvm",
+                "-accel",
+                "tcg",


isnt hardcoded kvm together with "cpu": "cortex-a53", from SUPPORTED_MACHINES going to crash as CPU will be rejected?

KVM is the accelerator selected with highest priority. If that is not supported QEMU will select tcg. In this case QNX x86_64 runs with KVM and ebclfsa AARCH64 runs with tcg. If that would crash we would have a failing build already.

PiotrKorkus · 2026-08-10T16:43:34Z

+            "-device",
+            f"{self.__arch_config['block_device']},drive=vd0",
+            "-drive",
+            f"if=none,format=qcow2,file={self.__rootfs},id=vd0",


format hardcoded, use _get_image_format()

There I can be quite sure that it is qcow2, because that one has been created at https://github.com/elektrobit-contrib/eclipse-score_itf/blob/3ac4f262cda9c6f5dd94fd42f3c3c0d075a36a89/score/itf/plugins/qemu/__init__.py#L37-L61

PiotrKorkus · 2026-08-10T16:45:16Z

+    if not qemu_kernel and not rootfs:
+        raise ValueError("At least one of --qemu-kernel or --qemu-rootfs must be specified for QEMU tests.")


Previously omitting --qemu-image gave a QemuTarget(None, config) talking to a booted target

I see. I wondered what nullcontext() was used for. I will fix it tomorrow

PiotrKorkus · 2026-08-10T16:56:09Z

+exports_files(
+    [
+        "qemu_config.json",
+    ],
+    visibility = ["//:__subpackages__"],
+)


is it needed when we have filegroup below?

github-project-automation Bot added this to TST - Testing Community Jul 15, 2026

github-project-automation Bot moved this to Backlog in TST - Testing Community Jul 15, 2026

lurtz mentioned this pull request Jul 15, 2026

feat(qemu): add _extra_qemu_args() hook #122

Merged

lurtz requested a review from Copilot July 15, 2026 20:41

Copilot started reviewing on behalf of lurtz July 15, 2026 20:41 View session

Copilot AI reviewed Jul 15, 2026

View reviewed changes

lurtz force-pushed the add-eb-linux-support-to-qemu-plugin branch from e3ae98a to bfd4ccf Compare July 16, 2026 09:01

lurtz marked this pull request as ready for review July 16, 2026 09:55

clanghans suggested changes Aug 3, 2026

View reviewed changes

clanghans reviewed Aug 6, 2026

View reviewed changes

Comment thread score/itf/plugins/qemu/__init__.py Outdated

clanghans approved these changes Aug 10, 2026

View reviewed changes

lurtz force-pushed the add-eb-linux-support-to-qemu-plugin branch from c276dd0 to d6aa834 Compare August 10, 2026 09:31

lurtz added 2 commits August 10, 2026 09:39

restrict Docker and Python based tests to Linux

9676ace

revert unneeded changes

feat: Add QEMU plugin support for running EB Linux

3ac4f26

This requires using a different machine and architecture, as well as specifying a kernel and rootfs image with kernel command line parameters.

lurtz force-pushed the add-eb-linux-support-to-qemu-plugin branch from d6aa834 to 3ac4f26 Compare August 10, 2026 09:40

lurtz enabled auto-merge August 10, 2026 11:34

PiotrKorkus reviewed Aug 10, 2026

View reviewed changes

PiotrKorkus requested review from draganbjedov and ltekieli August 10, 2026 16:57

lurtz added 3 commits August 11, 2026 10:08

remove enforcement of image and rootfs parameters

52f7dab

remove unneeded exports_files

48d3c29

determine image format using qemu-img info

816f108

lurtz force-pushed the add-eb-linux-support-to-qemu-plugin branch from de6c878 to 816f108 Compare August 11, 2026 14:38

draganbjedov previously approved these changes Aug 12, 2026

View reviewed changes

lurtz added this pull request to the merge queue Aug 12, 2026

github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 12, 2026

make image format public function for stable testing

96c6081

lurtz dismissed draganbjedov’s stale review via 96c6081 August 12, 2026 13:47

lurtz enabled auto-merge August 12, 2026 13:48

lurtz mentioned this pull request Aug 12, 2026

feat: Use more generic ITF QEMU plugin eclipse-score/inc_someip_gateway#177

Draft

3 tasks

	self._qemu = Qemu(
	self._path_to_qemu_image,
	self._available_ram,
	self._available_cores,
	network_adapters=self._network_adapters,
	port_forwarding=self._port_forwarding,
	)

		if not qemu_kernel and not rootfs:
		raise ValueError("At least one of --qemu-kernel or --qemu-rootfs must be specified for QEMU tests.")

Uh oh!

Conversation

lurtz commented Jul 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Next steps

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

clanghans left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

lurtz commented Jul 15, 2026 •

edited

Loading