[1/3] xfsprogs: include headers to fix sparse warnings about statics

Commit Message

Eric Sandeen Dec. 18, 2019, 10:54 p.m. UTC

From: Eric Sandeen <[e-mail]>

Addresses many "foo was not declared. Should it be static?"
warnings from sparse.

Signed-off-by: Eric Sandeen <[e-mail]>
---

Comments

Darrick J. Wong Dec. 18, 2019, 11:10 p.m. UTC | #1

On Wed, Dec 18, 2019 at 04:54:46PM -0600, Eric Sandeen wrote:
> From: Eric Sandeen <[e-mail]>
> 
> Addresses many "foo was not declared. Should it be static?"
> warnings from sparse.
> 
> Signed-off-by: Eric Sandeen <[e-mail]>
> ---
> 
> diff --git a/libfrog/linux.c b/libfrog/linux.c
> index 79bd79eb..d7264108 100644
> --- a/libfrog/linux.c
> +++ b/libfrog/linux.c
> @@ -9,6 +9,7 @@
>  #include <sys/ioctl.h>
>  #include <sys/sysinfo.h>
>  
> +#include "libfrog/platform.h"
>  #include "libxfs_priv.h"

libfrog code shouldn't depend on anything in libxfs/ but I'll send my
own patch to fix that.

Looks ok otherwise
Reviewed-by: Darrick J. Wong <[e-mail]>

--D


>  #include "xfs_fs.h"
>  #include "init.h"
> diff --git a/libxfs/util.c b/libxfs/util.c
> index 885dd42b..cd303341 100644
> --- a/libxfs/util.c
> +++ b/libxfs/util.c
> @@ -28,6 +28,7 @@
>  #include "xfs_da_format.h"
>  #include "xfs_da_btree.h"
>  #include "xfs_dir2_priv.h"
> +#include "xfs_health.h"
>  
>  /*
>   * Calculate the worst case log unit reservation for a given superblock
> diff --git a/libxfs/xfs_ag_resv.c b/libxfs/xfs_ag_resv.c
> index 1328e5de..1fe13bf4 100644
> --- a/libxfs/xfs_ag_resv.c
> +++ b/libxfs/xfs_ag_resv.c
> @@ -15,6 +15,8 @@
>  #include "xfs_btree.h"
>  #include "xfs_refcount_btree.h"
>  #include "xfs_ialloc_btree.h"
> +#include "xfs_sb.h"
> +#include "xfs_ag_resv.h"
>  
>  /*
>   * Per-AG Block Reservations
> diff --git a/libxfs/xfs_attr_remote.c b/libxfs/xfs_attr_remote.c
> index 7234f86c..1ce6359a 100644
> --- a/libxfs/xfs_attr_remote.c
> +++ b/libxfs/xfs_attr_remote.c
> @@ -18,6 +18,7 @@
>  #include "xfs_inode.h"
>  #include "xfs_trans.h"
>  #include "xfs_bmap.h"
> +#include "xfs_attr_remote.h"
>  #include "xfs_trace.h"
>  
>  #define ATTR_RMTVALUE_MAPSIZE	1	/* # of map entries at once */
> diff --git a/libxfs/xfs_bit.c b/libxfs/xfs_bit.c
> index 6a76a5ad..3f97fa3e 100644
> --- a/libxfs/xfs_bit.c
> +++ b/libxfs/xfs_bit.c
> @@ -5,6 +5,7 @@
>   */
>  #include "libxfs_priv.h"
>  #include "xfs_log_format.h"
> +#include "xfs_bit.h"
>  
>  /*
>   * XFS bit manipulation routines, used in non-realtime code.
> diff --git a/libxfs/xfs_dir2_data.c b/libxfs/xfs_dir2_data.c
> index 68da426e..044f1272 100644
> --- a/libxfs/xfs_dir2_data.c
> +++ b/libxfs/xfs_dir2_data.c
> @@ -13,6 +13,7 @@
>  #include "xfs_mount.h"
>  #include "xfs_inode.h"
>  #include "xfs_dir2.h"
> +#include "xfs_dir2_priv.h"
>  #include "xfs_trans.h"
>  
>  static xfs_failaddr_t xfs_dir2_data_freefind_verify(
> diff --git a/libxfs/xfs_sb.c b/libxfs/xfs_sb.c
> index 583d29aa..4f750d19 100644
> --- a/libxfs/xfs_sb.c
> +++ b/libxfs/xfs_sb.c
> @@ -10,6 +10,7 @@
>  #include "xfs_log_format.h"
>  #include "xfs_trans_resv.h"
>  #include "xfs_bit.h"
> +#include "xfs_sb.h"
>  #include "xfs_mount.h"
>  #include "xfs_ialloc.h"
>  #include "xfs_alloc.h"
>

Patch

diff --git a/libfrog/linux.c b/libfrog/linux.c
index 79bd79eb..d7264108 100644
--- a/libfrog/linux.c
+++ b/libfrog/linux.c
@@ -9,6 +9,7 @@ 
 #include <sys/ioctl.h>
 #include <sys/sysinfo.h>
 
+#include "libfrog/platform.h"
 #include "libxfs_priv.h"
 #include "xfs_fs.h"
 #include "init.h"
diff --git a/libxfs/util.c b/libxfs/util.c
index 885dd42b..cd303341 100644
--- a/libxfs/util.c
+++ b/libxfs/util.c
@@ -28,6 +28,7 @@ 
 #include "xfs_da_format.h"
 #include "xfs_da_btree.h"
 #include "xfs_dir2_priv.h"
+#include "xfs_health.h"
 
 /*
  * Calculate the worst case log unit reservation for a given superblock
diff --git a/libxfs/xfs_ag_resv.c b/libxfs/xfs_ag_resv.c
index 1328e5de..1fe13bf4 100644
--- a/libxfs/xfs_ag_resv.c
+++ b/libxfs/xfs_ag_resv.c
@@ -15,6 +15,8 @@ 
 #include "xfs_btree.h"
 #include "xfs_refcount_btree.h"
 #include "xfs_ialloc_btree.h"
+#include "xfs_sb.h"
+#include "xfs_ag_resv.h"
 
 /*
  * Per-AG Block Reservations
diff --git a/libxfs/xfs_attr_remote.c b/libxfs/xfs_attr_remote.c
index 7234f86c..1ce6359a 100644
--- a/libxfs/xfs_attr_remote.c
+++ b/libxfs/xfs_attr_remote.c
@@ -18,6 +18,7 @@ 
 #include "xfs_inode.h"
 #include "xfs_trans.h"
 #include "xfs_bmap.h"
+#include "xfs_attr_remote.h"
 #include "xfs_trace.h"
 
 #define ATTR_RMTVALUE_MAPSIZE	1	/* # of map entries at once */
