target/arm: Amend erroneous BRAA/BLRAA opcode check

[Qemu-devel] [PATCH 0/1] target/arm: Amend erroneous BRAA/BLRAA opcode check

Mathew Maidment posted 1 patch 5 years, 3 months ago

Download series mbox

Test docker-mingw@fedora passed

Test asan passed

Test docker-clang@ubuntu passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190130202243.79545-1-mathew1800@gmail.com

Maintainers: Peter Maydell <peter.maydell@linaro.org>

target/arm/translate-a64.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/1] target/arm: Amend erroneous BRAA/BLRAA opcode check

Posted by Mathew Maidment 5 years, 3 months ago

Hi,

This is a patch that fixes a condition within disas_uncond_b_reg() related to
BRAA and BLRAA that would always result in the unallocated encoding path being
taken.

Hopefully everything is in order. This is only my second patch, so if anything
is wrong, that's my bad.

Thanks in advance for reviewing =)

Mathew Maidment (1):
  target/arm: Amend erroneous BRAA/BLRAA opcode check

 target/arm/translate-a64.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.17.2 (Apple Git-113)

Re: [Qemu-devel] [PATCH 0/1] target/arm: Amend erroneous BRAA/BLRAA opcode check

Posted by Peter Maydell 5 years, 3 months ago

On Wed, 30 Jan 2019 at 20:22, Mathew Maidment <mathew1800@gmail.com> wrote:
> This is a patch that fixes a condition within disas_uncond_b_reg() related to
> BRAA and BLRAA that would always result in the unallocated encoding path being
> taken.
>
> Hopefully everything is in order. This is only my second patch, so if anything
> is wrong, that's my bad.
>
> Thanks in advance for reviewing =)

Hi -- thanks for this patch. I think this bug is already
covered by this patch that went onto the mailing list last week:
https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg06728.html
-- you might like to test that series.


thanks
-- PMM