diff mbox

[v8,3/6] xfs: add missing include dependencies to xfs_dir2.h

Message ID 1484123050-11064-4-git-send-email-amir73il@gmail.com (mailing list archive)
State Superseded, archived
Headers show

Commit Message

Amir Goldstein Jan. 11, 2017, 8:24 a.m. UTC
xfs_dir2.h dereferences some data types in inline functions
and fails to include those type definitions, e.g.:
xfs_dir2_data_aoff_t, struct xfs_da_geometry.

Signed-off-by: Amir Goldstein <amir73il@gmail.com>
---
 fs/xfs/libxfs/xfs_dir2.h | 3 +++
 1 file changed, 3 insertions(+)

Comments

Christoph Hellwig Jan. 11, 2017, 8:38 a.m. UTC | #1
On Wed, Jan 11, 2017 at 10:24:07AM +0200, Amir Goldstein wrote:
> xfs_dir2.h dereferences some data types in inline functions
> and fails to include those type definitions, e.g.:
> xfs_dir2_data_aoff_t, struct xfs_da_geometry.
> 
> Signed-off-by: Amir Goldstein <amir73il@gmail.com>

I'd normally fold this into the patch that actually exposes the
dependency.  But except for that it looks fine to me:

Reviewed-by: Christoph Hellwig <hch@lst.de>
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/fs/xfs/libxfs/xfs_dir2.h b/fs/xfs/libxfs/xfs_dir2.h
index 0197590..72df0dc 100644
--- a/fs/xfs/libxfs/xfs_dir2.h
+++ b/fs/xfs/libxfs/xfs_dir2.h
@@ -18,6 +18,9 @@ 
 #ifndef __XFS_DIR2_H__
 #define __XFS_DIR2_H__
 
+#include "xfs_da_format.h"
+#include "xfs_da_btree.h"
+
 struct xfs_defer_ops;
 struct xfs_da_args;
 struct xfs_inode;