Skip to content

TEST: verify qemu#85 fix catches system-gzip regression (DO NOT MERGE) - #15

Closed
widgetii wants to merge 1 commit into
masterfrom
test/verify-qemu-85-rejects-system-gzip
Closed

TEST: verify qemu#85 fix catches system-gzip regression (DO NOT MERGE)#15
widgetii wants to merge 1 commit into
masterfrom
test/verify-qemu-85-rejects-system-gzip

Conversation

@widgetii

@widgetii widgetii commented May 8, 2026

Copy link
Copy Markdown
Member

Test scaffold — do not merge

Reverts arch/arm/cpu/armv7/gk7205v300/Makefile to use system
gzip (32 KiB window), reproducing the May 8 hardware regression
fixed by #14. The other 3 SoCs are unchanged as controls.

Expected outcome

  • qemu_smoke (gk7205v300) FAILS with "Uncompress Fail!"
    → the qemu#85 fix is correctly enforcing the 8 KiB window.
  • qemu_smoke (gk7202v300, gk7205v200, gk7605v100) all PASS
    → the gate is targeted, not over-strict.

Rollback

After verification, this branch and PR get closed/deleted; no
merge to master.

🤖 Generated with Claude Code

DO NOT MERGE. This is a deliberate revert of the gk7205v300
SoC Makefile to system gzip (32 KiB window) — the exact format
that broke real Goke V4 silicon and that PR #14 fixed.

Expected behavior under qemu-hisilicon master (now includes #86,
the fix for #85): qemu_smoke (gk7205v300) FAILS with "Uncompress
Fail!" on the UART. The other 3 SoCs (gk7202v300, gk7205v200,
gk7605v100) stay untouched as controls — they should still pass.

If the test passes (i.e., gk7205v300 fails as expected and the
others pass), the QEMU gate is now tight enough to have caught
the original PR #10 regression.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@widgetii

widgetii commented May 8, 2026

Copy link
Copy Markdown
Member Author

Test scaffold — qemu#85 reopened with root cause (Z_FINISH bypasses window cap). Closing this branch; will retest once #85 lands properly.

@widgetii widgetii closed this May 8, 2026
@widgetii
widgetii deleted the test/verify-qemu-85-rejects-system-gzip branch May 8, 2026 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants