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

From: Sergey Senozhatsky
Date: Tue Jul 07 2015 - 23:49:39 EST


On (07/08/15 12:04), Minchan Kim wrote:
> Hi Sergey,
>

Hi Minchan,

[..]
> > (a) we need something to signify that zs_unregister_shrinker() was successful
>
> I think a) is simple way to handle it now.
> I don't want to stuck with this issue.
>
> Please comment out why we need such boolean so after someone who has interest
> on shrinker clean-up is able to grab a chance.


OK, sure.

Do you think that (c) deserves a separate discussion (I can fork a new
discussion thread and Cc more people). It does look like we can do a bit
better and make shrinker less fragile (and probably cleanup/fix several
places in the kernel).

-ss
--
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/