diff --git a/libxfs/xfs_bit.c b/libxfs/xfs_bit.c
index 6a76a5ad..3f97fa3e 100644
--- a/libxfs/xfs_bit.c
+++ b/libxfs/xfs_bit.c
@@ -5,6 +5,7 @@ 
  */
 #include "libxfs_priv.h"
 #include "xfs_log_format.h"
+#include "xfs_bit.h"
 
 /*
  * XFS bit manipulation routines, used in non-realtime code.
diff --git a/libxfs/xfs_dir2_data.c b/libxfs/xfs_dir2_data.c
index 68da426e..044f1272 100644
--- a/libxfs/xfs_dir2_data.c
+++ b/libxfs/xfs_dir2_data.c
@@ -13,6 +13,7 @@ 
 #include "xfs_mount.h"
 #include "xfs_inode.h"
 #include "xfs_dir2.h"
+#include "xfs_dir2_priv.h"
 #include "xfs_trans.h"
 
 static xfs_failaddr_t xfs_dir2_data_freefind_verify(
diff --git a/libxfs/xfs_sb.c b/libxfs/xfs_sb.c
index 583d29aa..4f750d19 100644
--- a/libxfs/xfs_sb.c
+++ b/libxfs/xfs_sb.c
@@ -10,6 +10,7 @@ 
 #include "xfs_log_format.h"
 #include "xfs_trans_resv.h"
 #include "xfs_bit.h"
+#include "xfs_sb.h"
 #include "xfs_mount.h"
 #include "xfs_ialloc.h"
 #include "xfs_alloc.h"