grub-devel
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]