Skip to content

Commit e098154

Browse files
Al Virohsiangkao
authored andcommitted
erofs: mechanically convert erofs_read_metabuf() to offsets
just lift the call of erofs_pos() into the callers; it will collapse in most of them, but that's better done caller-by-caller. Signed-off-by: Al Viro <[email protected]> Link: https://lore.kernel.org/r/20240425195846.GC1031757@ZenIV Signed-off-by: Gao Xiang <[email protected]>
1 parent c34110e commit e098154

File tree

7 files changed

+13
-13
lines changed

7 files changed

+13
-13
lines changed

fs/erofs/data.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,10 @@ void erofs_init_metabuf(struct erofs_buf *buf, struct super_block *sb)
7272
}
7373

7474
void *erofs_read_metabuf(struct erofs_buf *buf, struct super_block *sb,
75-
erofs_blk_t blkaddr, enum erofs_kmap_type type)
75+
erofs_off_t offset, enum erofs_kmap_type type)
7676
{
7777
erofs_init_metabuf(buf, sb);
78-
return erofs_bread(buf, erofs_pos(sb, blkaddr), type);
78+
return erofs_bread(buf, offset, type);
7979
}
8080

8181
static int erofs_map_blocks_flatmode(struct inode *inode,
@@ -152,7 +152,7 @@ int erofs_map_blocks(struct inode *inode, struct erofs_map_blocks *map)
152152
pos = ALIGN(erofs_iloc(inode) + vi->inode_isize +
153153
vi->xattr_isize, unit) + unit * chunknr;
154154

155-
kaddr = erofs_read_metabuf(&buf, sb, erofs_blknr(sb, pos), EROFS_KMAP);
155+
kaddr = erofs_read_metabuf(&buf, sb, erofs_pos(sb, erofs_blknr(sb, pos)), EROFS_KMAP);
156156
if (IS_ERR(kaddr)) {
157157
err = PTR_ERR(kaddr);
158158
goto out;
@@ -295,7 +295,7 @@ static int erofs_iomap_begin(struct inode *inode, loff_t offset, loff_t length,
295295

296296
iomap->type = IOMAP_INLINE;
297297
ptr = erofs_read_metabuf(&buf, sb,
298-
erofs_blknr(sb, mdev.m_pa), EROFS_KMAP);
298+
erofs_pos(sb, erofs_blknr(sb, mdev.m_pa)), EROFS_KMAP);
299299
if (IS_ERR(ptr))
300300
return PTR_ERR(ptr);
301301
iomap->inline_data = ptr + erofs_blkoff(sb, mdev.m_pa);

fs/erofs/fscache.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,7 @@ static int erofs_fscache_data_read_slice(struct erofs_fscache_rq *req)
282282
blknr = erofs_blknr(sb, map.m_pa);
283283
size = map.m_llen;
284284

285-
src = erofs_read_metabuf(&buf, sb, blknr, EROFS_KMAP);
285+
src = erofs_read_metabuf(&buf, sb, erofs_pos(sb, blknr), EROFS_KMAP);
286286
if (IS_ERR(src))
287287
return PTR_ERR(src);
288288

fs/erofs/inode.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ static void *erofs_read_inode(struct erofs_buf *buf,
2626
blkaddr = erofs_blknr(sb, inode_loc);
2727
*ofs = erofs_blkoff(sb, inode_loc);
2828

29-
kaddr = erofs_read_metabuf(buf, sb, blkaddr, EROFS_KMAP);
29+
kaddr = erofs_read_metabuf(buf, sb, erofs_pos(sb, blkaddr), EROFS_KMAP);
3030
if (IS_ERR(kaddr)) {
3131
erofs_err(sb, "failed to get inode (nid: %llu) page, err %ld",
3232
vi->nid, PTR_ERR(kaddr));
@@ -66,7 +66,7 @@ static void *erofs_read_inode(struct erofs_buf *buf,
6666
goto err_out;
6767
}
6868
memcpy(copied, dic, gotten);
69-
kaddr = erofs_read_metabuf(buf, sb, blkaddr + 1,
69+
kaddr = erofs_read_metabuf(buf, sb, erofs_pos(sb, blkaddr + 1),
7070
EROFS_KMAP);
7171
if (IS_ERR(kaddr)) {
7272
erofs_err(sb, "failed to get inode payload block (nid: %llu), err %ld",

fs/erofs/internal.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,7 @@ void *erofs_bread(struct erofs_buf *buf, erofs_off_t offset,
403403
enum erofs_kmap_type type);
404404
void erofs_init_metabuf(struct erofs_buf *buf, struct super_block *sb);
405405
void *erofs_read_metabuf(struct erofs_buf *buf, struct super_block *sb,
406-
erofs_blk_t blkaddr, enum erofs_kmap_type type);
406+
erofs_off_t offset, enum erofs_kmap_type type);
407407
int erofs_map_dev(struct super_block *sb, struct erofs_map_dev *dev);
408408
int erofs_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo,
409409
u64 start, u64 len);

fs/erofs/super.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ static int erofs_init_device(struct erofs_buf *buf, struct super_block *sb,
180180
struct file *bdev_file;
181181
void *ptr;
182182

183-
ptr = erofs_read_metabuf(buf, sb, erofs_blknr(sb, *pos), EROFS_KMAP);
183+
ptr = erofs_read_metabuf(buf, sb, erofs_pos(sb, erofs_blknr(sb, *pos)), EROFS_KMAP);
184184
if (IS_ERR(ptr))
185185
return PTR_ERR(ptr);
186186
dis = ptr + erofs_blkoff(sb, *pos);

fs/erofs/zdata.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -868,7 +868,7 @@ static int z_erofs_pcluster_begin(struct z_erofs_decompress_frontend *fe)
868868
} else {
869869
void *mptr;
870870

871-
mptr = erofs_read_metabuf(&map->buf, sb, blknr, EROFS_NO_KMAP);
871+
mptr = erofs_read_metabuf(&map->buf, sb, erofs_pos(sb, blknr), EROFS_NO_KMAP);
872872
if (IS_ERR(mptr)) {
873873
ret = PTR_ERR(mptr);
874874
erofs_err(sb, "failed to get inline data %d", ret);

fs/erofs/zmap.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ static int z_erofs_load_full_lcluster(struct z_erofs_maprecorder *m,
3434
unsigned int advise;
3535

3636
m->kaddr = erofs_read_metabuf(&m->map->buf, inode->i_sb,
37-
erofs_blknr(inode->i_sb, pos), EROFS_KMAP);
37+
erofs_pos(inode->i_sb, erofs_blknr(inode->i_sb, pos)), EROFS_KMAP);
3838
if (IS_ERR(m->kaddr))
3939
return PTR_ERR(m->kaddr);
4040

@@ -256,7 +256,7 @@ static int z_erofs_load_compact_lcluster(struct z_erofs_maprecorder *m,
256256
out:
257257
pos += lcn * (1 << amortizedshift);
258258
m->kaddr = erofs_read_metabuf(&m->map->buf, inode->i_sb,
259-
erofs_blknr(inode->i_sb, pos), EROFS_KMAP);
259+
erofs_pos(inode->i_sb, erofs_blknr(inode->i_sb, pos)), EROFS_KMAP);
260260
if (IS_ERR(m->kaddr))
261261
return PTR_ERR(m->kaddr);
262262
return unpack_compacted_index(m, amortizedshift, pos, lookahead);
@@ -590,7 +590,7 @@ static int z_erofs_fill_inode_lazy(struct inode *inode)
590590
goto out_unlock;
591591

592592
pos = ALIGN(erofs_iloc(inode) + vi->inode_isize + vi->xattr_isize, 8);
593-
kaddr = erofs_read_metabuf(&buf, sb, erofs_blknr(sb, pos), EROFS_KMAP);
593+
kaddr = erofs_read_metabuf(&buf, sb, erofs_pos(sb, erofs_blknr(sb, pos)), EROFS_KMAP);
594594
if (IS_ERR(kaddr)) {
595595
err = PTR_ERR(kaddr);
596596
goto out_unlock;

0 commit comments

Comments
 (0)