Re: [f2fs-dev] [PATCH v2] f2fs: merge lz4hc_compress_pages() to lz4_compress_pages()

[Chao Yu]

On 2023/3/31 0:49, Yangtao Li wrote:

+#ifdef CONFIG_F2FS_FS_LZ4HC
+       unsigned char level = F2FS_I(cc->inode)->i_compress_level;

if (level)

                len = LZ4_compress_HC(cc->rbuf, cc->cbuf->cdata, cc->rlen,
                                        cc->clen, level, cc->private);
        else

[snip]

  #endif

[snip]

+               len = LZ4_compress_default(cc->rbuf, cc->cbuf->cdata, cc->rlen,

It's weired that whole else xxx; statement is split by #endif.

What about?

        unsigned char level = 0;

#if
...
#endif
        if (!level)
                len = LZ4_compress_default()

Thanks,

                                                cc->clen, cc->private);
        if (!len)
                return -EAGAIN;

_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel