[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] ext2: support EXT4_FEATURE_INCOMPAT_ENCRYPT
From: |
Eric Biggers |
Subject: |
Re: [PATCH] ext2: support EXT4_FEATURE_INCOMPAT_ENCRYPT |
Date: |
Thu, 15 Jun 2017 12:42:18 -0700 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
Hi Vladimir,
On Thu, Jun 15, 2017 at 11:12:32AM +0000, Vladimir 'phcoder' Serbinenko wrote:
> Good. Thank you for doing this. Please don't mark obscure error messages
> for translation (N_). In this patch no string needs translation. Would it
> also be possible to add some tests?
Thanks for the review; I've sent v2 which removes the N_() calls and adds an
ext4_encrypt test to grub-fs-tester.
One caveat is that formatting and mounting a filesystem with '-O encrypt' is
only possible with e2fsprogs v1.43+ and Linux v4.1+. Perhaps those should be
detected as a prerequisite? I don't think there's a good way to detect the
encryption support other than trying it and seeing if it works, though.
Eric