From: Andrew Jones <drjones@redhat.com>
To: kvm@vger.kernel.org, kvm-ppc@vger.kernel.org
Cc: thuth@redhat.com, dgibson@redhat.com,
david@gibson.dropbear.id.au, agraf@suse.de, lvivier@redhat.com,
pbonzini@redhat.com, rkrcmar@redhat.com
Subject: [kvm-unit-tests PATCH v5 00/18] ppc64: initial drop
Date: Wed, 17 Feb 2016 19:40:41 +0100 [thread overview]
Message-ID: <1455734459-31902-1-git-send-email-drjones@redhat.com> (raw)
This series brings basic setup; starts a test's C entry point, main(),
and printf, exit, and malloc work. Three more series should follow this
one which must bring; vector support, mmu support, and smp support, at
which point I believe the framework could just evolve with the creation
of unit tests.
Tested on TCG and a P8 kvm_pr machine, and Laurent has tested it on
both a PowerMac G5 (kvm_pr) and a kvm_hv machine. I'm looking forward
to hearing more testing feedback from others though.
Patches also available here
https://github.com/rhdrjones/kvm-unit-tests/commits/ppc64/initial-drop-v5
v5:
- add vprintf [David & drew]
- move hcall and hcall_patch_broken_sc1 to assembler [David]
- get cache line size from DT [David]
(I left Thomas' r-b on "powerpc/ppc64: adapt arm's setup", but Thomas
will probably want to look again, as this change added a few lines
there. The commit message was updated to point out the changes.)
- rename uart_lock (as it's not a uart) [David]
- drop unused variables from powerpc makefiles
(LOADADDR, phys_base, kernel_offset) [David & drew]
- mkstandalone: only introduce a single firmware variable and set the
trap handler first [Radim]
- added more r-b's from David
v4:
- check for H_PRIVILEGE in broken sc1 probe [David]
- rename rtas' lock from lock to rtas_lock [Thomas]
- rtas: should only iterate to size/4 [drew]
- add powerpc/.gitignore (ignore boot_rom.bin) [drew]
v3:
Paolo's comments:
- got rid of zero-sized files
Thomas' review comments:
- remove useless casts in needed cast adding patch
- got rid of build_romfs building (and committing a binary), just
offset the rom binary by 256 bytes now
- added a better error message for a -smp N, N > NR_CPUS user error
- got rid of infinite loop with broken rtas-poweroff
- one r-b (wasn't sure the "looks good"'s counted, so didn't add more)
Laurent's testing led me to
- fix a problem on kvm_pr with broken sc1
- fix handling of memory regions, expect more than one
- and a fix to DT code so that works
Additionally:
- new patches
- cscope: ppc64 needs to look in lib/powerpc/asm too,
and arm64 in lib/arm/asm
- handle memory regions better also for arm
- a misc trivial fix for arm...
- align toc to 256 bytes for later gcc
v2:
Besides rebasing on latest master, v2 addresses all of David's comments
- assembler cleanup
- jump into the RTAS blob we get from DT, instead of reproducing it
- don't store the RTAS root node, always hunt it down
We still haven't addressed debug-exit. Plugging chr-testdev into an spapr
vty is probably what we should investigate first (as was suggested by Alex
Graf under the v1 review). For now we keep the hack from v1, but simplified.
Andrew Jones (18):
arm/arm64: trivial: another assert fix
Makefile: cscope: also look in arch shared asm
lib: asm-generic: add missing casts
devicetree: fix dt_get_memory_params
arm/arm64: setup improvements
lib: add vprintf
lib: share arm-selftest utility functions
config: no need to mix arch makefiles
powerpc/ppc64: start skeleton framework
powerpc/ppc64: ppc-ify makefiles and linker script
powerpc/ppc64: add a boot rom
powerpc/ppc64: add hcall support and putchar
powerpc/ppc64: adapt arm's setup
powerpc/ppc64: relocate linker VMAs
powerpc/ppc64: add run script and unittests.cfg
mkstandalone: add support for powerpc
powerpc/ppc64: add RTAS support
powerpc/ppc64: make a fake debug-exit
Makefile | 8 +-
README | 3 +-
arm/Makefile | 1 +
config/config-arm.mak => arm/Makefile.arm | 2 +-
config/config-arm64.mak => arm/Makefile.arm64 | 2 +-
.../config-arm-common.mak => arm/Makefile.common | 3 +-
arm/selftest.c | 45 ++-----
configure | 6 +
lib/arm/asm/setup.h | 9 ++
lib/arm/io.c | 3 +-
lib/arm/setup.c | 66 +++++++---
lib/asm-generic/io.h | 12 +-
lib/devicetree.c | 8 +-
lib/libcflat.h | 12 +-
lib/powerpc/.gitignore | 1 +
lib/powerpc/asm/hcall.h | 37 ++++++
lib/powerpc/asm/rtas.h | 26 ++++
lib/powerpc/asm/setup.h | 30 +++++
lib/powerpc/hcall.c | 33 +++++
lib/powerpc/io.c | 37 ++++++
lib/powerpc/rtas.c | 139 ++++++++++++++++++++
lib/powerpc/setup.c | 141 +++++++++++++++++++++
lib/ppc64/.gitignore | 1 +
lib/ppc64/asm-offsets.c | 12 ++
lib/ppc64/asm/asm-offsets.h | 1 +
lib/ppc64/asm/hcall.h | 1 +
lib/ppc64/asm/io.h | 5 +
lib/ppc64/asm/page.h | 1 +
lib/ppc64/asm/rtas.h | 1 +
lib/ppc64/asm/setup.h | 1 +
lib/ppc64/asm/spinlock.h | 11 ++
lib/ppc64/spinlock.c | 11 ++
lib/printf.c | 14 +-
lib/report.c | 18 ++-
lib/util.c | 18 +++
lib/util.h | 23 ++++
powerpc/.gitignore | 1 +
powerpc/Makefile | 1 +
powerpc/Makefile.common | 65 ++++++++++
powerpc/Makefile.ppc64 | 19 +++
powerpc/boot_rom.S | 5 +
powerpc/cstart64.S | 90 +++++++++++++
powerpc/flat.lds | 44 +++++++
powerpc/reloc64.c | 55 ++++++++
powerpc/run | 55 ++++++++
powerpc/selftest.c | 64 ++++++++++
powerpc/unittests.cfg | 30 +++++
{config => scripts}/asm-offsets.mak | 0
scripts/mkstandalone.sh | 5 +
x86/Makefile | 1 +
.../config-x86-common.mak => x86/Makefile.common | 0
config/config-i386.mak => x86/Makefile.i386 | 2 +-
config/config-x86_64.mak => x86/Makefile.x86_64 | 2 +-
53 files changed, 1102 insertions(+), 79 deletions(-)
create mode 100644 arm/Makefile
rename config/config-arm.mak => arm/Makefile.arm (90%)
rename config/config-arm64.mak => arm/Makefile.arm64 (89%)
rename config/config-arm-common.mak => arm/Makefile.common (96%)
create mode 100644 lib/powerpc/.gitignore
create mode 100644 lib/powerpc/asm/hcall.h
create mode 100644 lib/powerpc/asm/rtas.h
create mode 100644 lib/powerpc/asm/setup.h
create mode 100644 lib/powerpc/hcall.c
create mode 100644 lib/powerpc/io.c
create mode 100644 lib/powerpc/rtas.c
create mode 100644 lib/powerpc/setup.c
create mode 100644 lib/ppc64/.gitignore
create mode 100644 lib/ppc64/asm-offsets.c
create mode 100644 lib/ppc64/asm/asm-offsets.h
create mode 100644 lib/ppc64/asm/hcall.h
create mode 100644 lib/ppc64/asm/io.h
create mode 100644 lib/ppc64/asm/page.h
create mode 100644 lib/ppc64/asm/rtas.h
create mode 100644 lib/ppc64/asm/setup.h
create mode 100644 lib/ppc64/asm/spinlock.h
create mode 100644 lib/ppc64/spinlock.c
create mode 100644 lib/util.c
create mode 100644 lib/util.h
create mode 100644 powerpc/.gitignore
create mode 100644 powerpc/Makefile
create mode 100644 powerpc/Makefile.common
create mode 100644 powerpc/Makefile.ppc64
create mode 100644 powerpc/boot_rom.S
create mode 100644 powerpc/cstart64.S
create mode 100644 powerpc/flat.lds
create mode 100644 powerpc/reloc64.c
create mode 100755 powerpc/run
create mode 100644 powerpc/selftest.c
create mode 100644 powerpc/unittests.cfg
rename {config => scripts}/asm-offsets.mak (100%)
create mode 100644 x86/Makefile
rename config/config-x86-common.mak => x86/Makefile.common (100%)
rename config/config-i386.mak => x86/Makefile.i386 (91%)
rename config/config-x86_64.mak => x86/Makefile.x86_64 (93%)
--
2.4.3
next reply other threads:[~2016-02-17 18:41 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-17 18:40 Andrew Jones [this message]
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 01/18] arm/arm64: trivial: another assert fix Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 02/18] Makefile: cscope: also look in arch shared asm Andrew Jones
2016-02-18 2:36 ` David Gibson
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 03/18] lib: asm-generic: add missing casts Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 04/18] devicetree: fix dt_get_memory_params Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 05/18] arm/arm64: setup improvements Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 06/18] lib: add vprintf Andrew Jones
2016-02-18 2:44 ` David Gibson
2016-02-18 6:34 ` Thomas Huth
2016-02-18 10:21 ` Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 07/18] lib: share arm-selftest utility functions Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 08/18] config: no need to mix arch makefiles Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 09/18] powerpc/ppc64: start skeleton framework Andrew Jones
2016-02-19 2:11 ` David Gibson
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 10/18] powerpc/ppc64: ppc-ify makefiles and linker script Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 11/18] powerpc/ppc64: add a boot rom Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 12/18] powerpc/ppc64: add hcall support and putchar Andrew Jones
2016-02-19 3:10 ` David Gibson
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 13/18] powerpc/ppc64: adapt arm's setup Andrew Jones
2016-02-19 3:40 ` David Gibson
2016-02-19 7:49 ` Andrew Jones
2016-02-22 1:00 ` David Gibson
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 14/18] powerpc/ppc64: relocate linker VMAs Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 15/18] powerpc/ppc64: add run script and unittests.cfg Andrew Jones
2016-02-19 3:43 ` David Gibson
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 16/18] mkstandalone: add support for powerpc Andrew Jones
2016-02-18 14:51 ` Radim Krčmář
2016-02-19 3:53 ` David Gibson
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 17/18] powerpc/ppc64: add RTAS support Andrew Jones
2016-02-19 4:08 ` David Gibson
2016-02-19 7:52 ` Andrew Jones
2016-02-17 18:40 ` [kvm-unit-tests PATCH v5 18/18] powerpc/ppc64: make a fake debug-exit Andrew Jones
2016-02-19 4:12 ` David Gibson
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=1455734459-31902-1-git-send-email-drjones@redhat.com \
--to=drjones@redhat.com \
--cc=agraf@suse.de \
--cc=david@gibson.dropbear.id.au \
--cc=dgibson@redhat.com \
--cc=kvm-ppc@vger.kernel.org \
--cc=kvm@vger.kernel.org \
--cc=lvivier@redhat.com \
--cc=pbonzini@redhat.com \
--cc=rkrcmar@redhat.com \
--cc=thuth@redhat.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;
as well as URLs for NNTP newsgroup(s).