aboutsummaryrefslogtreecommitdiffstats
path: root/fs/xfs
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@lst.de>2018-10-18 02:17:50 -0400
committerDave Chinner <david@fromorbit.com>2018-10-18 02:17:50 -0400
commit97e5a6e6dc44b9ea660f85de084f6e38cb5cf39c (patch)
tree4142e678a7dddab3427d1192b327bd6fe0b37335 /fs/xfs
parentb39989009bdb84992915c9869f58094ed5becf10 (diff)
xfs: remove XFS_IO_INVALID
The invalid state isn't any different from a hole, so merge the two states. Use the more descriptive hole name, but keep it as the first value of the enum to catch uninitialized fields. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Brian Foster <bfoster@redhat.com> Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs')
-rw-r--r--fs/xfs/xfs_aops.c4
-rw-r--r--fs/xfs/xfs_aops.h14
2 files changed, 8 insertions, 10 deletions
diff --git a/fs/xfs/xfs_aops.c b/fs/xfs/xfs_aops.c
index 49f5f5896a43..338b9d9984e0 100644
--- a/fs/xfs/xfs_aops.c
+++ b/fs/xfs/xfs_aops.c
@@ -917,7 +917,7 @@ xfs_vm_writepage(
917 struct writeback_control *wbc) 917 struct writeback_control *wbc)
918{ 918{
919 struct xfs_writepage_ctx wpc = { 919 struct xfs_writepage_ctx wpc = {
920 .io_type = XFS_IO_INVALID, 920 .io_type = XFS_IO_HOLE,
921 }; 921 };
922 int ret; 922 int ret;
923 923
@@ -933,7 +933,7 @@ xfs_vm_writepages(
933 struct writeback_control *wbc) 933 struct writeback_control *wbc)
934{ 934{
935 struct xfs_writepage_ctx wpc = { 935 struct xfs_writepage_ctx wpc = {
936 .io_type = XFS_IO_INVALID, 936 .io_type = XFS_IO_HOLE,
937 }; 937 };
938 int ret; 938 int ret;
939 939
diff --git a/fs/xfs/xfs_aops.h b/fs/xfs/xfs_aops.h
index 9af867951a10..494b4338446e 100644
--- a/fs/xfs/xfs_aops.h
+++ b/fs/xfs/xfs_aops.h
@@ -12,21 +12,19 @@ extern struct bio_set xfs_ioend_bioset;
12 * Types of I/O for bmap clustering and I/O completion tracking. 12 * Types of I/O for bmap clustering and I/O completion tracking.
13 */ 13 */
14enum { 14enum {
15 XFS_IO_INVALID, /* initial state */ 15 XFS_IO_HOLE, /* covers region without any block allocation */
16 XFS_IO_DELALLOC, /* covers delalloc region */ 16 XFS_IO_DELALLOC, /* covers delalloc region */
17 XFS_IO_UNWRITTEN, /* covers allocated but uninitialized data */ 17 XFS_IO_UNWRITTEN, /* covers allocated but uninitialized data */
18 XFS_IO_OVERWRITE, /* covers already allocated extent */ 18 XFS_IO_OVERWRITE, /* covers already allocated extent */
19 XFS_IO_COW, /* covers copy-on-write extent */ 19 XFS_IO_COW, /* covers copy-on-write extent */
20 XFS_IO_HOLE, /* covers region without any block allocation */
21}; 20};
22 21
23#define XFS_IO_TYPES \ 22#define XFS_IO_TYPES \
24 { XFS_IO_INVALID, "invalid" }, \ 23 { XFS_IO_HOLE, "hole" }, \
25 { XFS_IO_DELALLOC, "delalloc" }, \ 24 { XFS_IO_DELALLOC, "delalloc" }, \
26 { XFS_IO_UNWRITTEN, "unwritten" }, \ 25 { XFS_IO_UNWRITTEN, "unwritten" }, \
27 { XFS_IO_OVERWRITE, "overwrite" }, \ 26 { XFS_IO_OVERWRITE, "overwrite" }, \
28 { XFS_IO_COW, "CoW" }, \ 27 { XFS_IO_COW, "CoW" }
29 { XFS_IO_HOLE, "hole" }
30 28
31/* 29/*
32 * Structure for buffered I/O completions. 30 * Structure for buffered I/O completions.