From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753882AbbGKJqj (ORCPT ); Sat, 11 Jul 2015 05:46:39 -0400 Received: from mail-pd0-f180.google.com ([209.85.192.180]:34621 "EHLO mail-pd0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753824AbbGKJqh (ORCPT ); Sat, 11 Jul 2015 05:46:37 -0400 From: Sergey Senozhatsky To: Minchan Kim Cc: Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Sergey Senozhatsky , Sergey Senozhatsky Subject: [PATCH 3/3] zsmalloc: do not take class lock in zs_pages_to_compact() Date: Sat, 11 Jul 2015 18:45:32 +0900 Message-Id: <1436607932-7116-4-git-send-email-sergey.senozhatsky@gmail.com> X-Mailer: git-send-email 2.4.5 In-Reply-To: <1436607932-7116-1-git-send-email-sergey.senozhatsky@gmail.com> References: <1436607932-7116-1-git-send-email-sergey.senozhatsky@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org We can avoid taking class ->lock around zs_can_compact() in zs_pages_to_compact(), because the number that we return back is outdated in general case, by design. We have different source that are able to change class's state right after we return from zs_can_compact() -- ongoing IO operations, manually triggered compaction or automatic compaction, or all three simultaneously. We re-do this calculations during compaction on a per class basis anyway. zs_unregister_shrinker() will not return until we have an active shrinker, so classes won't unexpectedly disappear while zs_pages_to_compact(), invoked by zs_shrinker_count(), iterates them. When called from zram, we are protected by zram's ->init_lock, so, again, classes will be there until zs_pages_to_compact() iterates them. Signed-off-by: Sergey Senozhatsky --- mm/zsmalloc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c index b10a228..824c182 100644 --- a/mm/zsmalloc.c +++ b/mm/zsmalloc.c @@ -1811,9 +1811,7 @@ unsigned long zs_pages_to_compact(struct zs_pool *pool) if (class->index != i) continue; - spin_lock(&class->lock); pages_to_free += zs_can_compact(class); - spin_unlock(&class->lock); } return pages_to_free; -- 2.4.5