From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 825B5C4363D for ; Wed, 23 Sep 2020 05:10:10 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 32CE021D43 for ; Wed, 23 Sep 2020 05:10:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1600837810; bh=tTgmoQ8WAIDqb0vLRAcBgAkHFi6g5HuRQSPr5H4Pduk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=hXW/1bHWQ34P0ofGd7ZFL4Vrw17PlkrKRecM1ztfcotNymGpWCPMlsWbdLnLfiml6 +YPP2H//fGHozMAZ8jykY9dY3Ac4K3QEhst8cTyMk5qtgRrOSz09IwHSa7hBO6+74A z9OTHAutLlTUFdsEEnvr9LwRrBR3KwrHSYLgGKaM= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726884AbgIWFKI (ORCPT ); Wed, 23 Sep 2020 01:10:08 -0400 Received: from mail.kernel.org ([198.145.29.99]:36178 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726834AbgIWFKI (ORCPT ); Wed, 23 Sep 2020 01:10:08 -0400 Received: from localhost (83-86-74-64.cable.dynamic.v4.ziggo.nl [83.86.74.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 3950F21941; Wed, 23 Sep 2020 05:10:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1600837807; bh=tTgmoQ8WAIDqb0vLRAcBgAkHFi6g5HuRQSPr5H4Pduk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Q3LzTJGXCkLJ/2TZFYkn889BGPESGk7bKQEJOlMhADRceQF76fHriVkxMWo81devO xgv8oJUhtXkwCPCxuNJrmqv+NlM8PTc7bnOMr1lMUXivvhms5hF02CLnAkn4b0od5j cXa0hUSnA06U5jYBwIIAmi5PwGgN7uVdGvrcFR/E= Date: Wed, 23 Sep 2020 07:10:27 +0200 From: Greg KH To: Shuah Khan Cc: arve@android.com, tkjos@android.com, maco@android.com, joel@joelfernandes.org, christian@brauner.io, hridya@google.com, surenb@google.com, keescook@chromium.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH 07/11] drivers/android/binder: convert stats, transaction_log to counter_atomic Message-ID: <20200923051027.GA2578443@kroah.com> References: <4fe28fc5f315657e4af276b8a3c71d80a5eaa379.1600816121.git.skhan@linuxfoundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4fe28fc5f315657e4af276b8a3c71d80a5eaa379.1600816121.git.skhan@linuxfoundation.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Sep 22, 2020 at 07:43:36PM -0600, Shuah Khan wrote: > counter_atomic is introduced to be used when a variable is used as > a simple counter and doesn't guard object lifetimes. This clearly > differentiates atomic_t usages that guard object lifetimes. > > counter_atomic variables will wrap around to 0 when it overflows and > should not be used to guard resource lifetimes, device usage and > open counts that control state changes, and pm states. > > stats tracks per-process binder statistics. Unsure if there is a chance > of this overflowing, other than stats getting reset to 0. Convert it to > use counter_atomic. > > binder_transaction_log:cur is used to keep track of the current log entry > location. Overflow is handled in the code. Since it is used as a > counter, convert it to use counter_atomic. > > This conversion doesn't change the oveflow wrap around behavior. > > Signed-off-by: Shuah Khan > --- > drivers/android/binder.c | 41 ++++++++++++++++--------------- > drivers/android/binder_internal.h | 3 ++- > 2 files changed, 23 insertions(+), 21 deletions(-) > > diff --git a/drivers/android/binder.c b/drivers/android/binder.c > index f936530a19b0..11a0407c46df 100644 > --- a/drivers/android/binder.c > +++ b/drivers/android/binder.c > @@ -66,6 +66,7 @@ > #include > #include > #include > +#include > > #include > #include > @@ -172,22 +173,22 @@ enum binder_stat_types { > }; > > struct binder_stats { > - atomic_t br[_IOC_NR(BR_FAILED_REPLY) + 1]; > - atomic_t bc[_IOC_NR(BC_REPLY_SG) + 1]; > - atomic_t obj_created[BINDER_STAT_COUNT]; > - atomic_t obj_deleted[BINDER_STAT_COUNT]; > + struct counter_atomic br[_IOC_NR(BR_FAILED_REPLY) + 1]; > + struct counter_atomic bc[_IOC_NR(BC_REPLY_SG) + 1]; > + struct counter_atomic obj_created[BINDER_STAT_COUNT]; > + struct counter_atomic obj_deleted[BINDER_STAT_COUNT]; These are just debugging statistics, no reason they have to be atomic variables at all and they should be able to just be "struct counter" variables instead. thanks for looking into all of these! greg k-h