[Concept,01/10] ext4l: Fix unused variable warnings in extents_status.c

Message ID 20251220232355.845414-2-sjg@u-boot.org
State New
Headers
Series ext4l: Add more ext4 files to the build (part D) |

Commit Message

Simon Glass Dec. 20, 2025, 11:23 p.m. UTC
  From: Simon Glass <simon.glass@canonical.com>

Mark variables as __maybe_unused where they are only used in macro
calls that expand to no-ops in U-Boot's single-threaded environment:
- tree in ext4_es_try_to_merge_left/right
- es_stats in ext4_seq_es_shrinker_info_show
- ei in ext4_clear_inode_es, ext4_remove_pending, ext4_is_pending

Co-developed-by: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 fs/ext4l/extents_status.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
  

Patch

diff --git a/fs/ext4l/extents_status.c b/fs/ext4l/extents_status.c
index 9c303fbe48f..72377fadd70 100644
--- a/fs/ext4l/extents_status.c
+++ b/fs/ext4l/extents_status.c
@@ -599,7 +599,7 @@  static int ext4_es_can_be_merged(struct extent_status *es1,
 static struct extent_status *
 ext4_es_try_to_merge_left(struct inode *inode, struct extent_status *es)
 {
-	struct ext4_es_tree *tree = &EXT4_I(inode)->i_es_tree;
+	struct ext4_es_tree *tree __maybe_unused = &EXT4_I(inode)->i_es_tree;
 	struct extent_status *es1;
 	struct rb_node *node;
 
@@ -623,7 +623,7 @@  ext4_es_try_to_merge_left(struct inode *inode, struct extent_status *es)
 static struct extent_status *
 ext4_es_try_to_merge_right(struct inode *inode, struct extent_status *es)
 {
-	struct ext4_es_tree *tree = &EXT4_I(inode)->i_es_tree;
+	struct ext4_es_tree *tree __maybe_unused = &EXT4_I(inode)->i_es_tree;
 	struct extent_status *es1;
 	struct rb_node *node;
 
@@ -1704,7 +1704,7 @@  static unsigned long ext4_es_scan(struct shrinker *shrink,
 int ext4_seq_es_shrinker_info_show(struct seq_file *seq, void *v)
 {
 	struct ext4_sb_info *sbi = EXT4_SB((struct super_block *) seq->private);
-	struct ext4_es_stats *es_stats = &sbi->s_es_stats;
+	struct ext4_es_stats *es_stats __maybe_unused = &sbi->s_es_stats;
 	struct ext4_inode_info *ei, *max = NULL;
 	unsigned int inode_cnt = 0;
 
@@ -1886,7 +1886,7 @@  static int es_reclaim_extents(struct ext4_inode_info *ei, int *nr_to_scan)
  */
 void ext4_clear_inode_es(struct inode *inode)
 {
-	struct ext4_inode_info *ei = EXT4_I(inode);
+	struct ext4_inode_info *ei __maybe_unused = EXT4_I(inode);
 	struct extent_status *es;
 	struct ext4_es_tree *tree;
 	struct rb_node *node;
@@ -2069,7 +2069,7 @@  static void __remove_pending(struct inode *inode, ext4_lblk_t lblk)
  */
 void ext4_remove_pending(struct inode *inode, ext4_lblk_t lblk)
 {
-	struct ext4_inode_info *ei = EXT4_I(inode);
+	struct ext4_inode_info *ei __maybe_unused = EXT4_I(inode);
 
 	write_lock(&ei->i_es_lock);
 	__remove_pending(inode, lblk);
@@ -2089,7 +2089,7 @@  void ext4_remove_pending(struct inode *inode, ext4_lblk_t lblk)
 bool ext4_is_pending(struct inode *inode, ext4_lblk_t lblk)
 {
 	struct ext4_sb_info *sbi = EXT4_SB(inode->i_sb);
-	struct ext4_inode_info *ei = EXT4_I(inode);
+	struct ext4_inode_info *ei __maybe_unused = EXT4_I(inode);
 	bool ret;
 
 	read_lock(&ei->i_es_lock);