Messages in this thread | | | Date | Wed, 8 Jul 2015 12:49:28 +0900 | From | Sergey Senozhatsky <> | Subject | Re: [PATCH v6 7/7] zsmalloc: use shrinker to trigger auto-compaction |
| |
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
| |