From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760954AbcBYR1F (ORCPT ); Thu, 25 Feb 2016 12:27:05 -0500 Received: from mail-pf0-f195.google.com ([209.85.192.195]:36281 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758976AbcBYR1D (ORCPT ); Thu, 25 Feb 2016 12:27:03 -0500 Subject: Re: [PATCH RESEND 1/3] tracing/syscalls: Rename variable 'nr' to 'syscall_nr' To: Steven Rostedt References: <1456413858-13901-1-git-send-email-treeze.taeung@gmail.com> <20160225104741.2cd3bf14@gandalf.local.home> <56CF319B.8060806@gmail.com> <20160225120501.5a99408c@gandalf.local.home> <20160225120907.0a30d792@gandalf.local.home> Cc: Ingo Molnar , Arnaldo Carvalho de Melo , Jiri Olsa , Namhyung Kim , linux-kernel@vger.kernel.org, Thomas Gleixner , Lai Jiangshan From: Taeung Song Message-ID: <56CF3962.9060400@gmail.com> Date: Fri, 26 Feb 2016 02:26:58 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <20160225120907.0a30d792@gandalf.local.home> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/26/2016 02:09 AM, Steven Rostedt wrote: > On Thu, 25 Feb 2016 12:05:01 -0500 > Steven Rostedt wrote: > >> On Fri, 26 Feb 2016 01:53:47 +0900 >> Taeung Song wrote: >> >>> >>> The underlying cause of this problem is not because of >>> the python script. This problem have relevance to tracing/syscalls >>> as below. >>> >>> # cat >>> /sys/kernel/debug/tracing/events/syscalls/sys_enter_io_getevents/format >>> name: sys_enter_io_getevents >>> ID: 739 >>> format: >>> field:unsigned short common_type; offset:0;size:2;signed:0; >>> field:unsigned char common_flags; offset:2;size:1;signed:0; >>> field:unsigned char common_preempt_count; offset:3;size:1;signed:0; >>> field:int common_pid; offset:4;size:4;signed:1; >>> field:int nr; offset:8;size:4;signed:1; >>> field:aio_context_t ctx_id; offset:16;size:8;signed:0; >>> field:long min_nr; offset:24;size:8;signed:0; >>> field:long nr; offset:32;size:8;signed:0; >>> field:struct io_event * events; offset:40;size:8;signed:0; >>> field:struct timespec * timeout; offset:48;size:8;signed:0; >>> >>> print fmt: "ctx_id: 0x%08lx, min_nr: 0x%08lx, nr: 0x%08lx, events: >>> 0x%08lx, timeout: 0x%08lx", ((unsigned long)(REC->ctx_id)), ((unsigned >>> long)(REC->min_nr)), ((unsigned long)(REC->nr)), ((unsigned >>> long)(REC->events)), ((unsigned long)(REC->timeout)) >>> >>> >>> As you see, there are duplicated variable name among fields. >>> So I renamed the variable name 'nr' to 'syscall_nr' to avoid this >>> duplication. As you know 'nr' in print_syscall_enter() mean >>> system call number so, I changed 'nr' to 'syscall_nr'(IMHO). >>> >> >> The above should have been in the change log. That makes much more >> sense. >> Sorry, this change log is misinterpretable. So I'm rewriting change log for this patch. I will resend it soon. > > I'm a bit nervous about this change though. Because it is an ABI > change. Hopefully there's no tools that parse the syscall tracepoints > and expect to find a 'nr' in there. > AFIAIK, I think so but resend this modified patch. Thanks, Taeung