riscv: Add the support for c.ebreak check in is_valid_bugaddr()

Browse source

The macro __BUG_INSN currently is defined as the "ebreak" opcode.
The is_valid_bugaddr() function compares the instruction pointed to by
$sepc with macro __BUG_INSN to check whether the current trap exception
is caused by an "ebreak" instruction. However, this check flow is possibly
erroneous because if C extension is supported, the expected trap
instruction "ebreak" is possibly translated to "c.ebreak" by the assembler.
Therefore, it requires a mechanism to distinguish the length of the
instruction in $spec and compare it to the correct trap instruction.

Signed-off-by: Vincent Chen <vincentc@andestech.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Palmer Dabbelt <palmer@sifive.com>

...

This commit is contained in:

Vincent Chen 2019-03-05 11:23:34 +08:00 • committed by Palmer Dabbelt

parent d18ebc274c

commit ee72e0e70c

No known key found for this signature in database

GPG key ID: EF4CA1502CCBAB41

2 changed files with 23 additions and 4 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

7

arch/riscv/include/asm/bug.h

View file

20

arch/riscv/kernel/traps.c

View file