Re: [PATCH 0/5] zram clean up
Hi Andrew, Could you drop this patchset? I will send updated version based on Sergey with some bug fixes. zram-handle-multiple-pages-attached-bios-bvec.patch zram-partial-io-refactoring.patch zram-use-zram_slot_lock-instead-of-raw-bit_spin_lock-op.patch zram-remove-zram_meta-structure.patch zram-introduce-zram-data-accessor.patch Thanks! On Mon, Apr 03, 2017 at 02:17:28PM +0900, Minchan Kim wrote: > This patchset aims zram clean-up. > > [1] clean up multiple pages's bvec handling. > [2] clean up partial IO handling > [3-5] clean up zram via using accessor and removing pointless structure. > > With [2-5] applied, we can get a few hundred bytes as well as huge > readibility enhance. > > This patchset is based on v4.11-rc4-mmotm-2017-03-30-16-31 and > *drop* zram-factor-out-partial-io-routine.patch. > > x86: 708 byte save > > add/remove: 1/1 grow/shrink: 0/11 up/down: 478/-1186 (-708) > function old new delta > zram_special_page_read - 478+478 > zram_reset_device317 314 -3 > mem_used_max_store 131 128 -3 > compact_store 96 93 -3 > mm_stat_show 203 197 -6 > zram_add 719 712 -7 > zram_slot_free_notify229 214 -15 > zram_make_request819 803 -16 > zram_meta_free 128 111 -17 > zram_free_page 180 151 -29 > disksize_store 432 361 -71 > zram_decompress_page.isra504 --504 > zram_bvec_rw25922080-512 > Total: Before=25350773, After=25350065, chg -0.00% > > ppc64: 231 byte save > > add/remove: 2/0 grow/shrink: 1/9 up/down: 681/-912 (-231) > function old new delta > zram_special_page_read - 480+480 > zram_slot_lock - 200+200 > vermagic 39 40 +1 > mm_stat_show 256 248 -8 > zram_meta_free 200 184 -16 > zram_add 944 912 -32 > zram_free_page 348 308 -40 > disksize_store 572 492 -80 > zram_decompress_page 664 564-100 > zram_slot_free_notify292 160-132 > zram_make_request 11321000-132 > zram_bvec_rw27682396-372 > Total: Before=17565825, After=17565594, chg -0.00% > > Minchan Kim (5): > [1] zram: handle multiple pages attached bio's bvec > [2] zram: partial IO refactoring > [3] zram: use zram_slot_lock instead of raw bit_spin_lock op > [4] zram: remove zram_meta structure > [5] zram: introduce zram data accessor > > drivers/block/zram/zram_drv.c | 532 > +- > drivers/block/zram/zram_drv.h | 6 +- > 2 files changed, 270 insertions(+), 268 deletions(-) > > -- > 2.7.4 >
Re: [PATCH 0/5] zram clean up
Hi Andrew, Could you drop this patchset? I will send updated version based on Sergey with some bug fixes. zram-handle-multiple-pages-attached-bios-bvec.patch zram-partial-io-refactoring.patch zram-use-zram_slot_lock-instead-of-raw-bit_spin_lock-op.patch zram-remove-zram_meta-structure.patch zram-introduce-zram-data-accessor.patch Thanks! On Mon, Apr 03, 2017 at 02:17:28PM +0900, Minchan Kim wrote: > This patchset aims zram clean-up. > > [1] clean up multiple pages's bvec handling. > [2] clean up partial IO handling > [3-5] clean up zram via using accessor and removing pointless structure. > > With [2-5] applied, we can get a few hundred bytes as well as huge > readibility enhance. > > This patchset is based on v4.11-rc4-mmotm-2017-03-30-16-31 and > *drop* zram-factor-out-partial-io-routine.patch. > > x86: 708 byte save > > add/remove: 1/1 grow/shrink: 0/11 up/down: 478/-1186 (-708) > function old new delta > zram_special_page_read - 478+478 > zram_reset_device317 314 -3 > mem_used_max_store 131 128 -3 > compact_store 96 93 -3 > mm_stat_show 203 197 -6 > zram_add 719 712 -7 > zram_slot_free_notify229 214 -15 > zram_make_request819 803 -16 > zram_meta_free 128 111 -17 > zram_free_page 180 151 -29 > disksize_store 432 361 -71 > zram_decompress_page.isra504 --504 > zram_bvec_rw25922080-512 > Total: Before=25350773, After=25350065, chg -0.00% > > ppc64: 231 byte save > > add/remove: 2/0 grow/shrink: 1/9 up/down: 681/-912 (-231) > function old new delta > zram_special_page_read - 480+480 > zram_slot_lock - 200+200 > vermagic 39 40 +1 > mm_stat_show 256 248 -8 > zram_meta_free 200 184 -16 > zram_add 944 912 -32 > zram_free_page 348 308 -40 > disksize_store 572 492 -80 > zram_decompress_page 664 564-100 > zram_slot_free_notify292 160-132 > zram_make_request 11321000-132 > zram_bvec_rw27682396-372 > Total: Before=17565825, After=17565594, chg -0.00% > > Minchan Kim (5): > [1] zram: handle multiple pages attached bio's bvec > [2] zram: partial IO refactoring > [3] zram: use zram_slot_lock instead of raw bit_spin_lock op > [4] zram: remove zram_meta structure > [5] zram: introduce zram data accessor > > drivers/block/zram/zram_drv.c | 532 > +- > drivers/block/zram/zram_drv.h | 6 +- > 2 files changed, 270 insertions(+), 268 deletions(-) > > -- > 2.7.4 >
[PATCH 0/5] zram clean up
This patchset aims zram clean-up. [1] clean up multiple pages's bvec handling. [2] clean up partial IO handling [3-5] clean up zram via using accessor and removing pointless structure. With [2-5] applied, we can get a few hundred bytes as well as huge readibility enhance. This patchset is based on v4.11-rc4-mmotm-2017-03-30-16-31 and *drop* zram-factor-out-partial-io-routine.patch. x86: 708 byte save add/remove: 1/1 grow/shrink: 0/11 up/down: 478/-1186 (-708) function old new delta zram_special_page_read - 478+478 zram_reset_device317 314 -3 mem_used_max_store 131 128 -3 compact_store 96 93 -3 mm_stat_show 203 197 -6 zram_add 719 712 -7 zram_slot_free_notify229 214 -15 zram_make_request819 803 -16 zram_meta_free 128 111 -17 zram_free_page 180 151 -29 disksize_store 432 361 -71 zram_decompress_page.isra504 --504 zram_bvec_rw25922080-512 Total: Before=25350773, After=25350065, chg -0.00% ppc64: 231 byte save add/remove: 2/0 grow/shrink: 1/9 up/down: 681/-912 (-231) function old new delta zram_special_page_read - 480+480 zram_slot_lock - 200+200 vermagic 39 40 +1 mm_stat_show 256 248 -8 zram_meta_free 200 184 -16 zram_add 944 912 -32 zram_free_page 348 308 -40 disksize_store 572 492 -80 zram_decompress_page 664 564-100 zram_slot_free_notify292 160-132 zram_make_request 11321000-132 zram_bvec_rw27682396-372 Total: Before=17565825, After=17565594, chg -0.00% Minchan Kim (5): [1] zram: handle multiple pages attached bio's bvec [2] zram: partial IO refactoring [3] zram: use zram_slot_lock instead of raw bit_spin_lock op [4] zram: remove zram_meta structure [5] zram: introduce zram data accessor drivers/block/zram/zram_drv.c | 532 +- drivers/block/zram/zram_drv.h | 6 +- 2 files changed, 270 insertions(+), 268 deletions(-) -- 2.7.4
[PATCH 0/5] zram clean up
This patchset aims zram clean-up. [1] clean up multiple pages's bvec handling. [2] clean up partial IO handling [3-5] clean up zram via using accessor and removing pointless structure. With [2-5] applied, we can get a few hundred bytes as well as huge readibility enhance. This patchset is based on v4.11-rc4-mmotm-2017-03-30-16-31 and *drop* zram-factor-out-partial-io-routine.patch. x86: 708 byte save add/remove: 1/1 grow/shrink: 0/11 up/down: 478/-1186 (-708) function old new delta zram_special_page_read - 478+478 zram_reset_device317 314 -3 mem_used_max_store 131 128 -3 compact_store 96 93 -3 mm_stat_show 203 197 -6 zram_add 719 712 -7 zram_slot_free_notify229 214 -15 zram_make_request819 803 -16 zram_meta_free 128 111 -17 zram_free_page 180 151 -29 disksize_store 432 361 -71 zram_decompress_page.isra504 --504 zram_bvec_rw25922080-512 Total: Before=25350773, After=25350065, chg -0.00% ppc64: 231 byte save add/remove: 2/0 grow/shrink: 1/9 up/down: 681/-912 (-231) function old new delta zram_special_page_read - 480+480 zram_slot_lock - 200+200 vermagic 39 40 +1 mm_stat_show 256 248 -8 zram_meta_free 200 184 -16 zram_add 944 912 -32 zram_free_page 348 308 -40 disksize_store 572 492 -80 zram_decompress_page 664 564-100 zram_slot_free_notify292 160-132 zram_make_request 11321000-132 zram_bvec_rw27682396-372 Total: Before=17565825, After=17565594, chg -0.00% Minchan Kim (5): [1] zram: handle multiple pages attached bio's bvec [2] zram: partial IO refactoring [3] zram: use zram_slot_lock instead of raw bit_spin_lock op [4] zram: remove zram_meta structure [5] zram: introduce zram data accessor drivers/block/zram/zram_drv.c | 532 +- drivers/block/zram/zram_drv.h | 6 +- 2 files changed, 270 insertions(+), 268 deletions(-) -- 2.7.4