| Message ID | 20200309102420.24498-1-eric.auger@redhat.com |
|---|---|
| Headers | show |
| Series | arm/arm64: Add ITS tests | expand |
On Mon, Mar 09, 2020 at 11:24:07AM +0100, Eric Auger wrote: > This series is a revival of an RFC series sent in Dec 2016 [1]. > Given the amount of code and the lack of traction at that time, > I haven't respinned until now. However a recent bug found related > to the ITS migration convinced me that this work may deserve to be > respinned and enhanced. > > Tests exercise main ITS commands and also test migration. > With the migration framework, we are able to trigger the > migration from guest and that is very practical actually. > > What is particular with the ITS programming is that most of > the commands are passed through queues and there is real error > handling. Invalid commands are just ignored and that is not > really tester friendly. > > The series can be fount at: > https://github.com/eauger/kut/tree/its-v4 > > Best Regards > > Eric > > History: > v3 -> v4: > - addressed comments from Drew and Zenghui > - added "page_alloc: Introduce get_order()" > - removed "arm: gic: Provide per-IRQ helper functions" > - ITS files moved to lib64 > - and many more, see individual logs > > v2 -> v3: > - fix 32b compilation > - take into account Drew's comments (see individual diff logs) > > v1 -> v2: > - took into account Zenghui's comments > - collect R-b's from Thomas > > References: > [1] [kvm-unit-tests RFC 00/15] arm/arm64: add ITS framework > https://lists.gnu.org/archive/html/qemu-devel/2016-12/msg00575.html > > Execution: > x For other ITS tests: > ./run_tests.sh -g its > > x non migration tests can be launched invidually. For instance: > ./arm-run arm/gic.flat -smp 8 -append 'its-trigger' > > Eric Auger (13): > libcflat: Add other size defines > page_alloc: Introduce get_order() > arm/arm64: gic: Introduce setup_irq() helper > arm/arm64: gicv3: Add some re-distributor defines > arm/arm64: gicv3: Set the LPI config and pending tables > arm/arm64: ITS: Introspection tests > arm/arm64: ITS: its_enable_defaults > arm/arm64: ITS: Device and collection Initialization > arm/arm64: ITS: Commands > arm/arm64: ITS: INT functional tests > arm/run: Allow Migration tests > arm/arm64: ITS: migration tests > arm/arm64: ITS: pending table migration test > > arm/Makefile.arm64 | 1 + > arm/Makefile.common | 2 +- > arm/gic.c | 469 +++++++++++++++++++++++++++++++++++-- > arm/run | 2 +- > arm/unittests.cfg | 38 +++ > lib/alloc_page.c | 7 +- > lib/alloc_page.h | 1 + > lib/arm/asm/gic-v3-its.h | 39 +++ > lib/arm/asm/gic-v3.h | 32 +++ > lib/arm/gic-v3.c | 76 ++++++ > lib/arm/gic.c | 34 ++- > lib/arm/io.c | 28 +++ > lib/arm64/asm/gic-v3-its.h | 172 ++++++++++++++ > lib/arm64/gic-v3-its-cmd.c | 463 ++++++++++++++++++++++++++++++++++++ > lib/arm64/gic-v3-its.c | 172 ++++++++++++++ > lib/libcflat.h | 3 + > 16 files changed, 1510 insertions(+), 29 deletions(-) > create mode 100644 lib/arm/asm/gic-v3-its.h > create mode 100644 lib/arm64/asm/gic-v3-its.h > create mode 100644 lib/arm64/gic-v3-its-cmd.c > create mode 100644 lib/arm64/gic-v3-its.c > > -- > 2.20.1 > > This looks pretty good to me. It just needs some resquashing cleanups. Does Andre plan to review? I've only been reviewing with respect to the framework, not ITS. If no other reviews are expected, then I'll queue the next version. Thanks, drew
On Mon, Mar 09, 2020 at 12:57:51PM +0100, Andrew Jones wrote: > This looks pretty good to me. It just needs some resquashing cleanups. > Does Andre plan to review? I've only been reviewing with respect to > the framework, not ITS. If no other reviews are expected, then I'll > queue the next version. Oops, sorry Zenghui, I forgot to ask if you'll be reviewing again as well. Thanks, drew
On 2020/3/9 20:00, Andrew Jones wrote: > On Mon, Mar 09, 2020 at 12:57:51PM +0100, Andrew Jones wrote: >> This looks pretty good to me. It just needs some resquashing cleanups. >> Does Andre plan to review? I've only been reviewing with respect to >> the framework, not ITS. If no other reviews are expected, then I'll >> queue the next version. > > Oops, sorry Zenghui, I forgot to ask if you'll be reviewing again as > well. I'd like to have a look again this week if time permits :-). Thanks for reminding me. Zenghui
Hi Zenghui, On 3/10/20 11:18 AM, Zenghui Yu wrote: > On 2020/3/9 20:00, Andrew Jones wrote: >> On Mon, Mar 09, 2020 at 12:57:51PM +0100, Andrew Jones wrote: >>> This looks pretty good to me. It just needs some resquashing cleanups. >>> Does Andre plan to review? I've only been reviewing with respect to >>> the framework, not ITS. If no other reviews are expected, then I'll >>> queue the next version. >> >> Oops, sorry Zenghui, I forgot to ask if you'll be reviewing again as >> well. > > I'd like to have a look again this week if time permits :-). Thanks > for reminding me. I have just sent: [kvm-unit-tests PATCH v5 00/13] arm/arm64: Add ITS tests where I took last Drew's comments Thanks Eric > > > Zenghui >
This series is a revival of an RFC series sent in Dec 2016 [1]. Given the amount of code and the lack of traction at that time, I haven't respinned until now. However a recent bug found related to the ITS migration convinced me that this work may deserve to be respinned and enhanced. Tests exercise main ITS commands and also test migration. With the migration framework, we are able to trigger the migration from guest and that is very practical actually. What is particular with the ITS programming is that most of the commands are passed through queues and there is real error handling. Invalid commands are just ignored and that is not really tester friendly. The series can be fount at: https://github.com/eauger/kut/tree/its-v4 Best Regards Eric History: v3 -> v4: - addressed comments from Drew and Zenghui - added "page_alloc: Introduce get_order()" - removed "arm: gic: Provide per-IRQ helper functions" - ITS files moved to lib64 - and many more, see individual logs v2 -> v3: - fix 32b compilation - take into account Drew's comments (see individual diff logs) v1 -> v2: - took into account Zenghui's comments - collect R-b's from Thomas References: [1] [kvm-unit-tests RFC 00/15] arm/arm64: add ITS framework https://lists.gnu.org/archive/html/qemu-devel/2016-12/msg00575.html Execution: x For other ITS tests: ./run_tests.sh -g its x non migration tests can be launched invidually. For instance: ./arm-run arm/gic.flat -smp 8 -append 'its-trigger' Eric Auger (13): libcflat: Add other size defines page_alloc: Introduce get_order() arm/arm64: gic: Introduce setup_irq() helper arm/arm64: gicv3: Add some re-distributor defines arm/arm64: gicv3: Set the LPI config and pending tables arm/arm64: ITS: Introspection tests arm/arm64: ITS: its_enable_defaults arm/arm64: ITS: Device and collection Initialization arm/arm64: ITS: Commands arm/arm64: ITS: INT functional tests arm/run: Allow Migration tests arm/arm64: ITS: migration tests arm/arm64: ITS: pending table migration test arm/Makefile.arm64 | 1 + arm/Makefile.common | 2 +- arm/gic.c | 469 +++++++++++++++++++++++++++++++++++-- arm/run | 2 +- arm/unittests.cfg | 38 +++ lib/alloc_page.c | 7 +- lib/alloc_page.h | 1 + lib/arm/asm/gic-v3-its.h | 39 +++ lib/arm/asm/gic-v3.h | 32 +++ lib/arm/gic-v3.c | 76 ++++++ lib/arm/gic.c | 34 ++- lib/arm/io.c | 28 +++ lib/arm64/asm/gic-v3-its.h | 172 ++++++++++++++ lib/arm64/gic-v3-its-cmd.c | 463 ++++++++++++++++++++++++++++++++++++ lib/arm64/gic-v3-its.c | 172 ++++++++++++++ lib/libcflat.h | 3 + 16 files changed, 1510 insertions(+), 29 deletions(-) create mode 100644 lib/arm/asm/gic-v3-its.h create mode 100644 lib/arm64/asm/gic-v3-its.h create mode 100644 lib/arm64/gic-v3-its-cmd.c create mode 100644 lib/arm64/gic-v3-its.c