From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 100955 invoked by alias); 26 Nov 2015 16:04:02 -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 100932 invoked by uid 89); 26 Nov 2015 16:04:01 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.3 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-pa0-f50.google.com Received: from mail-pa0-f50.google.com (HELO mail-pa0-f50.google.com) (209.85.220.50) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Thu, 26 Nov 2015 16:03:59 +0000 Received: by pacdm15 with SMTP id dm15so89840955pac.3 for ; Thu, 26 Nov 2015 08:03:58 -0800 (PST) X-Received: by 10.66.122.105 with SMTP id lr9mr61190939pab.8.1448553837986; Thu, 26 Nov 2015 08:03:57 -0800 (PST) Received: from [127.0.0.1] (gcc1-power7.osuosl.org. [140.211.15.137]) by smtp.gmail.com with ESMTPSA id v2sm28394893pfi.44.2015.11.26.08.03.55 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 26 Nov 2015 08:03:57 -0800 (PST) Subject: Re: [PATCH v3 08/10] Support software single step on ARM in GDBServer. To: Antoine Tremblay References: <1448287968-12907-1-git-send-email-antoine.tremblay@ericsson.com> <1448287968-12907-9-git-send-email-antoine.tremblay@ericsson.com> <86ziy1vsdr.fsf@gmail.com> <5657213B.30504@ericsson.com> Cc: gdb-patches@sourceware.org From: Yao Qi Message-ID: <56572D68.90107@gmail.com> Date: Thu, 26 Nov 2015 16:04:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <5657213B.30504@ericsson.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2015-11/txt/msg00567.txt.bz2 On 26/11/15 15:11, Antoine Tremblay wrote: > This is the same link as the previous one... > Oops, sorry, https://sourceware.org/ml/gdb-patches/2007-06/msg00087.html > IMO, it is >> better to use regcache than frame. We have two options, >> >> #1, switch from frame apis to regcache apis to access registers in arm >> software single step. We can get regcache by get_current_regcache (). >> #2, change argument of gdbarch method software_single_step from frame >> to regcache, which means all its implementations need update, and >> switch to regcache apis to access registers. >> >> #2 is the right way to go in long term, and we really need to improve >> software_single_step. Let me what do you think. >> > > Looking at the impacts of #2, I do not feel comfortable including these > changes in this patch set. I feel they would require a patch set of > their own. > > However #1 seems like something possible I would start by this option if > that's fine with you ? Yes. > > Also, I can still do the refactoring before this patch but it will > require more work since I'll have to diff the functions moved etc.. do > you feel it's required to do so or the refactoring could be done after > this patch ? I prefer doing the refactor first, and separately, because after this refactor, your patch #8 will be simplified a lot. In this series, we want to share the code on arm software single step, however, registers are accessed through frame in GDB side, while through regcache in GDBserver. In order to share code, we should unify them as much as we can, that is, access registers through regcache as well in GDB side. Then, we can move the code from arm-tdep.c to arch/ directory, to support software single step in GDBserver. -- Yao (齐尧)