Commit f0c519fc authored by Gao Xiang's avatar Gao Xiang Committed by Greg Kroah-Hartman

staging: erofs: rename strange variable names in z_erofs_vle_frontend

Previously, 2 members called `initial' and `cachedzone_la' are used
for applying caching policy (whether the workgroup is at either end),
which are hard to understand, rename them to `backmost' and `headoffset'.
Reviewed-by: default avatarChao Yu <yuchao0@huawei.com>
Signed-off-by: default avatarGao Xiang <gaoxiang25@huawei.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 2d9b5dcd
...@@ -587,10 +587,9 @@ struct z_erofs_vle_frontend { ...@@ -587,10 +587,9 @@ struct z_erofs_vle_frontend {
z_erofs_vle_owned_workgrp_t owned_head; z_erofs_vle_owned_workgrp_t owned_head;
bool initial; /* used for applying cache strategy on the fly */
#if (EROFS_FS_ZIP_CACHE_LVL >= 2) bool backmost;
erofs_off_t cachedzone_la; erofs_off_t headoffset;
#endif
}; };
#define VLE_FRONTEND_INIT(__i) { \ #define VLE_FRONTEND_INIT(__i) { \
...@@ -601,7 +600,7 @@ struct z_erofs_vle_frontend { ...@@ -601,7 +600,7 @@ struct z_erofs_vle_frontend {
}, \ }, \
.builder = VLE_WORK_BUILDER_INIT(), \ .builder = VLE_WORK_BUILDER_INIT(), \
.owned_head = Z_EROFS_VLE_WORKGRP_TAIL, \ .owned_head = Z_EROFS_VLE_WORKGRP_TAIL, \
.initial = true, } .backmost = true, }
static int z_erofs_do_read_page(struct z_erofs_vle_frontend *fe, static int z_erofs_do_read_page(struct z_erofs_vle_frontend *fe,
struct page *page, struct page *page,
...@@ -644,7 +643,7 @@ static int z_erofs_do_read_page(struct z_erofs_vle_frontend *fe, ...@@ -644,7 +643,7 @@ static int z_erofs_do_read_page(struct z_erofs_vle_frontend *fe,
debugln("%s: [out-of-range] pos %llu", __func__, offset + cur); debugln("%s: [out-of-range] pos %llu", __func__, offset + cur);
if (z_erofs_vle_work_iter_end(builder)) if (z_erofs_vle_work_iter_end(builder))
fe->initial = false; fe->backmost = false;
map->m_la = offset + cur; map->m_la = offset + cur;
map->m_llen = 0; map->m_llen = 0;
...@@ -670,8 +669,8 @@ static int z_erofs_do_read_page(struct z_erofs_vle_frontend *fe, ...@@ -670,8 +669,8 @@ static int z_erofs_do_read_page(struct z_erofs_vle_frontend *fe,
erofs_blknr(map->m_pa), erofs_blknr(map->m_pa),
grp->compressed_pages, erofs_blknr(map->m_plen), grp->compressed_pages, erofs_blknr(map->m_plen),
/* compressed page caching selection strategy */ /* compressed page caching selection strategy */
fe->initial | (EROFS_FS_ZIP_CACHE_LVL >= 2 ? fe->backmost | (EROFS_FS_ZIP_CACHE_LVL >= 2 ?
map->m_la < fe->cachedzone_la : 0)); map->m_la < fe->headoffset : 0));
if (noio_outoforder && builder_is_followed(builder)) if (noio_outoforder && builder_is_followed(builder))
builder->role = Z_EROFS_VLE_WORK_PRIMARY; builder->role = Z_EROFS_VLE_WORK_PRIMARY;
...@@ -1317,9 +1316,8 @@ static int z_erofs_vle_normalaccess_readpage(struct file *file, ...@@ -1317,9 +1316,8 @@ static int z_erofs_vle_normalaccess_readpage(struct file *file,
trace_erofs_readpage(page, false); trace_erofs_readpage(page, false);
#if (EROFS_FS_ZIP_CACHE_LVL >= 2) f.headoffset = (erofs_off_t)page->index << PAGE_SHIFT;
f.cachedzone_la = (erofs_off_t)page->index << PAGE_SHIFT;
#endif
err = z_erofs_do_read_page(&f, page, &pagepool); err = z_erofs_do_read_page(&f, page, &pagepool);
(void)z_erofs_vle_work_iter_end(&f.builder); (void)z_erofs_vle_work_iter_end(&f.builder);
...@@ -1355,9 +1353,8 @@ static int z_erofs_vle_normalaccess_readpages(struct file *filp, ...@@ -1355,9 +1353,8 @@ static int z_erofs_vle_normalaccess_readpages(struct file *filp,
trace_erofs_readpages(mapping->host, lru_to_page(pages), trace_erofs_readpages(mapping->host, lru_to_page(pages),
nr_pages, false); nr_pages, false);
#if (EROFS_FS_ZIP_CACHE_LVL >= 2) f.headoffset = (erofs_off_t)lru_to_page(pages)->index << PAGE_SHIFT;
f.cachedzone_la = (erofs_off_t)lru_to_page(pages)->index << PAGE_SHIFT;
#endif
for (; nr_pages; --nr_pages) { for (; nr_pages; --nr_pages) {
struct page *page = lru_to_page(pages); struct page *page = lru_to_page(pages);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment