On 2025/1/16 8:41, Sergey Senozhatsky wrote: > On (25/01/15 22:55), Wenchao Hao wrote: >> @@ -1561,11 +1561,6 @@ static int zram_read_from_zspool(struct zram *zram, struct page *page, >> >> size = zram_get_obj_size(zram, index); >> >> - if (size != PAGE_SIZE) { >> - prio = zram_get_priority(zram, index); >> - zstrm = zcomp_stream_get(zram->comps[prio]); >> - } >> - >> src = zs_map_object(zram->mem_pool, handle, ZS_MM_RO); >> if (size == PAGE_SIZE) { >> dst = kmap_local_page(page); >> @@ -1573,6 +1568,9 @@ static int zram_read_from_zspool(struct zram *zram, struct page *page, >> kunmap_local(dst); >> ret = 0; >> } else { >> + prio = zram_get_priority(zram, index); >> + zstrm = zcomp_stream_get(zram->comps[prio]); >> + >> dst = kmap_local_page(page); >> ret = zcomp_decompress(zram->comps[prio], zstrm, >> src, size, dst); > > I think you are looking at some old code base (or maybe vanilla kernel), > that function does not look like this in the current mm tree (or linux-next). Yes, it's from https://github.com/torvalds/linux. Sorry I did not examine the latest code, please ignore this thread. Thanks.