Re: [PATCH v6 7/7] zsmalloc: use shrinker to trigger auto-compaction

From: Minchan Kim
Date: Tue Jul 07 2015 - 11:02:15 EST


On Tue, Jul 07, 2015 at 11:41:07PM +0900, Sergey Senozhatsky wrote:
> On (07/07/15 22:44), Minchan Kim wrote:
> [..]
> > IMO, there is no value to maintain just in case of
> > failing register_shrinker in practice.
> >
> > Let's remove shrinker_enabled and abort pool creation if shrinker register
> > is failed.
>
> But why would we do this? Yes, it's kinda-sorta bad -- we were not
> able to register zspool shrinker, so there will be no automatic
> compaction... And that's it.
>
> It does not affect zsmalloc/zram functionality by any means. Including
> compaction itself -- user still has a way to compact zspool (manually).
> And in some scenarios user will never even see automatic compaction in
> action (assuming that there is a plenty of RAM available).
>
> Can you explain your decision?

I don't think it would fail in *real practice*.
Althout it might happen, what does zram could help in that cases?

If it were failed, it means there is already little memory on the system
so zram could not be helpful for those environment.
IOW, zram should be enabled earlier.

If you want it strongly, please reproduce such failing and prove that
zram was helpful for the system.

on that situation.



>
> -ss

--
Kind regards,
Minchan Kim
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/