Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Pedro Alves <palves@redhat.com>
To: Tristan Gingold <gingold@adacore.com>,
	       Christopher Friedt <chrisfriedt@gmail.com>
Cc: gdb-patches@sourceware.org
Subject: Re: cortex-m xml register descriptions for m-system
Date: Tue, 15 Dec 2015 12:11:00 -0000	[thread overview]
Message-ID: <56700356.8070706@redhat.com> (raw)
In-Reply-To: <DA04697A-0269-4CCB-B40D-8C727E696C64@adacore.com>

On 12/15/2015 11:44 AM, Tristan Gingold wrote:
> 
>> On 15 Dec 2015, at 00:11, Christopher Friedt <chrisfriedt@gmail.com> wrote:
>>
>> On Dec 14, 2015 1:55 PM, "Pedro Alves" <palves@redhat.com> wrote:
>>>
>>> On 12/14/2015 05:04 PM, Christopher Friedt wrote:
>>>> Hi list,
>>>>
>>>> I've been using GDB and OpenOCD to debug ARM Cortex-M devices for
>>>> quite a while. One thing that I always noticed when using OpenOCD is
>>>> that the m-system registers are listed, which is *incredibly* useful
>>>> for writing code on just about any Cortex-M microcontroller.
>>>>
>>>> Somewhat recently, Qemu has also begun to support Cortex-M based
>>>> virtual devices, and it seems to be fairly usable.
>>>>
>>>> The down side, is that they do not expose the m-system registers,
>>>> simply because binutils-gdb does not (at this time) have an XML file
>>>> for them.
>>>>
>>>> Just to catch anyone up to speed who might be reading this, the
>>>> m-system registers are
>>>>
>>>> MSP (main stack pointer)
>>>> PSP (process stack pointer)
>>>> PRIMASK (1-bit register that says if interrupts are enabled)
>>>> BASEPRI (8-bit register that sets the NVIC base priority)
>>>> FAULTMASK (1-bit register that says if fault interrupts are enabled)
>>>> CONTROL (3-bit register that indicates presence of FP, whether PSP is
>>>> selected, and whether running in unprivileged mode)
>>>>
>>>> Now, these are "system" registers, and on a full blown microprocessor,
>>>> it might be unusual to expose them, but on a microcontroller, it's
>>>> quite important. The other debuggers that I have seen (IAR,
>>>> specifically) also list the m-system registers along with the general
>>>> purpose ones for Cortex-M.
>>>>
>>>> The following XML is sufficient to describe the m-system registers so
>>>> that they appear to the GDB client.
>>>>
>>>> <feature name="org.gnu.gdb.arm.m-system">
>>>>  <reg name="msp" bitsize="32" type="data_ptr"/>
>>>>  <reg name="psp" bitsize="32" type="data_ptr"/>
>>>>  <reg name="primask" bitsize="1" type="int8"/>
>>>>  <reg name="basepri" bitsize="8" type="int8"/>
>>>>  <reg name="faultmask" bitsize="1" type="int8"/>
>>>>  <reg name="control" bitsize="3" type="int8"/>
>>>> </feature>
>>>
>>> Does GDB need to be aware of these registers at all?  That is, does gdb
>>> need to be aware of org.gnu.gdb.arm.m-system?  Usually GDB needs to
>>> be aware of specific registers if for instance Dwarf can refer to them.
>>> Otherwise, the design of xml descriptions is such that you're free
>>> to send any additional registers you want without a specific feature.
>>> GDB will show them.
>>
>> Hmm... It's hard for me to say. The MSP and PSP are banked stack
>> pointers, control instructs the core which stack pointer to use, and
>> they are also tightly coupled to exception entry, so I would lean
>> towards yes?
> 
> I do think so too.
> 
> I have just written a patch so that gdb unwinds correctly on cortex-m
> exceptions, and this of course requires that gdb knows about at least
> psp.
> 
> I plan to submit it early January.
> 
> Tristan.
> 


Thanks,
Pedro Alves


  parent reply	other threads:[~2015-12-15 12:11 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-14 17:05 Christopher Friedt
2015-12-14 18:55 ` Pedro Alves
2015-12-14 23:11   ` Christopher Friedt
2015-12-15  0:13     ` Pedro Alves
2015-12-15 15:35       ` Christopher Friedt
2015-12-16 13:51         ` Tristan Gingold
2015-12-16 17:13           ` Christopher Friedt
2015-12-17  8:32             ` Tristan Gingold
2018-08-14 18:14               ` Christopher Friedt
2015-12-15 11:44     ` Tristan Gingold
2015-12-15 11:59       ` Pedro Alves
2015-12-15 12:11       ` Pedro Alves [this message]
2015-12-15 12:13         ` Pedro Alves
2015-12-15  8:55 ` Yao Qi
2015-12-15 10:25   ` Pedro Alves
2015-12-15 10:32     ` Pedro Alves
2015-12-16  9:49     ` Yao Qi
2015-12-15  9:13 ` Yao Qi
2015-12-15 12:20   ` Christopher Friedt
2015-12-15 15:09     ` Christopher Friedt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56700356.8070706@redhat.com \
    --to=palves@redhat.com \
    --cc=chrisfriedt@gmail.com \
    --cc=gdb-patches@sourceware.org \
    --cc=gingold@adacore.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox