From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753043Ab1G2VZK (ORCPT ); Fri, 29 Jul 2011 17:25:10 -0400 Received: from hrndva-omtalb.mail.rr.com ([71.74.56.124]:45734 "EHLO hrndva-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752687Ab1G2VZJ (ORCPT ); Fri, 29 Jul 2011 17:25:09 -0400 X-Authority-Analysis: v=1.1 cv=s3eDhkhcaTLnj7IEXy8aaXUiY7FbET0mf+/2Xe0elbc= c=1 sm=0 a=oZG1OEA3kxQA:10 a=5SG0PmZfjMsA:10 a=Q9fys5e9bTEA:10 a=OPBmh+XkhLl+Enan7BmTLg==:17 a=meVymXHHAAAA:8 a=wSm1IEvobLJBBnI8H78A:9 a=PUjeQqilurYA:10 a=jeBq3FmKZ4MA:10 a=OPBmh+XkhLl+Enan7BmTLg==:117 X-Cloudmark-Score: 0 X-Originating-IP: 67.242.120.143 Subject: Re: [PATCH 3/5] trace: Add per_cpu ring buffer control files From: Steven Rostedt To: Vaibhav Nagarnaik Cc: Frederic Weisbecker , Ingo Molnar , Michael Rubin , David Sharp , linux-kernel@vger.kernel.org In-Reply-To: References: <1311721194-12164-1-git-send-email-vnagarnaik@google.com> <1311721194-12164-4-git-send-email-vnagarnaik@google.com> <1311963247.21143.76.camel@gandalf.stny.rr.com> Content-Type: text/plain; charset="ISO-8859-15" Date: Fri, 29 Jul 2011 17:25:07 -0400 Message-ID: <1311974707.21143.93.camel@gandalf.stny.rr.com> Mime-Version: 1.0 X-Mailer: Evolution 2.32.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2011-07-29 at 12:13 -0700, Vaibhav Nagarnaik wrote: > On Fri, Jul 29, 2011 at 11:14 AM, Steven Rostedt wrote: > > On Tue, 2011-07-26 at 15:59 -0700, Vaibhav Nagarnaik wrote: > >> Add a debugfs entry under per_cpu/ folder for each cpu called > >> buffer_size_kb to control the ring buffer size for each CPU > >> independently. > >> > >> If the global file buffer_size_kb is used to set size, the individual > >> ring buffers will be adjusted to the given size. The buffer_size_kb will > >> report the common size to maintain backward compatibility. > >> > >> If the buffer_size_kb file under the per_cpu/ directory is used to > >> change buffer size for a specific CPU, only the size of the respective > >> ring buffer is updated. When tracing/buffer_size_kb is read, it reports > >> the ring buffer sizes of all the CPUs at that point. > > > > No, buffer_size_kb should not change in the what it reports. This is why > > you have a buffer_total_size_kb. Use that. If the per_cpu buffers are > > changed, then this should just report "various" or something to that > > affect. This will be a good way to know if the per_cpu buffers are the > > same or not. > > Aargh. > > I updated the code to return 'X' when individual cpu buffers don't match > up in size and forgot to update the changelog, I will do it now. > Heh, I only looked at the changelog. I was going to say 'X' but then thought that 'various' would be good too. But since we already use 'X' for the filters and enabled files, it is better to stay consistent. So 'X' is fine. -- Steve