[Qemu-devel] [PATCH v1 0/3] Some armv7m fixes

KONRAD Frederic posted 3 patches 6 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1498728533-23160-1-git-send-email-frederic.konrad@adacore.com
Test FreeBSD passed
Test checkpatch passed
Test docker failed
Test s390x passed
hw/timer/armv7m_systick.c |  3 +++
include/exec/memory.h     | 10 ++++++++++
memory.c                  | 22 ++++++++++++++++++++--
target/arm/cpu.c          | 14 ++++++++++++++
4 files changed, 47 insertions(+), 2 deletions(-)
[Qemu-devel] [PATCH v1 0/3] Some armv7m fixes
Posted by KONRAD Frederic 6 years, 10 months ago
Hi,

While playing with armv7m, I found two little bugs:
  - When there is an alias @0x00000000 to a flash memory the cpu state isn't
    reset correctly which leads later to an exception as ARM instruction-set is
    used. Presumably this bug might be present with the netduino2 board.
  - If the developer omits to set system_clock_rate we later go in a livelock
    when systick is triggered. Better aborting before to avoid the pain chasing
    the livelock.

Thanks,
Fred

KONRAD Frederic (3):
  add memory_region_get_offset_within_address_space
  arm: fix the armv7m reset state
  armv7m_systick: abort instead of locking on a bad rate

 hw/timer/armv7m_systick.c |  3 +++
 include/exec/memory.h     | 10 ++++++++++
 memory.c                  | 22 ++++++++++++++++++++--
 target/arm/cpu.c          | 14 ++++++++++++++
 4 files changed, 47 insertions(+), 2 deletions(-)

-- 
1.8.3.1


Re: [Qemu-devel] [PATCH v1 0/3] Some armv7m fixes
Posted by no-reply@patchew.org 6 years, 9 months ago
Hi,

This series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.

Subject: [Qemu-devel] [PATCH v1 0/3] Some armv7m fixes
Message-id: 1498728533-23160-1-git-send-email-frederic.konrad@adacore.com
Type: series

=== TEST SCRIPT BEGIN ===
#!/bin/bash
set -e
git submodule update --init dtc
# Let docker tests dump environment info
export SHOW_ENV=1
export J=8
time make docker-test-quick@centos6
time make docker-test-build@min-glib
time make docker-test-mingw@fedora
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
fatal: Cannot update paths and switch to branch 'test' at the same time.
Did you intend to checkout 'origin/patchew/1498728533-23160-1-git-send-email-frederic.konrad@adacore.com' which can not be resolved as commit?
Traceback (most recent call last):
  File "/home/fam/bin/patchew", line 440, in test_one
    git_clone_repo(clone, r["repo"], r["head"], logf)
  File "/home/fam/bin/patchew", line 53, in git_clone_repo
    cwd=clone)
  File "/usr/lib64/python3.5/subprocess.py", line 271, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['git', 'checkout', 'origin/patchew/1498728533-23160-1-git-send-email-frederic.konrad@adacore.com', '-b', 'test']' returned non-zero exit status 128



---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@freelists.org
Re: [Qemu-devel] [PATCH v1 0/3] Some armv7m fixes
Posted by Fam Zheng 6 years, 9 months ago
On Thu, 07/06 16:49, no-reply@patchew.org wrote:
> Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
> fatal: Cannot update paths and switch to branch 'test' at the same time.
> Did you intend to checkout 'origin/patchew/1498728533-23160-1-git-send-email-frederic.konrad@adacore.com' which can not be resolved as commit?
> Traceback (most recent call last):
>   File "/home/fam/bin/patchew", line 440, in test_one
>     git_clone_repo(clone, r["repo"], r["head"], logf)
>   File "/home/fam/bin/patchew", line 53, in git_clone_repo
>     cwd=clone)
>   File "/usr/lib64/python3.5/subprocess.py", line 271, in check_call
>     raise CalledProcessError(retcode, cmd)
> subprocess.CalledProcessError: Command '['git', 'checkout', 'origin/patchew/1498728533-23160-1-git-send-email-frederic.konrad@adacore.com', '-b', 'test']' returned non-zero exit status 128

Ignore this please, patchew is recovering from a bad state.