Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752487Ab0FWSEk (ORCPT ); Wed, 23 Jun 2010 14:04:40 -0400 Received: from mx1.redhat.com ([209.132.183.28]:55714 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751804Ab0FWSEj (ORCPT ); Wed, 23 Jun 2010 14:04:39 -0400 Date: Wed, 23 Jun 2010 14:03:54 -0400 From: Jason Baron To: Ian Munsie Cc: linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org, Frederic Weisbecker , Steven Rostedt , Ingo Molnar , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Andrew Morton , Christoph Hellwig , Jesper Nilsson , David Howells , Ingo Molnar Subject: Re: [PATCH 39/40] trace syscalls: Clean confusing {s,r,}name and fix ABI breakage Message-ID: <20100623180353.GB2680@redhat.com> References: <1277287401-28571-1-git-send-email-imunsie@au1.ibm.com> <1277287401-28571-40-git-send-email-imunsie@au1.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1277287401-28571-40-git-send-email-imunsie@au1.ibm.com> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1978 Lines: 47 On Wed, Jun 23, 2010 at 08:03:20PM +1000, Ian Munsie wrote: > From: Ian Munsie > > This patch cleans up the preprocessor macros defining system calls by > standidising on the parameters passing around the system call name, with > and without it's prefix. > > Overall this makes the preprocessor code easier to understand and > follow and less likely to introduce bugs due to misunderstanding what to > place into each parameter. > > The parameters henceforth should be named: > > sname is the system call name WITHOUT the prefix (e.g. read) > prefix is the prefix INCLUDING the trailing underscore (e.g. sys_) > > These are mashed together to form the full syscall name when required > like blah##prefix##sname. For just prefix##sname please use the provided > SYSNAME(prefix,sname) macro instead as it will be safe if prefix itself > is a macro. > > This patch also fixes an ABI breakage - the ftrace events are once again > named like 'sys_enter_read' instead of 'enter_sys_read'. The rwtop > script shipped with perf relies on this ABI. Others may as well. > > Signed-off-by: Ian Munsie > --- overall this patch is a major improvement! My question though is about the naming of the compat syscalls in the context of events. I believe this patch differeniates compat syscall event names as: "sys32_enter_sname", and "compat_sys_enter_sname". I agree that we keep that distinction for purposes of defining the actual syscall function. However, we had discuessed previously about keeping the event name the same for all compat syscalls. ie they are all called "compat_sys_sname" or some such. Reason being you could just do "compat_*" to match all compat events. thanks, -Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/