From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 90995 invoked by alias); 10 Dec 2015 16:02:26 -0000 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 Received: (qmail 90982 invoked by uid 89); 10 Dec 2015 16:02:25 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.6 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.2 X-HELO: usplmg21.ericsson.net Received: from usplmg21.ericsson.net (HELO usplmg21.ericsson.net) (198.24.6.65) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Thu, 10 Dec 2015 16:02:15 +0000 Received: from EUSAAHC007.ericsson.se (Unknown_Domain [147.117.188.93]) by usplmg21.ericsson.net (Symantec Mail Security) with SMTP id 48.BC.32102.002A9665; Thu, 10 Dec 2015 17:02:08 +0100 (CET) Received: from [142.133.110.95] (147.117.188.8) by smtp-am.internal.ericsson.com (147.117.188.95) with Microsoft SMTP Server id 14.3.248.2; Thu, 10 Dec 2015 11:02:12 -0500 Subject: Re: [PATCH v7 0/8] Support software single step and conditional breakpoints on ARM in GDBServer. To: Yao Qi References: <1449583641-18156-1-git-send-email-antoine.tremblay@ericsson.com> <86poyeqopn.fsf@gmail.com> CC: From: Antoine Tremblay Message-ID: <5669A204.1000602@ericsson.com> Date: Thu, 10 Dec 2015 16:02:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <86poyeqopn.fsf@gmail.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2015-12/txt/msg00196.txt.bz2 On 12/10/2015 10:59 AM, Yao Qi wrote: > Antoine Tremblay writes: > >> In this v7: >> >> * New patch 3 changes magic numbers for syscall check to numbers computed >> from xml-syscall. >> >> * Patch 4 now uses the new xml-syscall function to compare syscalls. >> It also uses the INT_REGISTER_SIZE and ARM_PC_REGNUM defines. >> >> * Patch 5: Fixed Makefile changelog. >> >> * Patch 6: >> ** Cleanup where arm-get-next-pcs.h is included. >> ** Move unrelated functions and macros from arm-get-next-pcs.h/.c to arm.h/.c. >> ** Fixed arm/thumb_deal_with_atomic_sequence calls. >> ** Fixed changelog Makefile entries. >> ** Fixed changelog configure.tgt/srv entries. >> ** Use __NR_sigreturn and __NR_rt_sigreturn to check for sigreturn syscall >> in GDBServer. > > Hi Antoine, thanks for the update... > > FYI, I am reviewing your patches and trying to run various regression > tests for them. Even without your patch series, there is a regression > (in gdb.base/advance.exp) in aarch64 mutli-arch debugging that is, > breakpoint on main isn't hit when aarch64 GDBserver debugs arm program > in arm mode (everything works fine if the arm program is in thumb mode). > I suspect that it is related to my recent patch turning Z0 packet on for > aarch64 mutli-arch debugging. I really want to clean the room first > before we add news things in, so I'll fix the regression first, and then > come back to review your patches. > Sure, thanks for letting me know. Regards, Antoine