Skip to content

Commit dd23bf3

Browse files
Matthew Wilcox (Oracle)kleikamp
authored andcommitted
jfs: Convert drop_metapage and remove_metapage to take a folio
All callers now have a folio, so pass it in instead of the page. Removes a couple of calls to compound_head(). Signed-off-by: Matthew Wilcox (Oracle) <[email protected]> Signed-off-by: Dave Kleikamp <[email protected]>
1 parent 40e1bd1 commit dd23bf3

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

fs/jfs/jfs_metapage.c

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -112,10 +112,10 @@ static inline int insert_metapage(struct folio *folio, struct metapage *mp)
112112
return 0;
113113
}
114114

115-
static inline void remove_metapage(struct page *page, struct metapage *mp)
115+
static inline void remove_metapage(struct folio *folio, struct metapage *mp)
116116
{
117-
struct meta_anchor *a = mp_anchor(page);
118-
int l2mp_blocks = L2PSIZE - page->mapping->host->i_blkbits;
117+
struct meta_anchor *a = folio->private;
118+
int l2mp_blocks = L2PSIZE - folio->mapping->host->i_blkbits;
119119
int index;
120120

121121
index = (mp->index >> l2mp_blocks) & (MPS_PER_PAGE - 1);
@@ -125,8 +125,8 @@ static inline void remove_metapage(struct page *page, struct metapage *mp)
125125
a->mp[index] = NULL;
126126
if (--a->mp_count == 0) {
127127
kfree(a);
128-
detach_page_private(page);
129-
kunmap(page);
128+
folio_detach_private(folio);
129+
kunmap(&folio->page);
130130
}
131131
}
132132

@@ -156,10 +156,10 @@ static inline int insert_metapage(struct folio *folio, struct metapage *mp)
156156
return 0;
157157
}
158158

159-
static inline void remove_metapage(struct page *page, struct metapage *mp)
159+
static inline void remove_metapage(struct folio *folio, struct metapage *mp)
160160
{
161-
detach_page_private(page);
162-
kunmap(page);
161+
folio_detach_private(folio);
162+
kunmap(&folio->page);
163163
}
164164

165165
#define inc_io(page) do {} while(0)
@@ -214,12 +214,12 @@ void metapage_exit(void)
214214
kmem_cache_destroy(metapage_cache);
215215
}
216216

217-
static inline void drop_metapage(struct page *page, struct metapage *mp)
217+
static inline void drop_metapage(struct folio *folio, struct metapage *mp)
218218
{
219219
if (mp->count || mp->nohomeok || test_bit(META_dirty, &mp->flag) ||
220220
test_bit(META_io, &mp->flag))
221221
return;
222-
remove_metapage(page, mp);
222+
remove_metapage(folio, mp);
223223
INCREMENT(mpStat.pagefree);
224224
free_metapage(mp);
225225
}
@@ -539,7 +539,7 @@ static bool metapage_release_folio(struct folio *folio, gfp_t gfp_mask)
539539
}
540540
if (mp->lsn)
541541
remove_from_logsync(mp);
542-
remove_metapage(&folio->page, mp);
542+
remove_metapage(folio, mp);
543543
INCREMENT(mpStat.pagefree);
544544
free_metapage(mp);
545545
}
@@ -774,7 +774,7 @@ void release_metapage(struct metapage * mp)
774774
remove_from_logsync(mp);
775775

776776
/* Try to keep metapages from using up too much memory */
777-
drop_metapage(&folio->page, mp);
777+
drop_metapage(folio, mp);
778778

779779
folio_unlock(folio);
780780
folio_put(folio);

0 commit comments

Comments
 (0)