From: Tejun Heo Subject: Re: [PATCH 2/3] percpu_stats: Simple per-cpu statistics count helper functions Date: Thu, 7 Apr 2016 12:06:23 -0400 Message-ID: <20160407160623.GF7822@mtj.duckdns.org> References: <1459566578-30221-1-git-send-email-Waiman.Long@hpe.com> <1459566578-30221-3-git-send-email-Waiman.Long@hpe.com> <20160404160228.GW7822@mtj.duckdns.org> <570584F1.10909@hpe.com> <20160406225424.GK24661@htj.duckdns.org> <57068395.1080703@hpe.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Theodore Ts'o , Andreas Dilger , Christoph Lameter , linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org, Scott J Norton , Douglas Hatch , Toshimitsu Kani To: Waiman Long Return-path: Received: from mail-yw0-f175.google.com ([209.85.161.175]:32929 "EHLO mail-yw0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751559AbcDGQG0 (ORCPT ); Thu, 7 Apr 2016 12:06:26 -0400 Content-Disposition: inline In-Reply-To: <57068395.1080703@hpe.com> Sender: linux-ext4-owner@vger.kernel.org List-ID: Hello, Waiman. On Thu, Apr 07, 2016 at 11:58:13AM -0400, Waiman Long wrote: > We can certainly make it watertight. However, that will certainly require > adding performance overhead in the percpu stats update fast path which I am > not willing to pay. There are multiple options depending on the specific balance you want to hit. Reset can be made very heavy (involving RCU sync operation) to make hot path overhead minimal, local locking or atomic ops can also be used which while more expensive than this_cpu_*() ops still avoids cacheline bouncing. > The purpose of this stat counters reset functionality is to allow developers > to reset the stat counters, run certain workload and see how things are > going in the kernel when the workload completes assuming that those stat > counters are exposed via sysfs, debugfs, etc. The developers can certainly > check the stat counters after the reset to make sure that they are properly > reset. So I don't think we need an airtight way of doing it. If you have > scenarios in your mind that require airtight reset of the stat counters, > please let me know and I will see what I can do about it. No matter what, don't create something which can yield a completely surprising result once in a blue moon. You might think it's okay because the likelihood is low but that just means that the resulting malfunctions will be that much more obscure and difficult to reproduce. Thanks. -- tejun