From: Eric Sandeen <sandeen@redhat.com>
To: linux-xfs@vger.kernel.org
Cc: Eric Sandeen <sandeen@redhat.com>
Subject: [PATCH 14/14] xfs: remove unused mp argument from xfs_verify_fileext()
Date: Mon, 14 Sep 2026 15:46:19 -0500 [thread overview]
Message-ID: <20260914205221.1516254-15-sandeen@redhat.com> (raw)
In-Reply-To: <20260914205221.1516254-1-sandeen@redhat.com>
(Follows from last patch)
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---
fs/xfs/libxfs/xfs_bmap.c | 2 +-
fs/xfs/libxfs/xfs_rmap.c | 4 ++--
fs/xfs/libxfs/xfs_types.c | 1 -
fs/xfs/libxfs/xfs_types.h | 3 +--
fs/xfs/scrub/bmap.c | 4 ++--
fs/xfs/scrub/bmap_repair.c | 4 ++--
fs/xfs/xfs_bmap_item.c | 2 +-
fs/xfs/xfs_exchmaps_item.c | 4 ++--
fs/xfs/xfs_rmap_item.c | 2 +-
9 files changed, 12 insertions(+), 14 deletions(-)
diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
index ca147dfaed05..ae91f63455c5 100644
--- a/fs/xfs/libxfs/xfs_bmap.c
+++ b/fs/xfs/libxfs/xfs_bmap.c
@@ -6087,7 +6087,7 @@ xfs_bmap_validate_extent_raw(
int whichfork,
struct xfs_bmbt_irec *irec)
{
- if (!xfs_verify_fileext(mp, irec->br_startoff, irec->br_blockcount))
+ if (!xfs_verify_fileext(irec->br_startoff, irec->br_blockcount))
return __this_address;
if (rtfile && whichfork == XFS_DATA_FORK) {
diff --git a/fs/xfs/libxfs/xfs_rmap.c b/fs/xfs/libxfs/xfs_rmap.c
index 37780a0526d8..14aef87837a9 100644
--- a/fs/xfs/libxfs/xfs_rmap.c
+++ b/fs/xfs/libxfs/xfs_rmap.c
@@ -260,7 +260,7 @@ xfs_rmap_check_irec(
/* Check for a valid fork offset, if applicable. */
if (is_inode && !is_bmbt &&
- !xfs_verify_fileext(mp, irec->rm_offset, irec->rm_blockcount))
+ !xfs_verify_fileext(irec->rm_offset, irec->rm_blockcount))
return __this_address;
return NULL;
@@ -310,7 +310,7 @@ xfs_rtrmap_check_inode_irec(
return __this_address;
if (!xfs_verify_rgbext(rtg, irec->rm_startblock, irec->rm_blockcount))
return __this_address;
- if (!xfs_verify_fileext(mp, irec->rm_offset, irec->rm_blockcount))
+ if (!xfs_verify_fileext(irec->rm_offset, irec->rm_blockcount))
return __this_address;
return NULL;
}
diff --git a/fs/xfs/libxfs/xfs_types.c b/fs/xfs/libxfs/xfs_types.c
index 32e6c0aaf467..f195a04dbf66 100644
--- a/fs/xfs/libxfs/xfs_types.c
+++ b/fs/xfs/libxfs/xfs_types.c
@@ -240,7 +240,6 @@ xfs_verify_fileoff(
/* Check that a range of file block offsets do not exceed the maximum. */
bool
xfs_verify_fileext(
- struct xfs_mount *mp,
xfs_fileoff_t off,
xfs_fileoff_t len)
{
diff --git a/fs/xfs/libxfs/xfs_types.h b/fs/xfs/libxfs/xfs_types.h
index adae81149680..19dd5e7c8b12 100644
--- a/fs/xfs/libxfs/xfs_types.h
+++ b/fs/xfs/libxfs/xfs_types.h
@@ -281,7 +281,6 @@ bool xfs_verify_dablk(xfs_fileoff_t off);
void xfs_icount_range(struct xfs_mount *mp, unsigned long long *min,
unsigned long long *max);
bool xfs_verify_fileoff(xfs_fileoff_t off);
-bool xfs_verify_fileext(struct xfs_mount *mp, xfs_fileoff_t off,
- xfs_fileoff_t len);
+bool xfs_verify_fileext(xfs_fileoff_t off, xfs_fileoff_t len);
#endif /* __XFS_TYPES_H__ */
diff --git a/fs/xfs/scrub/bmap.c b/fs/xfs/scrub/bmap.c
index dff2329649a4..b56a63c73595 100644
--- a/fs/xfs/scrub/bmap.c
+++ b/fs/xfs/scrub/bmap.c
@@ -485,7 +485,7 @@ xchk_bmap_iextent(
xchk_fblock_set_corrupt(info->sc, info->whichfork,
irec->br_startoff);
- if (!xfs_verify_fileext(mp, irec->br_startoff, irec->br_blockcount))
+ if (!xfs_verify_fileext(irec->br_startoff, irec->br_blockcount))
xchk_fblock_set_corrupt(info->sc, info->whichfork,
irec->br_startoff);
@@ -887,7 +887,7 @@ xchk_bmap_iextent_delalloc(
xchk_fblock_set_corrupt(info->sc, info->whichfork,
irec->br_startoff);
- if (!xfs_verify_fileext(mp, irec->br_startoff, irec->br_blockcount))
+ if (!xfs_verify_fileext(irec->br_startoff, irec->br_blockcount))
xchk_fblock_set_corrupt(info->sc, info->whichfork,
irec->br_startoff);
diff --git a/fs/xfs/scrub/bmap_repair.c b/fs/xfs/scrub/bmap_repair.c
index eabffba47776..03af6cb92fcf 100644
--- a/fs/xfs/scrub/bmap_repair.c
+++ b/fs/xfs/scrub/bmap_repair.c
@@ -211,7 +211,7 @@ xrep_bmap_check_fork_rmap(
/* Check the file offset range. */
if (!(rec->rm_flags & XFS_RMAP_BMBT_BLOCK) &&
- !xfs_verify_fileext(sc->mp, rec->rm_offset, rec->rm_blockcount))
+ !xfs_verify_fileext(rec->rm_offset, rec->rm_blockcount))
return -EFSCORRUPTED;
/* No contradictory flags. */
@@ -389,7 +389,7 @@ xrep_bmap_check_rtfork_rmap(
return -EFSCORRUPTED;
/* Check the file offsets and physical extents. */
- if (!xfs_verify_fileext(sc->mp, rec->rm_offset, rec->rm_blockcount))
+ if (!xfs_verify_fileext(rec->rm_offset, rec->rm_blockcount))
return -EFSCORRUPTED;
/* Check that this is within the rtgroup. */
diff --git a/fs/xfs/xfs_bmap_item.c b/fs/xfs/xfs_bmap_item.c
index 89f6e79a955f..782dc24df3a0 100644
--- a/fs/xfs/xfs_bmap_item.c
+++ b/fs/xfs/xfs_bmap_item.c
@@ -442,7 +442,7 @@ xfs_bui_validate(
if (!xfs_verify_ino(mp, map->me_owner))
return false;
- if (!xfs_verify_fileext(mp, map->me_startoff, map->me_len))
+ if (!xfs_verify_fileext(map->me_startoff, map->me_len))
return false;
if (map->me_flags & XFS_BMAP_EXTENT_REALTIME)
diff --git a/fs/xfs/xfs_exchmaps_item.c b/fs/xfs/xfs_exchmaps_item.c
index c3745d33e54e..85955467d249 100644
--- a/fs/xfs/xfs_exchmaps_item.c
+++ b/fs/xfs/xfs_exchmaps_item.c
@@ -341,10 +341,10 @@ xfs_xmi_validate(
!xfs_verify_ino(mp, xlf->xmi_inode2))
return false;
- if (!xfs_verify_fileext(mp, xlf->xmi_startoff1, xlf->xmi_blockcount))
+ if (!xfs_verify_fileext(xlf->xmi_startoff1, xlf->xmi_blockcount))
return false;
- return xfs_verify_fileext(mp, xlf->xmi_startoff2, xlf->xmi_blockcount);
+ return xfs_verify_fileext(xlf->xmi_startoff2, xlf->xmi_blockcount);
}
/*
diff --git a/fs/xfs/xfs_rmap_item.c b/fs/xfs/xfs_rmap_item.c
index 2a3a73a8566d..0066983bc452 100644
--- a/fs/xfs/xfs_rmap_item.c
+++ b/fs/xfs/xfs_rmap_item.c
@@ -494,7 +494,7 @@ xfs_rui_validate_map(
!xfs_verify_ino(mp, map->me_owner))
return false;
- if (!xfs_verify_fileext(mp, map->me_startoff, map->me_len))
+ if (!xfs_verify_fileext(map->me_startoff, map->me_len))
return false;
if (isrt)
--
2.55.0
prev parent reply other threads:[~2026-09-14 20:52 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-14 20:46 [PATCH 0/14] xfs: remove unused arguments from many functions Eric Sandeen
2026-09-14 20:46 ` [PATCH 01/14] xfs: remove unused args argument from xfs_attr_node_lookup() Eric Sandeen
2026-09-14 20:46 ` [PATCH 02/14] xfs: remove unused rsvd argument from xfs_bmap_add_attrfork() Eric Sandeen
2026-09-14 20:46 ` [PATCH 03/14] xfs: remove unused mp and ops arguments from xfbtree_rec_bytes() Eric Sandeen
2026-09-14 20:46 ` [PATCH 04/14] xfs: remove unused mp argument from xfs_exchmaps_check_forks() Eric Sandeen
2026-09-14 20:46 ` [PATCH 05/14] xfs: remove unused mp argument from xfs_inobt_rec_check_count() Eric Sandeen
2026-09-14 20:46 ` [PATCH 06/14] xfs: remove unused mp argument from xfs_parent_finish() Eric Sandeen
2026-09-14 20:46 ` [PATCH 07/14] xfs: remove unused tp argument from xfs_rmap_update_hook() Eric Sandeen
2026-09-14 20:46 ` [PATCH 08/14] xfs: remove unused mp argument from xfs_rtrefcount_broot_space_calc() Eric Sandeen
2026-09-14 20:46 ` [PATCH 09/14] xfs: remove unused mp argument from xfs_rtrefcount_broot_space() Eric Sandeen
2026-09-14 20:46 ` [PATCH 10/14] xfs: remove unused mp argument from xfs_rtrmap_broot_space_calc() Eric Sandeen
2026-09-14 20:46 ` [PATCH 11/14] xfs: remove unused mp argument from xfs_calc_default_atomic_ioend_reservation() Eric Sandeen
2026-09-14 20:46 ` [PATCH 12/14] xfs: remove unused mp argument from xfs_verify_dablk() Eric Sandeen
2026-09-14 20:46 ` [PATCH 13/14] xfs: remove unused mp argument from xfs_verify_fileoff() Eric Sandeen
2026-09-14 20:46 ` Eric Sandeen [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260914205221.1516254-15-sandeen@redhat.com \
--to=sandeen@redhat.com \
--cc=linux-xfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox