Re: [PATCH] zram: revive swap_slot_free_notify

From: Minchan Kim
Date: Mon Mar 21 2016 - 19:51:04 EST


Hello Andrew,

On Mon, Mar 21, 2016 at 01:30:17PM -0700, Andrew Morton wrote:
> On Fri, 18 Mar 2016 16:58:31 +0900 Minchan Kim <minchan@xxxxxxxxxx> wrote:
>
> > <b430e9d1c6d4> "remove compressed copy from zram in-memory"
> > applied swap_slot_free_notify call in *end_swap_bio_read* to
> > remove duplicated memory between zram and memory.
> >
> > However, with introducing rw_page in zram <8c7f01025f7b>
> > "zram: implement rw_page operation of zram", it became void
> > because rw_page doesn't need bio.
> >
> > This patch restores the function for rw_page.
>
> This is a bit mysterious. What is the actual runtime effect of the
> patch? I assume that 8c7f01025f7b caused duplication of memory and
> that the only problem is additional resource consumption? If so, what
> are the observable effects? etcetera, please.
>

As you already pointed out, only problem is memory duplication.
In b430e9d1c6d4 description, I wrote down how it saves memory(e.g,
about 19M in kernel build workload).
Memory footprint is really important in embedded platforms which
have small memory, for example, 512M) recently because it could
start to kill processes if memory footprint exceeds some threshold
by LMK or some similar memory management modules.