From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18918 invoked by alias); 24 Nov 2009 15:40:35 -0000 Received: (qmail 18911 invoked by uid 22791); 24 Nov 2009 15:40:35 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 24 Nov 2009 15:40:27 +0000 Received: from int-mx04.intmail.prod.int.phx2.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.17]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id nAOFeQg0009707 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Tue, 24 Nov 2009 10:40:26 -0500 Received: from fche.csb (vpn-243-193.phx2.redhat.com [10.3.243.193]) by int-mx04.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id nAOFePmo010419; Tue, 24 Nov 2009 10:40:25 -0500 Received: by fche.csb (Postfix, from userid 2569) id 186C5580B0; Tue, 24 Nov 2009 10:40:25 -0500 (EST) To: Frederic Weisbecker Cc: Masami Hiramatsu , Ingo Molnar , Ananth N Mavinakayanahalli , lkml , systemtap , DLE , Jim Keniston , Srikar Dronamraju , Christoph Hellwig , Steven Rostedt , "H. Peter Anvin" , Anders Kaseorg , Tim Abbott , Andi Kleen , Jason Baron , Mathieu Desnoyers Subject: Re: [PATCH -tip v5 06/10] kprobes/x86: Cleanup save/restore registers References: <20091123232115.22071.71558.stgit@dhcp-100-2-132.bos.redhat.com> <20091123232204.22071.60562.stgit@dhcp-100-2-132.bos.redhat.com> <20091124025115.GB6752@nowhere> From: fche@redhat.com (Frank Ch. Eigler) Date: Tue, 24 Nov 2009 15:40:00 -0000 In-Reply-To: <20091124025115.GB6752@nowhere> (Frederic Weisbecker's message of "Tue, 24 Nov 2009 03:51:17 +0100") Message-ID: User-Agent: Gnus/5.1008 (Gnus v5.10.8) Emacs/21.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Mailing-List: contact systemtap-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: systemtap-owner@sourceware.org X-SW-Source: 2009-q4/txt/msg00663.txt.bz2 Frederic Weisbecker writes: > [...] >> +#define SAVE_REGS_STRING \ >> +#define RESTORE_REGS_STRING \ > > BTW, do you really need to push/pop every registers > before/after calling a probe handler? It's part of the definition of a kprobe, that a populated pt_regs* value is passed. Clients can rely on that in order to access registers etc. - FChE