Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp3515999pxj; Tue, 11 May 2021 06:19:40 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzgQpGnfEzTrwS+aXWajhEw0Dbz4Xl2NiV9oPOTUzvuirGHWHRYoa3OepgZ+L/S2IfxQPa+ X-Received: by 2002:a05:6402:c03:: with SMTP id co3mr35765224edb.133.1620739180639; Tue, 11 May 2021 06:19:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1620739180; cv=none; d=google.com; s=arc-20160816; b=gt35/udakpj0iIioba+ZQkmD661rZM6kt5/YK+7IS0704DY6A5+jJ1T9i4QW8RwB4R 55BWsaCTyG0A4odLriKOwYu0DWxb0psjbxqooDgv0uobMZeUUF75Hd957km38W49EhLH nMWPhrLMV1vDsAeOYkRydZgbUD8o7Np6X7MCoc6ozZVzmBF1xBRA+3zqnmIQokIBCf3w rLbaNTXgv4vPkPnn/BivS8DD33polNRSFlfcJJ4DYTWIsvM/h1o94XVsgMLTJdybCPj2 D2lZi5Com6iNHJ1bUQ7QspvJG/lN2+uxTGoHIdmGGISd1WemNwOgJiMzPh9B3Q7oPC+3 eQsQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=pd3fEfzBrzOrkq1JY1d5cYXuP5xKjB+6k3CX00vHtuk=; b=U7s1cVAmkYQDSWH5VZXzSPwVK5890up982s+JepPSMehyocN8N7IKByVTPFd7IRlRv RvFP5JbLyFXqONjIkRlvuXhNp2ScgLpUiGfBGmxA/Pn+Q2eRbb2fJS4RMcfqNgn2RXMc 3f8ITCblCy6msiMuAMUO8e4g1VhzOHvhG9trvcPt/TeUJ2EnI6MvpXTubUaCYMCvUnp1 r5Q44jwcBrwiDDp/dUebAnuSey2DPlo0p0sYKmKwCMYiMtvO3yq/i818sTYZ5l06P6Tm G5AaIlkF92AbawDOHyPYFoSM75SGXqolZ1QD2khvwBgIyXHBzejdWarG+21CG1OnnBp6 FMRw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@suse.com header.s=susede1 header.b="r/bgiUlX"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=suse.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id hp22si16506596ejc.174.2021.05.11.06.19.16; Tue, 11 May 2021 06:19:40 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@suse.com header.s=susede1 header.b="r/bgiUlX"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=suse.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231305AbhEKNRo (ORCPT + 99 others); Tue, 11 May 2021 09:17:44 -0400 Received: from mx2.suse.de ([195.135.220.15]:39448 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230483AbhEKNRo (ORCPT ); Tue, 11 May 2021 09:17:44 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1620738997; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=pd3fEfzBrzOrkq1JY1d5cYXuP5xKjB+6k3CX00vHtuk=; b=r/bgiUlXW9PNoPsuD0Pqvk9xQfqLyWg/STLoDWjztuGG/v9NhLlsvgP+sDXt1P1fGRJZSl MokOmDx+hhvqp38bWfqNTs8z9NictHfylZzTI2vetR1S3sMGzDnWjBIguPzymU9/OP8Ch0 KNRYykuuIMf5Jku/fR5p4gadoqyA41I= Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id EF8C3B1D7; Tue, 11 May 2021 13:16:36 +0000 (UTC) From: Nikolay Borisov To: dennis@kernel.org Cc: tj@kernel.org, linux-kernel@vger.kernel.org, Nikolay Borisov Subject: [PATCH] percpu_ref: Don't opencode percpu_ref_is_dying Date: Tue, 11 May 2021 16:16:33 +0300 Message-Id: <20210511131633.185463-1-nborisov@suse.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Signed-off-by: Nikolay Borisov --- lib/percpu-refcount.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/percpu-refcount.c b/lib/percpu-refcount.c index a1071cdefb5a..af9302141bcf 100644 --- a/lib/percpu-refcount.c +++ b/lib/percpu-refcount.c @@ -275,7 +275,7 @@ static void __percpu_ref_switch_mode(struct percpu_ref *ref, wait_event_lock_irq(percpu_ref_switch_waitq, !data->confirm_switch, percpu_ref_switch_lock); - if (data->force_atomic || (ref->percpu_count_ptr & __PERCPU_REF_DEAD)) + if (data->force_atomic || percpu_ref_is_dying(ref)) __percpu_ref_switch_to_atomic(ref, confirm_switch); else __percpu_ref_switch_to_percpu(ref); @@ -385,7 +385,7 @@ void percpu_ref_kill_and_confirm(struct percpu_ref *ref, spin_lock_irqsave(&percpu_ref_switch_lock, flags); - WARN_ONCE(ref->percpu_count_ptr & __PERCPU_REF_DEAD, + WARN_ONCE(percpu_ref_is_dying(ref), "%s called more than once on %ps!", __func__, ref->data->release); @@ -465,7 +465,7 @@ void percpu_ref_resurrect(struct percpu_ref *ref) spin_lock_irqsave(&percpu_ref_switch_lock, flags); - WARN_ON_ONCE(!(ref->percpu_count_ptr & __PERCPU_REF_DEAD)); + WARN_ON_ONCE(!percpu_ref_is_dying(ref)); WARN_ON_ONCE(__ref_is_percpu(ref, &percpu_count)); ref->percpu_count_ptr &= ~__PERCPU_REF_DEAD; -- 2.25.1