This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch, avr, pr19401] Update PC after break
- From: Pitchumani Sivanupandi <pitchumani dot s at atmel dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: <gdb-patches at sourceware dot org>, <chertykov at gmail dot com>, <troth at openavr dot org>, <vapier at gentoo dot org>, <Senthil_Kumar dot Selvaraj at atmel dot com>
- Date: Thu, 24 Mar 2016 19:10:33 +0530
- Subject: Re: [patch, avr, pr19401] Update PC after break
- Authentication-results: sourceware.org; auth=none
- References: <20160323141426 dot GA10252 at CHELT0346> <56F2AAC3 dot 8000101 at redhat dot com>
On Wed, Mar 23, 2016 at 02:40:03PM +0000, Pedro Alves wrote:
> On 03/23/2016 02:14 PM, Pitchumani Sivanupandi wrote:
> > avr-gdb doesn't seem to be rewinding the PC after break. If a breakpoint
> > is at four byte instruction, it resumes from the middle of the
> > instruction. This caused the target to reject the next (half) instruction
> > as illegal. In case of breakpoint at two byte instruction, it resumes from
> > the the next instruction. Instruction at breakpoint location was skipped
> > as the PC was rewinded after break.
> >
> > Test case in PR19401 is the example for the first situation. Below
> > example is for second situation.
>
> How was this not noticed before?
This test case was working till gdb-7.9. May be a regression from 7.10.
> > command line: avr-gcc test.c -mmcu=atmega2560 -g -o test.elf
> > avr-gdb test.elf
> > (gdb) target sim
> > (gdb) load
> > (gdb) b main
> > (gdb) run
> > 0x00000124 in main () at sim1.c:5
>
> I would expect to see mention of a SIGTRAP here, and for all
> breakpoints, as gdb won't be able to map the current PC address
> with any GDB breakpoint. Doesn't that happen?
copy-paster error. SIGTRAP occurs for this case.
Update test case and gdb log:
volatile int a = 12;
int main ()
{
a = 0x0123;
while (1)
a++;
return 0;
}
avr-gcc sim1.c -mmcu=atmega2560 -g -o sim1.elf
<snip>
Reading symbols from sim1.elf...done.
(gdb) target sim
Connected to the simulator.
(gdb) load
Loading section .data, size 0x2 lma 0x146
Loading section .text, size 0x146 lma 0x0
Start address 0x0
Transfer rate: 2624 bits in <1 sec.
(gdb) b main
Breakpoint 1 at 0x122: file sim1.c, line 5.
(gdb) run
Starting program: /home/zero/build/avr-trunk/sim1.elf
Program received signal SIGTRAP, Trace/breakpoint trap.
0x00000124 in main () at sim1.c:5
5 a = 0x0123;
(gdb) disassemble
Dump of assembler code for function main:
0x0000011a <+0>: push r28
0x0000011c <+2>: push r29
0x0000011e <+4>: in r28, 0x3d ; 61
0x00000120 <+6>: in r29, 0x3e ; 62
0x00000122 <+8>: ldi r24, 0x23 ; 35
=> 0x00000124 <+10>: ldi r25, 0x01 ; 1
0x00000126 <+12>: sts 0x0201, r25
0x0000012a <+16>: sts 0x0200, r24
0x0000012e <+20>: lds r24, 0x0200
0x00000132 <+24>: lds r25, 0x0201
0x00000136 <+28>: adiw r24, 0x01 ; 1
0x00000138 <+30>: sts 0x0201, r25
0x0000013c <+34>: sts 0x0200, r24
0x00000140 <+38>: rjmp .-20 ; 0x12e <main+20>
End of assembler dump.
(gdb) si
0x00000126 5 a = 0x0123;
(gdb) p $r24
$1 = 0
(gdb) p $r25
$2 = 1
(gdb)
<snip>
> Is the simulator behaving differently from real hardware? Are existing
Yes, it seems. I checked atmega2560 device with avarice, avr-dragon and
avr-gdb. PC is 0x122 when target hits breakpoint, as expected.
> stubs rewinding the PC themselves, or using a different breakpoint
> instruction (by implementing the z/Z packets)?
Sorry, I don't understand what you mean by existing stubs.
Regards,
Pitchumani