| Message ID | 20211112114734.3058678-1-alex.bennee@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | GIC ITS tests | expand |
On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: > Hi, > > Sorry this has been sitting in my tree so long. The changes are fairly > minor from v2. I no longer split the tests up into TCG and KVM > versions and instead just ensure that ERRATA_FORCE is always set when > run under TCG. > > Alex Bennée (3): > arm64: remove invalid check from its-trigger test > arm64: enable its-migration tests for TCG > arch-run: do not process ERRATA when running under TCG > > scripts/arch-run.bash | 4 +++- > arm/gic.c | 16 ++++++---------- > arm/unittests.cfg | 3 --- > 3 files changed, 9 insertions(+), 14 deletions(-) > > -- > 2.30.2 > > _______________________________________________ > kvmarm mailing list > kvmarm@lists.cs.columbia.edu > https://lists.cs.columbia.edu/mailman/listinfo/kvmarm Hi Alex, Thanks for this. I've applied to arm/queue, but I see that FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received consistently fails for me. Is that expected? Does it work for you? Thanks, drew
Andrew Jones <drjones@redhat.com> writes: > On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: >> Hi, >> >> Sorry this has been sitting in my tree so long. The changes are fairly >> minor from v2. I no longer split the tests up into TCG and KVM >> versions and instead just ensure that ERRATA_FORCE is always set when >> run under TCG. >> >> Alex Bennée (3): >> arm64: remove invalid check from its-trigger test >> arm64: enable its-migration tests for TCG >> arch-run: do not process ERRATA when running under TCG >> >> scripts/arch-run.bash | 4 +++- >> arm/gic.c | 16 ++++++---------- >> arm/unittests.cfg | 3 --- >> 3 files changed, 9 insertions(+), 14 deletions(-) >> >> -- >> 2.30.2 >> >> _______________________________________________ >> kvmarm mailing list >> kvmarm@lists.cs.columbia.edu >> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm > > Hi Alex, > > Thanks for this. I've applied to arm/queue, but I see that > > FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received > > consistently fails for me. Is that expected? Does it work for you? doh - looks like I cocked up the merge conflict... Did it fail for TCG or for KVM (or both)?
Andrew Jones <drjones@redhat.com> writes: > On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: >> Hi, >> >> Sorry this has been sitting in my tree so long. The changes are fairly >> minor from v2. I no longer split the tests up into TCG and KVM >> versions and instead just ensure that ERRATA_FORCE is always set when >> run under TCG. >> >> Alex Bennée (3): >> arm64: remove invalid check from its-trigger test >> arm64: enable its-migration tests for TCG >> arch-run: do not process ERRATA when running under TCG >> >> scripts/arch-run.bash | 4 +++- >> arm/gic.c | 16 ++++++---------- >> arm/unittests.cfg | 3 --- >> 3 files changed, 9 insertions(+), 14 deletions(-) >> >> -- >> 2.30.2 >> >> _______________________________________________ >> kvmarm mailing list >> kvmarm@lists.cs.columbia.edu >> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm > > Hi Alex, > > Thanks for this. I've applied to arm/queue, but I see that > > FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received > > consistently fails for me. Is that expected? Does it work for you? Hmm so it works with KVM so I suspect it's showing up an error in the TCG ITS implementation which I didn't see in earlier runs. > > Thanks, > drew
On Fri, Nov 12, 2021 at 02:08:01PM +0000, Alex Bennée wrote: > > Andrew Jones <drjones@redhat.com> writes: > > > On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: > >> Hi, > >> > >> Sorry this has been sitting in my tree so long. The changes are fairly > >> minor from v2. I no longer split the tests up into TCG and KVM > >> versions and instead just ensure that ERRATA_FORCE is always set when > >> run under TCG. > >> > >> Alex Bennée (3): > >> arm64: remove invalid check from its-trigger test > >> arm64: enable its-migration tests for TCG > >> arch-run: do not process ERRATA when running under TCG > >> > >> scripts/arch-run.bash | 4 +++- > >> arm/gic.c | 16 ++++++---------- > >> arm/unittests.cfg | 3 --- > >> 3 files changed, 9 insertions(+), 14 deletions(-) > >> > >> -- > >> 2.30.2 > >> > >> _______________________________________________ > >> kvmarm mailing list > >> kvmarm@lists.cs.columbia.edu > >> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm > > > > Hi Alex, > > > > Thanks for this. I've applied to arm/queue, but I see that > > > > FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received > > > > consistently fails for me. Is that expected? Does it work for you? > > doh - looks like I cocked up the merge conflict... > > Did it fail for TCG or for KVM (or both)? Just TCG, which was why I was wondering if it was expected. I've never run these tests with TCG before. Thanks, drew
Andrew Jones <drjones@redhat.com> writes: > On Fri, Nov 12, 2021 at 02:08:01PM +0000, Alex Bennée wrote: >> >> Andrew Jones <drjones@redhat.com> writes: >> >> > On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: >> >> Hi, >> >> >> >> Sorry this has been sitting in my tree so long. The changes are fairly >> >> minor from v2. I no longer split the tests up into TCG and KVM >> >> versions and instead just ensure that ERRATA_FORCE is always set when >> >> run under TCG. >> >> >> >> Alex Bennée (3): >> >> arm64: remove invalid check from its-trigger test >> >> arm64: enable its-migration tests for TCG >> >> arch-run: do not process ERRATA when running under TCG >> >> >> >> scripts/arch-run.bash | 4 +++- >> >> arm/gic.c | 16 ++++++---------- >> >> arm/unittests.cfg | 3 --- >> >> 3 files changed, 9 insertions(+), 14 deletions(-) >> >> >> >> -- >> >> 2.30.2 >> >> >> >> _______________________________________________ >> >> kvmarm mailing list >> >> kvmarm@lists.cs.columbia.edu >> >> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm >> > >> > Hi Alex, >> > >> > Thanks for this. I've applied to arm/queue, but I see that >> > >> > FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received >> > >> > consistently fails for me. Is that expected? Does it work for you? >> >> doh - looks like I cocked up the merge conflict... >> >> Did it fail for TCG or for KVM (or both)? > > Just TCG, which was why I was wondering if it was expected. I've never run > these tests with TCG before. Hmm I think expecting the IRQ at all is broken so I think I should delete the whole pending test. > > Thanks, > drew
On Fri, Nov 19, 2021 at 04:30:47PM +0000, Alex Bennée wrote: > > Andrew Jones <drjones@redhat.com> writes: > > > On Fri, Nov 12, 2021 at 02:08:01PM +0000, Alex Bennée wrote: > >> > >> Andrew Jones <drjones@redhat.com> writes: > >> > >> > On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: > >> >> Hi, > >> >> > >> >> Sorry this has been sitting in my tree so long. The changes are fairly > >> >> minor from v2. I no longer split the tests up into TCG and KVM > >> >> versions and instead just ensure that ERRATA_FORCE is always set when > >> >> run under TCG. > >> >> > >> >> Alex Bennée (3): > >> >> arm64: remove invalid check from its-trigger test > >> >> arm64: enable its-migration tests for TCG > >> >> arch-run: do not process ERRATA when running under TCG > >> >> > >> >> scripts/arch-run.bash | 4 +++- > >> >> arm/gic.c | 16 ++++++---------- > >> >> arm/unittests.cfg | 3 --- > >> >> 3 files changed, 9 insertions(+), 14 deletions(-) > >> >> > >> >> -- > >> >> 2.30.2 > >> >> > >> >> _______________________________________________ > >> >> kvmarm mailing list > >> >> kvmarm@lists.cs.columbia.edu > >> >> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm > >> > > >> > Hi Alex, > >> > > >> > Thanks for this. I've applied to arm/queue, but I see that > >> > > >> > FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received > >> > > >> > consistently fails for me. Is that expected? Does it work for you? > >> > >> doh - looks like I cocked up the merge conflict... > >> > >> Did it fail for TCG or for KVM (or both)? > > > > Just TCG, which was why I was wondering if it was expected. I've never run > > these tests with TCG before. > > Hmm I think expecting the IRQ at all is broken so I think I should > delete the whole pending test. Feel free to repost. I'll update the patches in arm/queue before my next MR. Thanks, drew
Andrew Jones <drjones@redhat.com> writes: > On Fri, Nov 19, 2021 at 04:30:47PM +0000, Alex Bennée wrote: >> >> Andrew Jones <drjones@redhat.com> writes: >> >> > On Fri, Nov 12, 2021 at 02:08:01PM +0000, Alex Bennée wrote: >> >> >> >> Andrew Jones <drjones@redhat.com> writes: >> >> >> >> > On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: >> >> >> Hi, >> >> >> >> >> >> Sorry this has been sitting in my tree so long. The changes are fairly >> >> >> minor from v2. I no longer split the tests up into TCG and KVM >> >> >> versions and instead just ensure that ERRATA_FORCE is always set when >> >> >> run under TCG. >> >> >> >> >> >> Alex Bennée (3): >> >> >> arm64: remove invalid check from its-trigger test >> >> >> arm64: enable its-migration tests for TCG >> >> >> arch-run: do not process ERRATA when running under TCG >> >> >> >> >> >> scripts/arch-run.bash | 4 +++- >> >> >> arm/gic.c | 16 ++++++---------- >> >> >> arm/unittests.cfg | 3 --- >> >> >> 3 files changed, 9 insertions(+), 14 deletions(-) >> >> >> >> >> >> -- >> >> >> 2.30.2 >> >> >> >> >> >> _______________________________________________ >> >> >> kvmarm mailing list >> >> >> kvmarm@lists.cs.columbia.edu >> >> >> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm >> >> > >> >> > Hi Alex, >> >> > >> >> > Thanks for this. I've applied to arm/queue, but I see that >> >> > >> >> > FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received >> >> > >> >> > consistently fails for me. Is that expected? Does it work for you? >> >> >> >> doh - looks like I cocked up the merge conflict... >> >> >> >> Did it fail for TCG or for KVM (or both)? >> > >> > Just TCG, which was why I was wondering if it was expected. I've never run >> > these tests with TCG before. >> >> Hmm I think expecting the IRQ at all is broken so I think I should >> delete the whole pending test. > > Feel free to repost. I'll update the patches in arm/queue before my next > MR. Actually I think the problem was with a regression in the TCG ITS support (now fixed in master). So I believe as of v3 everything is correct (and v4 should be ignored). Are you happy to apply this series or do you want me to repost it as v5? > > Thanks, > drew
On Tue, Nov 30, 2021 at 02:11:34PM +0000, Alex Bennée wrote: > > Andrew Jones <drjones@redhat.com> writes: > > > On Fri, Nov 19, 2021 at 04:30:47PM +0000, Alex Bennée wrote: > >> > >> Andrew Jones <drjones@redhat.com> writes: > >> > >> > On Fri, Nov 12, 2021 at 02:08:01PM +0000, Alex Bennée wrote: > >> >> > >> >> Andrew Jones <drjones@redhat.com> writes: > >> >> > >> >> > On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: > >> >> >> Hi, > >> >> >> > >> >> >> Sorry this has been sitting in my tree so long. The changes are fairly > >> >> >> minor from v2. I no longer split the tests up into TCG and KVM > >> >> >> versions and instead just ensure that ERRATA_FORCE is always set when > >> >> >> run under TCG. > >> >> >> > >> >> >> Alex Bennée (3): > >> >> >> arm64: remove invalid check from its-trigger test > >> >> >> arm64: enable its-migration tests for TCG > >> >> >> arch-run: do not process ERRATA when running under TCG > >> >> >> > >> >> >> scripts/arch-run.bash | 4 +++- > >> >> >> arm/gic.c | 16 ++++++---------- > >> >> >> arm/unittests.cfg | 3 --- > >> >> >> 3 files changed, 9 insertions(+), 14 deletions(-) > >> >> >> > >> >> >> -- > >> >> >> 2.30.2 > >> >> >> > >> >> >> _______________________________________________ > >> >> >> kvmarm mailing list > >> >> >> kvmarm@lists.cs.columbia.edu > >> >> >> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm > >> >> > > >> >> > Hi Alex, > >> >> > > >> >> > Thanks for this. I've applied to arm/queue, but I see that > >> >> > > >> >> > FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received > >> >> > > >> >> > consistently fails for me. Is that expected? Does it work for you? > >> >> > >> >> doh - looks like I cocked up the merge conflict... > >> >> > >> >> Did it fail for TCG or for KVM (or both)? > >> > > >> > Just TCG, which was why I was wondering if it was expected. I've never run > >> > these tests with TCG before. > >> > >> Hmm I think expecting the IRQ at all is broken so I think I should > >> delete the whole pending test. > > > > Feel free to repost. I'll update the patches in arm/queue before my next > > MR. > > Actually I think the problem was with a regression in the TCG ITS > support (now fixed in master). So I believe as of v3 everything is > correct (and v4 should be ignored). > > Are you happy to apply this series or do you want me to repost it as v5? No need to repost. I'll retest v3 with latest QEMU. Thanks, drew
Andrew Jones <drjones@redhat.com> writes: > On Tue, Nov 30, 2021 at 02:11:34PM +0000, Alex Bennée wrote: >> >> Andrew Jones <drjones@redhat.com> writes: >> >> > On Fri, Nov 19, 2021 at 04:30:47PM +0000, Alex Bennée wrote: >> >> >> >> Andrew Jones <drjones@redhat.com> writes: >> >> >> >> > On Fri, Nov 12, 2021 at 02:08:01PM +0000, Alex Bennée wrote: >> >> >> >> >> >> Andrew Jones <drjones@redhat.com> writes: >> >> >> >> >> >> > On Fri, Nov 12, 2021 at 11:47:31AM +0000, Alex Bennée wrote: >> >> >> >> Hi, >> >> >> >> >> >> >> >> Sorry this has been sitting in my tree so long. The changes are fairly >> >> >> >> minor from v2. I no longer split the tests up into TCG and KVM >> >> >> >> versions and instead just ensure that ERRATA_FORCE is always set when >> >> >> >> run under TCG. >> >> >> >> >> >> >> >> Alex Bennée (3): >> >> >> >> arm64: remove invalid check from its-trigger test >> >> >> >> arm64: enable its-migration tests for TCG >> >> >> >> arch-run: do not process ERRATA when running under TCG >> >> >> >> >> >> >> >> scripts/arch-run.bash | 4 +++- >> >> >> >> arm/gic.c | 16 ++++++---------- >> >> >> >> arm/unittests.cfg | 3 --- >> >> >> >> 3 files changed, 9 insertions(+), 14 deletions(-) >> >> >> >> >> >> >> >> -- >> >> >> >> 2.30.2 >> >> >> >> >> >> >> >> _______________________________________________ >> >> >> >> kvmarm mailing list >> >> >> >> kvmarm@lists.cs.columbia.edu >> >> >> >> https://lists.cs.columbia.edu/mailman/listinfo/kvmarm >> >> >> > >> >> >> > Hi Alex, >> >> >> > >> >> >> > Thanks for this. I've applied to arm/queue, but I see that >> >> >> > >> >> >> > FAIL: gicv3: its-trigger: inv/invall: dev2/eventid=20 pending LPI is received >> >> >> > >> >> >> > consistently fails for me. Is that expected? Does it work for you? >> >> >> >> >> >> doh - looks like I cocked up the merge conflict... >> >> >> >> >> >> Did it fail for TCG or for KVM (or both)? >> >> > >> >> > Just TCG, which was why I was wondering if it was expected. I've never run >> >> > these tests with TCG before. >> >> >> >> Hmm I think expecting the IRQ at all is broken so I think I should >> >> delete the whole pending test. >> > >> > Feel free to repost. I'll update the patches in arm/queue before my next >> > MR. >> >> Actually I think the problem was with a regression in the TCG ITS >> support (now fixed in master). So I believe as of v3 everything is >> correct (and v4 should be ignored). >> >> Are you happy to apply this series or do you want me to repost it as v5? > > No need to repost. I'll retest v3 with latest QEMU. Gentle ping, I'm trying to clear this off my internal JIRA so let me know if you want me to do anything to help. > > Thanks, > drew
On Tue, Feb 01, 2022 at 01:10:13PM +0000, Alex Bennée wrote: > > Gentle ping, I'm trying to clear this off my internal JIRA so let me > know if you want me to do anything to help. > Sorry Alex! I've been juggling too many balls lately and completely dropped this one. I'll rebase arm/queue now and run it through some sanity tests. If all it good, I'll do the MR right away. Thanks, drew