* [Cluster-devel] [PATCH] gfs2: Convert function bh_get to use iomap
@ 2022-02-11 15:50 Bob Peterson
2022-05-17 11:55 ` Andreas Gruenbacher
0 siblings, 1 reply; 2+ messages in thread
From: Bob Peterson @ 2022-02-11 15:50 UTC (permalink / raw)
To: cluster-devel.redhat.com
Before this patch, function bh_get used block_map to figure out the
block it needed to read in from the quota_change file. This patch
changes it to use iomap directly to make it more efficient.
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
---
fs/gfs2/quota.c | 15 +++++++++++----
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/fs/gfs2/quota.c b/fs/gfs2/quota.c
index dc77080a82bb..91bc3affe460 100644
--- a/fs/gfs2/quota.c
+++ b/fs/gfs2/quota.c
@@ -368,8 +368,8 @@ static int bh_get(struct gfs2_quota_data *qd)
struct gfs2_inode *ip = GFS2_I(sdp->sd_qc_inode);
unsigned int block, offset;
struct buffer_head *bh;
+ struct iomap iomap = { };
int error;
- struct buffer_head bh_map = { .b_state = 0, .b_blocknr = 0 };
mutex_lock(&sdp->sd_quota_mutex);
@@ -381,11 +381,18 @@ static int bh_get(struct gfs2_quota_data *qd)
block = qd->qd_slot / sdp->sd_qc_per_block;
offset = qd->qd_slot % sdp->sd_qc_per_block;
- bh_map.b_size = BIT(ip->i_inode.i_blkbits);
- error = gfs2_block_map(&ip->i_inode, block, &bh_map, 0);
+ error = gfs2_iomap_get(sdp->sd_qc_inode,
+ (loff_t)block << sdp->sd_qc_inode->i_blkbits,
+ sdp->sd_sb.sb_bsize, &iomap);
if (error)
goto fail;
- error = gfs2_meta_read(ip->i_gl, bh_map.b_blocknr, DIO_WAIT, 0, &bh);
+ if (iomap.addr == IOMAP_NULL_ADDR) {
+ error = -ENOENT;
+ goto fail;
+ }
+
+ error = gfs2_meta_read(ip->i_gl, iomap.addr >> sdp->sd_qc_inode->i_blkbits,
+ DIO_WAIT, 0, &bh);
if (error)
goto fail;
error = -EIO;
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Cluster-devel] [PATCH] gfs2: Convert function bh_get to use iomap
2022-02-11 15:50 [Cluster-devel] [PATCH] gfs2: Convert function bh_get to use iomap Bob Peterson
@ 2022-05-17 11:55 ` Andreas Gruenbacher
0 siblings, 0 replies; 2+ messages in thread
From: Andreas Gruenbacher @ 2022-05-17 11:55 UTC (permalink / raw)
To: cluster-devel.redhat.com
On Fri, Feb 11, 2022 at 4:51 PM Bob Peterson <rpeterso@redhat.com> wrote:
> Before this patch, function bh_get used block_map to figure out the
> block it needed to read in from the quota_change file. This patch
> changes it to use iomap directly to make it more efficient.
Thanks, I've applied this with minor cleanups.
There are still a few other users of gfs2_block_map around that should
be converted as well.
Andreas
> Signed-off-by: Bob Peterson <rpeterso@redhat.com>
> ---
> fs/gfs2/quota.c | 15 +++++++++++----
> 1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/fs/gfs2/quota.c b/fs/gfs2/quota.c
> index dc77080a82bb..91bc3affe460 100644
> --- a/fs/gfs2/quota.c
> +++ b/fs/gfs2/quota.c
> @@ -368,8 +368,8 @@ static int bh_get(struct gfs2_quota_data *qd)
> struct gfs2_inode *ip = GFS2_I(sdp->sd_qc_inode);
> unsigned int block, offset;
> struct buffer_head *bh;
> + struct iomap iomap = { };
> int error;
> - struct buffer_head bh_map = { .b_state = 0, .b_blocknr = 0 };
>
> mutex_lock(&sdp->sd_quota_mutex);
>
> @@ -381,11 +381,18 @@ static int bh_get(struct gfs2_quota_data *qd)
> block = qd->qd_slot / sdp->sd_qc_per_block;
> offset = qd->qd_slot % sdp->sd_qc_per_block;
>
> - bh_map.b_size = BIT(ip->i_inode.i_blkbits);
> - error = gfs2_block_map(&ip->i_inode, block, &bh_map, 0);
> + error = gfs2_iomap_get(sdp->sd_qc_inode,
> + (loff_t)block << sdp->sd_qc_inode->i_blkbits,
> + sdp->sd_sb.sb_bsize, &iomap);
> if (error)
> goto fail;
> - error = gfs2_meta_read(ip->i_gl, bh_map.b_blocknr, DIO_WAIT, 0, &bh);
> + if (iomap.addr == IOMAP_NULL_ADDR) {
> + error = -ENOENT;
> + goto fail;
> + }
> +
> + error = gfs2_meta_read(ip->i_gl, iomap.addr >> sdp->sd_qc_inode->i_blkbits,
> + DIO_WAIT, 0, &bh);
> if (error)
> goto fail;
> error = -EIO;
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-05-17 11:55 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-11 15:50 [Cluster-devel] [PATCH] gfs2: Convert function bh_get to use iomap Bob Peterson
2022-05-17 11:55 ` Andreas Gruenbacher
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.