Re: [PATCH 2/2] dm-integrity: introduce the Inline mode

From: Mikulas Patocka
Date: Tue Sep 03 2024 - 06:15:45 EST




On Mon, 26 Aug 2024, Geert Uytterhoeven wrote:

> Hi Mikulas,
>
> On Tue, 25 Jun 2024, Mikulas Patocka wrote:
> > This commit introduces a new 'I' mode for dm-integrity.
> >
> > The 'I' mode may be selected if the underlying device has non-power-of-2
> > sector size. In this mode, dm-integrity will store integrity data
> > directly in device's sectors and it will not use journal.
> >
> > This mode improves performance and reduces flash wear because there would
> > be no journal writes.
> >
> > Signed-off-by: Mikulas Patocka <mpatocka@xxxxxxxxxx>
>
> Thanks for your patch, which is now commit fb0987682c629c1d
> ("dm-integrity: introduce the Inline mode") in v6.11-rc1.
>
> > @@ -4433,9 +4737,12 @@ static int dm_integrity_ctr(struct dm_ta
> > ti->error = "Block size doesn't match the information in
> > superblock";
> > goto bad;
> > }
> > - if (!le32_to_cpu(ic->sb->journal_sections)) {
> > + if (!le32_to_cpu(ic->sb->journal_sections) != (ic->mode == 'I')) {
>
> As reporting before in e.g. [1], this is causing build failures with
> gcc-5 and -Werror:
>
> drivers/md/dm-integrity.c:4718:52: error: logical not is only applied to
> the left hand side of comparison [-Werror=logical-not-parentheses]
> if (!le32_to_cpu(ic->sb->journal_sections) != (ic->mode == 'I')) {

Hi

I fixed it. The fix will be submitted upstream in the next merge window.

Mikulas