On Fri, Dec 01, 2017 at 05:55:56AM +0800, Anand Jain wrote:
>
>
> On 12/01/2017 12:09 AM, David Sterba wrote:
> > There are several places opencoding this conversion, add a helper now
> > that we have 3 compression algorithms.
> >
> > Signed-off-by: David Sterba <dsterba@xxxxxxxx>
> > ---
> > fs/btrfs/compression.c | 15 +++++++++++++++
> > fs/btrfs/compression.h | 2 ++
> > 2 files changed, 17 insertions(+)
> >
> > diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c
> > index 5982c8a71f02..a1a5958ff328 100644
> > --- a/fs/btrfs/compression.c
> > +++ b/fs/btrfs/compression.c
> > @@ -45,6 +45,21 @@
> > #include "extent_io.h"
> > #include "extent_map.h"
> >
> > +static const char* const btrfs_compress_types[] = { "", "lzo", "zlib", "zstd" };
>
> This is wrong order.
>
> compression.h: BTRFS_COMPRESS_NONE = 0,
> compression.h: BTRFS_COMPRESS_ZLIB = 1,
> compression.h: BTRFS_COMPRESS_LZO = 2,
> compression.h: BTRFS_COMPRESS_ZSTD = 3,
> compression.h: BTRFS_COMPRESS_TYPES = 3,
This was unintentionally left as an exercise for reviewers. Will fix,
thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
