Re: [PATCH] MicroBlaze: Update software breakpoint machine code as per MicroBlaze ISA

[Michael Eager <eager@eagercon.com>]

On 9/9/25 5:23 AM, Gopi Kumar Bulusu wrote:
> namaskaaram
> 
> On Tue, Sep 9, 2025 at 5:26 PM Michael Eager <eager@eagercon.com 
> <mailto:eager@eagercon.com>> wrote:
> 
>     On 9/9/25 12:16 AM, Gopi Kumar Bulusu wrote:
>      > namaskaaram,
>      > The macro MICROBLAZE_BREAKPOINT is set to an instruction opcode that
>      > does not meet the criteria for software breakpoint instruction as
>     set
>      > forth in the AMD MicroBlaze ISA. This patch  addresses the problem.
>      >
>      > This simple patch concludes a series of patches that add AMD
>     MicroBlaze
>      > gdbserver target support for GNU/Linux.
>      >
>      > Updated NEWS to include MicroBlaze gdbserver target.
>      >
>      > Build/Test - tested by building microblazeel-linux-gdb and verifying
>      > that software breakpoint works
>      > using gdbserver as target.
> 
> 
>     Has this patch been tested with bare metal targets?
> 
> 
> 1. Yes, This patch has been part of the AMD/Xilinx git sources (Yocto 
> builds) for many many years. Breakpoints
> work with bare metal (qemu).  From what I understand - the bare-metal 
> case when using QEMU does not use a
> software breakpoint.
> 
> 2. No, I have not checked Baremetal - real target, but when using BDM/ 
> JTAG this may not matter. However I don't see
> currently JTAG supported from gdb for MicroBlaze (does it ?).
> 
> 3. I also don't see any indication that gdb for MicroBlaze includes 
> support for  a remote debug monitor for bare metal targets
> that may use software breakpoint.
> 
> Approved ?

Please apply.

-- 
Michael Eager