Skip to content

overlayfs: verity validation broken since f77f281b6118#1308

Closed
vfsci-bot[bot] wants to merge 1 commit into
vfs.base.cifrom
pw/1088602/vfs.base.ci
Closed

overlayfs: verity validation broken since f77f281b6118#1308
vfsci-bot[bot] wants to merge 1 commit into
vfs.base.cifrom
pw/1088602/vfs.base.ci

Conversation

@vfsci-bot

@vfsci-bot vfsci-bot Bot commented May 1, 2026

Copy link
Copy Markdown

Series: https://patchwork.kernel.org/project/linux-fsdevel/list/?series=1088602
Submitter: Colin Walters
Version: 1
Patches: 1/1
Message-ID: <bf0ba588-002b-4860-848e-e806b193840a@app.fastmail.com>
Base: vfs.base.ci
Lore: https://lore.kernel.org/linux-fsdevel/bf0ba588-002b-4860-848e-e806b193840a@app.fastmail.com


Automated by ml2pr

Hi Christoph & Eric,

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f77f281b6118 broke composefs's usage of overlayfs verity=require, this was reported originally in bootc-dev/bootc#2174

There's some output from an agent run I had in the <details> there, but here's an xfstests patch that passes on without that commit and fails with it.

>From 14231122bfd1e41337e4fb847acbbe038457c32a Mon Sep 17 00:00:00 2001
From: Colin Walters <walters@verbum.org>
Date: Fri, 1 May 2026 09:45:58 -0400
Subject: [PATCH] overlay/118: test fsverity lazy load through metacopy overlay

Reproduces the regression reported at:
bootc-dev/bootc#2174

A recent change in how fsverity state was cached in memory
I think caused inodes not in cache to appear to have
missing verity=require for overlayfs.

This test catches that.

Generated-by: OpenCode (Claude Sonnet 4.5)
Signed-off-by: Colin Walters <walters@verbum.org>
@vfsci-bot

vfsci-bot Bot commented May 15, 2026

Copy link
Copy Markdown
Author

This PR is older than 14 days. Closing automatically. If the series is still relevant, a new version will create a new PR.


Automated by ml2pr

@vfsci-bot vfsci-bot Bot closed this May 15, 2026
@vfsci-bot vfsci-bot Bot deleted the pw/1088602/vfs.base.ci branch May 15, 2026 19:31
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.

1 participant