Skip to content

Commit

Permalink
xfs: remove xfs_buf_target_name
Browse files Browse the repository at this point in the history
The calling convention that returns a pointer to a static buffer is
fairly nasty, so just opencode it in the only caller that is left.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Alex Elder <aelder@sgi.com>
  • Loading branch information
Christoph Hellwig authored and Alex Elder committed Oct 12, 2011
1 parent b38505b commit 02b102d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 10 deletions.
6 changes: 5 additions & 1 deletion fs/xfs/xfs_buf.c
Original file line number Diff line number Diff line change
Expand Up @@ -1457,9 +1457,13 @@ xfs_setsize_buftarg_flags(
btp->bt_smask = sectorsize - 1;

if (set_blocksize(btp->bt_bdev, sectorsize)) {
char name[BDEVNAME_SIZE];

bdevname(btp->bt_bdev, name);

xfs_warn(btp->bt_mount,
"Cannot set_blocksize to %u on device %s\n",
sectorsize, xfs_buf_target_name(btp));
sectorsize, name);
return EINVAL;
}

Expand Down
9 changes: 0 additions & 9 deletions fs/xfs/xfs_buf.h
Original file line number Diff line number Diff line change
Expand Up @@ -230,15 +230,6 @@ extern void xfs_buf_delwri_promote(struct xfs_buf *);
extern int xfs_buf_init(void);
extern void xfs_buf_terminate(void);

static inline const char *
xfs_buf_target_name(struct xfs_buftarg *target)
{
static char __b[BDEVNAME_SIZE];

return bdevname(target->bt_bdev, __b);
}


#define XFS_BUF_ZEROFLAGS(bp) \
((bp)->b_flags &= ~(XBF_READ|XBF_WRITE|XBF_ASYNC|XBF_DELWRI| \
XBF_SYNCIO|XBF_FUA|XBF_FLUSH))
Expand Down

0 comments on commit 02b102d

Please sign in to comment.