From patchwork Sun Jun 4 08:14:15 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: "Terekhov, Mikhail via Gdb-patches" X-Patchwork-Id: 20775 Received: (qmail 20696 invoked by alias); 4 Jun 2017 08:14:18 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 20673 invoked by uid 89); 4 Jun 2017 08:14:16 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00, FREEMAIL_FROM, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=ham version=3.3.2 spammy=2017-06-04, H*r:NNFMP, HX-Yahoo-Newman-Property:ymail-3, H*Ad:D*it X-HELO: nm35-vm9.bullet.mail.ir2.yahoo.com Received: from nm35-vm9.bullet.mail.ir2.yahoo.com (HELO nm35-vm9.bullet.mail.ir2.yahoo.com) (212.82.97.132) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 04 Jun 2017 08:14:14 +0000 Received: from [212.82.98.63] by nm35.bullet.mail.ir2.yahoo.com with NNFMP; 04 Jun 2017 08:14:16 -0000 Received: from [46.228.39.69] by tm16.bullet.mail.ir2.yahoo.com with NNFMP; 04 Jun 2017 08:14:16 -0000 Received: from [127.0.0.1] by smtp106.mail.ir2.yahoo.com with NNFMP; 04 Jun 2017 08:14:16 -0000 X-Yahoo-SMTP: RBKps0SswBBMl6DlXYPhfCWrkAuudq0GQgti Subject: Re: [PATCH] sim: microblaze: breakpoint inst check + a couple of questions To: Michael Eager , GDB Patches References: <295196608.1570236.1496487588855.ref@mail.yahoo.com> <295196608.1570236.1496487588855@mail.yahoo.com> <59330B7F.90802@eagerm.com> X-Patchwork-Original-From: "Andrea Corallo via gdb-patches" From: "Terekhov, Mikhail via Gdb-patches" Reply-To: Andrea Corallo Message-ID: <5416db91-3a90-7f4a-c901-97921aab1f9c@yahoo.it> Date: Sun, 4 Jun 2017 10:14:15 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <59330B7F.90802@eagerm.com> X-IsSubscribed: yes Ok understand. I think the best solution for now would be to apply this one here. In the simulator we stop executing just when we find a brki rD, 0x18 as the manual says, all other brk instructions are normal sw breakpoints and are gonna be executed. But we still want to check against any software breakpoint in the delayed slot cause is bad anyway. The next step will be to let gdb insert brki r0, 0x18 instead of brki r14, 96 when running on the simulator. Does all this make sense? I understand the sim was not developed since a while, anyway if appreciated I can put some effort to try to put it in a working state again. I'll have some more question but I guess is better to go step by step. Best Andrea sim/microblaze/ChangeLog: 2017-06-04 Andrea Corallo * interp.c (sim_engine_run): breakpoint check condition fix. --- sim/microblaze/interp.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) insts += 1; @@ -228,7 +228,7 @@ sim_engine_run (SIM_DESC sd, rb = GET_RB; ra = GET_RA; /* immword = IMM_W; */ - if (op == microblaze_brk) + if (op == microblaze_brk || op == brki) { if (STATE_VERBOSE_P (sd)) fprintf (stderr, "Breakpoint set in delay slot " On 03/06/2017 21:18, Michael Eager wrote: > On 06/03/2017 03:59 AM, Andrea Corallo via gdb-patches wrote: >> I have couple of questions related to microblaze debugging and its >> simulator: > > I can help with questions about MicroBlaze GDB, but can't say much > about the simulator. I looked at it briefly a long time, but most of > my debugging was on a development board. > > I don't believe that there has been any development on the MB sim for > many years. The QEMU simulator is much more up to date. > >> When gdb add a breakpoint writes to memory the following word >> 0xb9cc0060, this is defined in gdb/microblaze-tdep.h:120 >> >> /* MICROBLAZE_BREAKPOINT defines the breakpoint that should be used. >> Only used for native debugging. */ >> #define MICROBLAZE_BREAKPOINT {0xb9, 0xcc, 0x00, 0x60} >> >> This brki instruction cause the cpu to jump to 0x60 >> I guess this is because there is supposed to start a monitor program >> in some configuration correct? >> Because the simulator is not expecting any monitor program wouldn't >> be more appropriate to use hardware breakpoints instead? > > There is a note in the MicroBlaze Processor Reference Guide about the > use of "brk" and "brki" instructions: > > As a special case, when C_USE_DEBUG is set, and “brki rD, 0x18” is > executed, a > software breakpoint is signaled to the Xilinx Microprocesor > Debugger (XMD) tool, > irrespective of the value of C_BASE_VECTORS. > > (XMD is the JTAG pod used to debug using the GDB remote protocol.) > > Sim should do the something similar to this when running under GDB. > >> The other question is: the simulator is checking against the presence >> of a brk instruction but not brki making gdb not stopping on the >> breakpoint just inserted. >> Would make sense to check against both as in the following patch? >> >> sim/microblaze/ChangeLog: >> 2017-06-01 Andrea Corallo >> >> * interp.c (sim_engine_run): check also for breakpoint instruction brki. >> --- >> sim/microblaze/interp.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c >> index 75fc98b..d094a69 100644 >> --- a/sim/microblaze/interp.c >> +++ b/sim/microblaze/interp.c >> @@ -161,7 +161,7 @@ sim_engine_run (SIM_DESC sd, >> oldpc = PC; >> delay_slot_enable = 0; >> branch_taken = 0; >> - if (op == microblaze_brk) >> + if (op == microblaze_brk || op == brki) >> sim_engine_halt (sd, NULL, NULL, NULL_CIA, sim_stopped, SIM_SIGTRAP); >> else if (inst == MICROBLAZE_HALT_INST) >> { > > There is another use of microblaze_brk where a check is made whether a > "brk" instruction is being inserted in a delay slot. I believe that > this should also be updated to also check for a "brki" instruction. > diff --git a/sim/microblaze/interp.c b/sim/microblaze/interp.c index 75fc98b..48cc8cb 100644 --- a/sim/microblaze/interp.c +++ b/sim/microblaze/interp.c @@ -161,8 +161,8 @@ sim_engine_run (SIM_DESC sd, oldpc = PC; delay_slot_enable = 0; branch_taken = 0; - if (op == microblaze_brk) - sim_engine_halt (sd, NULL, NULL, NULL_CIA, sim_stopped, SIM_SIGTRAP); + if (op == brki && IMM == 0x18) + sim_engine_halt (sd, NULL, NULL, NULL_CIA, sim_stopped, SIM_SIGTRAP); else if (inst == MICROBLAZE_HALT_INST) {