Re: [PATCH 5/9] btrfs: swtich EXTENT_BUFFER_* to enums
Reviewed-by: Johannes Thumshirn -- Johannes ThumshirnSUSE Labs Filesystems jthumsh...@suse.de+49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
Re: [PATCH 5/9] btrfs: swtich EXTENT_BUFFER_* to enums
On 2018/11/28 上午3:53, David Sterba wrote: > We can use simple enum for values that are not part of on-disk format: > extent buffer flags; > > Signed-off-by: David Sterba Reviewed-by: Qu Wenruo Thanks, Qu > --- > fs/btrfs/extent_io.h | 28 > 1 file changed, 16 insertions(+), 12 deletions(-) > > diff --git a/fs/btrfs/extent_io.h b/fs/btrfs/extent_io.h > index a1d3ea5a0d32..fd42492e62e5 100644 > --- a/fs/btrfs/extent_io.h > +++ b/fs/btrfs/extent_io.h > @@ -37,18 +37,22 @@ > #define EXTENT_BIO_COMPRESSED 1 > #define EXTENT_BIO_FLAG_SHIFT 16 > > -/* these are bit numbers for test/set bit */ > -#define EXTENT_BUFFER_UPTODATE 0 > -#define EXTENT_BUFFER_DIRTY 2 > -#define EXTENT_BUFFER_CORRUPT 3 > -#define EXTENT_BUFFER_READAHEAD 4/* this got triggered by readahead */ > -#define EXTENT_BUFFER_TREE_REF 5 > -#define EXTENT_BUFFER_STALE 6 > -#define EXTENT_BUFFER_WRITEBACK 7 > -#define EXTENT_BUFFER_READ_ERR 8/* read IO error */ > -#define EXTENT_BUFFER_UNMAPPED 9 > -#define EXTENT_BUFFER_IN_TREE 10 > -#define EXTENT_BUFFER_WRITE_ERR 11/* write IO error */ > +enum { > + EXTENT_BUFFER_UPTODATE, > + EXTENT_BUFFER_DIRTY, > + EXTENT_BUFFER_CORRUPT, > + /* this got triggered by readahead */ > + EXTENT_BUFFER_READAHEAD, > + EXTENT_BUFFER_TREE_REF, > + EXTENT_BUFFER_STALE, > + EXTENT_BUFFER_WRITEBACK, > + /* read IO error */ > + EXTENT_BUFFER_READ_ERR, > + EXTENT_BUFFER_UNMAPPED, > + EXTENT_BUFFER_IN_TREE, > + /* write IO error */ > + EXTENT_BUFFER_WRITE_ERR, > +}; > > /* these are flags for __process_pages_contig */ > #define PAGE_UNLOCK (1 << 0) > signature.asc Description: OpenPGP digital signature
Re: [PATCH 5/9] btrfs: swtich EXTENT_BUFFER_* to enums
On Tue, Nov 27, 2018 at 08:53:50PM +0100, David Sterba wrote: > We can use simple enum for values that are not part of on-disk format: > extent buffer flags; This one has a "swtich" typo in the subject. Otherwise, Reviewed-by: Omar Sandoval > Signed-off-by: David Sterba > --- > fs/btrfs/extent_io.h | 28 > 1 file changed, 16 insertions(+), 12 deletions(-) > > diff --git a/fs/btrfs/extent_io.h b/fs/btrfs/extent_io.h > index a1d3ea5a0d32..fd42492e62e5 100644 > --- a/fs/btrfs/extent_io.h > +++ b/fs/btrfs/extent_io.h > @@ -37,18 +37,22 @@ > #define EXTENT_BIO_COMPRESSED 1 > #define EXTENT_BIO_FLAG_SHIFT 16 > > -/* these are bit numbers for test/set bit */ > -#define EXTENT_BUFFER_UPTODATE 0 > -#define EXTENT_BUFFER_DIRTY 2 > -#define EXTENT_BUFFER_CORRUPT 3 > -#define EXTENT_BUFFER_READAHEAD 4/* this got triggered by readahead */ > -#define EXTENT_BUFFER_TREE_REF 5 > -#define EXTENT_BUFFER_STALE 6 > -#define EXTENT_BUFFER_WRITEBACK 7 > -#define EXTENT_BUFFER_READ_ERR 8/* read IO error */ > -#define EXTENT_BUFFER_UNMAPPED 9 > -#define EXTENT_BUFFER_IN_TREE 10 > -#define EXTENT_BUFFER_WRITE_ERR 11/* write IO error */ > +enum { > + EXTENT_BUFFER_UPTODATE, > + EXTENT_BUFFER_DIRTY, > + EXTENT_BUFFER_CORRUPT, > + /* this got triggered by readahead */ > + EXTENT_BUFFER_READAHEAD, > + EXTENT_BUFFER_TREE_REF, > + EXTENT_BUFFER_STALE, > + EXTENT_BUFFER_WRITEBACK, > + /* read IO error */ > + EXTENT_BUFFER_READ_ERR, > + EXTENT_BUFFER_UNMAPPED, > + EXTENT_BUFFER_IN_TREE, > + /* write IO error */ > + EXTENT_BUFFER_WRITE_ERR, > +}; > > /* these are flags for __process_pages_contig */ > #define PAGE_UNLOCK (1 << 0) > -- > 2.19.1 >
[PATCH 5/9] btrfs: swtich EXTENT_BUFFER_* to enums
We can use simple enum for values that are not part of on-disk format: extent buffer flags; Signed-off-by: David Sterba --- fs/btrfs/extent_io.h | 28 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/fs/btrfs/extent_io.h b/fs/btrfs/extent_io.h index a1d3ea5a0d32..fd42492e62e5 100644 --- a/fs/btrfs/extent_io.h +++ b/fs/btrfs/extent_io.h @@ -37,18 +37,22 @@ #define EXTENT_BIO_COMPRESSED 1 #define EXTENT_BIO_FLAG_SHIFT 16 -/* these are bit numbers for test/set bit */ -#define EXTENT_BUFFER_UPTODATE 0 -#define EXTENT_BUFFER_DIRTY 2 -#define EXTENT_BUFFER_CORRUPT 3 -#define EXTENT_BUFFER_READAHEAD 4 /* this got triggered by readahead */ -#define EXTENT_BUFFER_TREE_REF 5 -#define EXTENT_BUFFER_STALE 6 -#define EXTENT_BUFFER_WRITEBACK 7 -#define EXTENT_BUFFER_READ_ERR 8/* read IO error */ -#define EXTENT_BUFFER_UNMAPPED 9 -#define EXTENT_BUFFER_IN_TREE 10 -#define EXTENT_BUFFER_WRITE_ERR 11/* write IO error */ +enum { + EXTENT_BUFFER_UPTODATE, + EXTENT_BUFFER_DIRTY, + EXTENT_BUFFER_CORRUPT, + /* this got triggered by readahead */ + EXTENT_BUFFER_READAHEAD, + EXTENT_BUFFER_TREE_REF, + EXTENT_BUFFER_STALE, + EXTENT_BUFFER_WRITEBACK, + /* read IO error */ + EXTENT_BUFFER_READ_ERR, + EXTENT_BUFFER_UNMAPPED, + EXTENT_BUFFER_IN_TREE, + /* write IO error */ + EXTENT_BUFFER_WRITE_ERR, +}; /* these are flags for __process_pages_contig */ #define PAGE_UNLOCK(1 << 0) -- 2.19.1