From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25110 invoked by alias); 2 Apr 2012 15:51:12 -0000 Received: (qmail 25095 invoked by uid 22791); 2 Apr 2012 15:51:10 -0000 X-SWARE-Spam-Status: No, hits=-7.1 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,KHOP_THREADED,RCVD_IN_DNSWL_HI,RCVD_IN_HOSTKARMA_W,SPF_HELO_PASS,T_RP_MATCHES_RCVD 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; Mon, 02 Apr 2012 15:50:54 +0000 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q32FoKgf005758 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Mon, 2 Apr 2012 11:50:21 -0400 Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.phx2.redhat.com [10.5.9.1]) by int-mx01.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id q32FoIvw006717; Mon, 2 Apr 2012 11:50:19 -0400 Message-ID: <4F79CABA.8010009@redhat.com> Date: Mon, 02 Apr 2012 15:51:00 -0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120316 Thunderbird/11.0 MIME-Version: 1.0 To: Roland Schwingel CC: gdb-patches@sourceware.org, Joel Brobecker Subject: Re: [PATCH v4] Add dll trampoline code handling for windows 64bit References: <4F79BD71.4010703@onevision.com> In-Reply-To: <4F79BD71.4010703@onevision.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-04/txt/msg00016.txt.bz2 On 04/02/2012 03:53 PM, Roland Schwingel wrote: >> Please send an updated patch, so we have in the archives the exact >> patch as what is >> checked in, and in case some other maintainer wants to take a look, >> best have him look at >> the refreshed patch. In fact, if you had sent it already in that >> email, there'd have been >> no extra noise, right? ;-) > Regarding the noise right. But not regarding generating additional work. Sorry, I can't be sympathetic to that. You would be making the changes anyway. I can't believe that pasting a patch at the end of an email is extra work by any valid measure. What's real extra work is someone reading an out of date patch, and trying to figure out from several messages in a thread what would be the final state of the patch. > 2012-04-02 Roland Schwingel ^ Should be two spaces after your name. > > * amd64-windows-tdep.c: #include "frame.h". > (amd64_windows_skip_trampoline_code): New function. > (amd64_windows_init_abi): Add trampoline registration. On 04/02/2012 03:53 PM, Roland Schwingel wrote: > +/* Check win64 DLL jmp trampolines and find jump destination. */ The correct spelling is "Win64" capitalized. > static void > amd64_windows_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch) > { > struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch); > > + /* Register trampoline handling code. */ > + set_gdbarch_skip_trampoline_code (gdbarch, amd64_windows_skip_trampoline_code); A nit, but it'd be cleaner/clearer to put this after the amd64_init_abi call, or better, near the end of the function, after set_gdbarch_skip_main_prologue. The current code reads "initialize the base arch, then install overrides.". This new call here breaks that flow. > + > amd64_init_abi (info, gdbarch); > > /* On Windows, "long"s are only 32bit. */ Having once written the equivalent arm-wince-tdep.c:arm_pe_skip_trampoline_code for ARM WinCE, this generally looks good to me too. -- Pedro Alves