mirror of
https://github.com/torvalds/linux.git
synced 2026-05-31 10:33:41 +02:00
xfs: remove the mount field from struct xfs_busy_extents
The mount field is only passed to xfs_extent_busy_clear, which never uses it. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Darrick J. Wong <djwong@kernel.org> Signed-off-by: Darrick J. Wong <djwong@kernel.org>
This commit is contained in:
parent
4a137e0915
commit
8dcf5e617f
|
|
@ -81,7 +81,7 @@ xfs_discard_endio_work(
|
|||
struct xfs_busy_extents *extents =
|
||||
container_of(work, struct xfs_busy_extents, endio_work);
|
||||
|
||||
xfs_extent_busy_clear(extents->mount, &extents->extent_list, false);
|
||||
xfs_extent_busy_clear(&extents->extent_list, false);
|
||||
kfree(extents->owner);
|
||||
}
|
||||
|
||||
|
|
@ -301,7 +301,7 @@ xfs_trim_gather_extents(
|
|||
* we aren't going to issue a discard on them any more.
|
||||
*/
|
||||
if (error)
|
||||
xfs_extent_busy_clear(mp, &extents->extent_list, false);
|
||||
xfs_extent_busy_clear(&extents->extent_list, false);
|
||||
out_del_cursor:
|
||||
xfs_btree_del_cursor(cur, error);
|
||||
out_trans_cancel:
|
||||
|
|
@ -347,7 +347,6 @@ xfs_trim_perag_extents(
|
|||
break;
|
||||
}
|
||||
|
||||
extents->mount = pag->pag_mount;
|
||||
extents->owner = extents;
|
||||
INIT_LIST_HEAD(&extents->extent_list);
|
||||
|
||||
|
|
|
|||
|
|
@ -545,7 +545,6 @@ xfs_extent_busy_clear_one(
|
|||
*/
|
||||
void
|
||||
xfs_extent_busy_clear(
|
||||
struct xfs_mount *mp,
|
||||
struct list_head *list,
|
||||
bool do_discard)
|
||||
{
|
||||
|
|
|
|||
|
|
@ -33,7 +33,6 @@ struct xfs_extent_busy {
|
|||
* to discard completion.
|
||||
*/
|
||||
struct xfs_busy_extents {
|
||||
struct xfs_mount *mount;
|
||||
struct list_head extent_list;
|
||||
struct work_struct endio_work;
|
||||
|
||||
|
|
@ -54,8 +53,7 @@ xfs_extent_busy_insert_discard(struct xfs_perag *pag, xfs_agblock_t bno,
|
|||
xfs_extlen_t len, struct list_head *busy_list);
|
||||
|
||||
void
|
||||
xfs_extent_busy_clear(struct xfs_mount *mp, struct list_head *list,
|
||||
bool do_discard);
|
||||
xfs_extent_busy_clear(struct list_head *list, bool do_discard);
|
||||
|
||||
int
|
||||
xfs_extent_busy_search(struct xfs_perag *pag, xfs_agblock_t bno,
|
||||
|
|
|
|||
|
|
@ -907,7 +907,7 @@ xlog_cil_committed(
|
|||
xlog_cil_ail_insert(ctx, abort);
|
||||
|
||||
xfs_extent_busy_sort(&ctx->busy_extents.extent_list);
|
||||
xfs_extent_busy_clear(mp, &ctx->busy_extents.extent_list,
|
||||
xfs_extent_busy_clear(&ctx->busy_extents.extent_list,
|
||||
xfs_has_discard(mp) && !abort);
|
||||
|
||||
spin_lock(&ctx->cil->xc_push_lock);
|
||||
|
|
@ -917,7 +917,6 @@ xlog_cil_committed(
|
|||
xlog_cil_free_logvec(&ctx->lv_chain);
|
||||
|
||||
if (!list_empty(&ctx->busy_extents.extent_list)) {
|
||||
ctx->busy_extents.mount = mp;
|
||||
ctx->busy_extents.owner = ctx;
|
||||
xfs_discard_extents(mp, &ctx->busy_extents);
|
||||
return;
|
||||
|
|
|
|||
|
|
@ -67,7 +67,7 @@ xfs_trans_free(
|
|||
struct xfs_trans *tp)
|
||||
{
|
||||
xfs_extent_busy_sort(&tp->t_busy);
|
||||
xfs_extent_busy_clear(tp->t_mountp, &tp->t_busy, false);
|
||||
xfs_extent_busy_clear(&tp->t_busy, false);
|
||||
|
||||
trace_xfs_trans_free(tp, _RET_IP_);
|
||||
xfs_trans_clear_context(tp);
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user