From: Eric Biggers <ebiggers@kernel.org>
To: Daniel Vacek <neelx@suse.com>
Cc: Christoph Hellwig <hch@infradead.org>,
fstests@vger.kernel.org, linux-btrfs@vger.kernel.org,
linux-ext4@vger.kernel.org, linux-xfs@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net, zlang@redhat.com,
djwong@kernel.org, David Sterba <dsterba@suse.com>,
linux-fscrypt@vger.kernel.org
Subject: Re: [PATCH 00/12] fstests: introduce fscrypt support for btrfs
Date: Mon, 27 Jul 2026 22:48:47 -0700 [thread overview]
Message-ID: <20260728054847.GB99797@sol> (raw)
In-Reply-To: <CAPjX3Fd1KPswrsNe=BeDpkDwi8CmL9HNKcPScMsgGN16MDyVEQ@mail.gmail.com>
On Tue, Jul 28, 2026 at 07:44:37AM +0200, Daniel Vacek wrote:
> On Tue, 28 Jul 2026 at 05:19, Christoph Hellwig <hch@infradead.org> wrote:
> > On Fri, Jul 24, 2026 at 03:33:16PM +0200, Daniel Vacek wrote:
> > > This is a rebase and cleanup of older patches [0] from Sweet Tea and Josef
> > > fixed to work with the recent kernel and btrfs-progs changes.
> >
> > I'm not sure if you post this just for reference or what the intent here
> > is, especially with a cover letter like this that doesn't have much
> > information.
> >
> > Before we doing anything for the btrfs-specific fscrypt changes in
> > xfstests, those changes should be accepted upstream first.
> >
> > Maybe some thing like splitting tests might make sense without that,
> > but then it should be posted separately.
>
> Hi Christoph. Thanks for looking into this.
>
> Yeah, I could have been more verbose, perhaps also using an RFC tag. I
> wanted to get feedback and a review from a wider audience to see if
> this is a viable direction. This is currently the base for my btrfs
> fscrypt development testing.
> Especially for upstream fscrypt changes, when sending those patches I
> was asked about tests support. I understand to keep fstests OOT as
> long as the kernel part is in development. But at the same time I'd
> like to make fstests complement the kernel part and ensure they are
> ready and available for everyone throughout the process. Kinda to
> "bundle" them all the next iteration.
>
> Would you suggest otherwise?
Please keep sending out the tests along with the kernel patches. But
yes, they aren't for immediate merging and should be tagged with RFC,
and the cover letter should mention that the kernel patches are pending.
Thanks!
- Eric
next prev parent reply other threads:[~2026-07-28 5:50 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-24 13:33 [PATCH 00/12] fstests: introduce fscrypt support for btrfs Daniel Vacek
2026-07-24 13:33 ` [PATCH 01/12] common/encrypt: separate data and inode nonces Daniel Vacek
2026-07-24 13:33 ` [PATCH 02/12] common/encrypt: add btrfs to get_encryption_*nonce Daniel Vacek
2026-07-24 13:33 ` [PATCH 03/12] common/encrypt: add btrfs to get_ciphertext_filename Daniel Vacek
2026-07-24 13:33 ` [PATCH 04/12] common/encrypt: enable making a encrypted btrfs filesystem Daniel Vacek
2026-07-24 13:33 ` [PATCH 05/12] common/verity: explicitly don't allow btrfs encryption Daniel Vacek
2026-07-24 13:33 ` [PATCH 06/12] btrfs: add simple test of reflink of encrypted data Daniel Vacek
2026-07-24 13:33 ` [PATCH 07/12] btrfs: test snapshotting encrypted subvol Daniel Vacek
2026-07-24 13:33 ` [PATCH 08/12] fstests: properly test for v1 encryption policies in encrypt tests Daniel Vacek
2026-07-24 13:33 ` [PATCH 09/12] fstests: split generic/580 into two tests Daniel Vacek
2026-07-24 13:33 ` [PATCH 10/12] fstests: split generic/581 " Daniel Vacek
2026-07-24 13:33 ` [PATCH 11/12] fstests: split generic/613 " Daniel Vacek
2026-07-24 13:33 ` [PATCH 12/12] fscrypt-crypt-util: add support for per extent KDF Daniel Vacek
2026-07-28 3:19 ` [PATCH 00/12] fstests: introduce fscrypt support for btrfs Christoph Hellwig
2026-07-28 5:44 ` Daniel Vacek
2026-07-28 5:48 ` Eric Biggers [this message]
2026-07-28 6:04 ` Christoph Hellwig
2026-07-28 6:07 ` Eric Biggers
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260728054847.GB99797@sol \
--to=ebiggers@kernel.org \
--cc=djwong@kernel.org \
--cc=dsterba@suse.com \
--cc=fstests@vger.kernel.org \
--cc=hch@infradead.org \
--cc=linux-btrfs@vger.kernel.org \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-fscrypt@vger.kernel.org \
--cc=linux-xfs@vger.kernel.org \
--cc=neelx@suse.com \
--cc=zlang@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox