Re: [PATCH v2 29/47] vmw_balloon: dynamically allocate the vmw-balloon shrinker

From: Muchun Song
Date: Wed Jul 26 2023 - 03:40:01 EST




> On Jul 24, 2023, at 17:43, Qi Zheng <zhengqi.arch@xxxxxxxxxxxxx> wrote:
>
> In preparation for implementing lockless slab shrink, use new APIs to
> dynamically allocate the vmw-balloon shrinker, so that it can be freed
> asynchronously using kfree_rcu(). Then it doesn't need to wait for RCU
> read-side critical section when releasing the struct vmballoon.
>
> And we can simply exit vmballoon_init() when registering the shrinker
> fails. So the shrinker_registered indication is redundant, just remove it.
>
> Signed-off-by: Qi Zheng <zhengqi.arch@xxxxxxxxxxxxx>

Nice cleanup.

Reviewed-by: Muchun Song <songmuchun@xxxxxxxxxxxxx>