Skip to content

image_types_qcom: hard link the ESP image into the qcomflash directory - #3110

Merged
Ricardo Salveti (ricardosalveti) merged 1 commit into
qualcomm-linux:masterfrom
ricardosalveti:perf-qcomflash-spdx
Sep 10, 2026
Merged

image_types_qcom: hard link the ESP image into the qcomflash directory#3110
Ricardo Salveti (ricardosalveti) merged 1 commit into
qualcomm-linux:masterfrom
ricardosalveti:perf-qcomflash-spdx

Conversation

@ricardosalveti

Copy link
Copy Markdown
Contributor

create_qcomflash_pkg installs the ESP image into a directory that exists only so it can be tarred, and never modifies it there, the same situation rootfs.img was in. The copy reads and writes the 512 MiB image once more per image, on a task that is already the heaviest disk writer of the image build, for a file whose only reader is the tar that follows.

Hard link it instead, so the tarball is built from the deployed file and nothing is written.

create_qcomflash_pkg installs the ESP image into a directory that exists
only so it can be tarred, and never modifies it there, the same situation
rootfs.img was in. The copy reads and writes the 512 MiB image once more
per image, on a task that is already the heaviest disk writer of the
image build, for a file whose only reader is the tar that follows.

Hard link it instead, so the tarball is built from the deployed file and
nothing is written.

Assisted-by: Claude Code:claude-fable-5
Signed-off-by: Ricardo Salveti <ricardo.salveti@oss.qualcomm.com>
@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown

Test run workflow

Test jobs for commit 77ff467

qcom-distro
Pass: 323 | Fail: 6 | Total: 357
qcom-distro_linux-qcom-6.18
Pass: 236 | Fail: 0 | Total: 259
nodistro
Pass: 10 | Fail: 0 | Total: 10

@test-reporting-app

test-reporting-app Bot commented Sep 10, 2026

Copy link
Copy Markdown

Test Results

  119 files  +   51    715 suites  +396   8h 38m 19s ⏱️ - 2h 58m 44s
  173 tests +   20    160 ✅ +   47   2 💤  -  1  11 ❌  - 26 
4 607 runs  +2 528  4 542 ✅ +2 548  51 💤 +33  14 ❌  - 53 

For more details on these failures, see this check.

Results for commit 77ff467. ± Comparison against base commit bc1ff3a.

This pull request removes 2 and adds 22 tests. Note that renamed tests count towards both.
lava ‑ auto-login-action
lava ‑ minimal-boot
10_Logging_Journalctl_Validation ‑ Logging_Journalctl_Validation
11_Partition_PostBoot_Validation ‑ Partition_PostBoot_Validation
12_Boot_Systemd_KPI_Loop ‑ Boot_Systemd_KPI_Loop
5_OpenCV ‑ OpenCV
6_Ethernet ‑ Ethernet
6_irq ‑ irq
7_BT_ON_OFF ‑ BT_ON_OFF
7_Docker_Kernel_Config ‑ Docker_Kernel_Config
8_AudioRecord ‑ AudioRecord
8_EFI_Variable_Validation ‑ EFI_Variable_Validation
…

♻️ This comment has been updated with latest results.

@ricardosalveti

Copy link
Copy Markdown
Contributor Author

Known failures.

@ricardosalveti
Ricardo Salveti (ricardosalveti) merged commit 2ce4322 into qualcomm-linux:master Sep 10, 2026
241 of 242 checks passed
@quic-yocto-ci

Copy link
Copy Markdown
Contributor

@qcomlnxci

Copy link
Copy Markdown

Test Coral run workflow

Test jobs for commit 77ff467

  • qcomdistro: multimedia image-prop
    Pass: 41 | Fail: 0 | Total: 41
  • qcomdistro: multimedia image
    Pass: 9 | Fail: 0 | Total: 9

Ricardo Salveti (ricardosalveti) added a commit that referenced this pull request Sep 11, 2026
… qcomflash directory (#3118)

# Description
Backport of #3110 to `wrynose`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants