From: Ricardo Koller <ricarkol@google.com> To: kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu, drjones@redhat.com Cc: maz@kernel.org, Paolo Bonzini <pbonzini@redhat.com>, oupton@google.com, reijiw@google.com, Ricardo Koller <ricarkol@google.com> Subject: [PATCH v2 0/5] kvm: selftests: aarch64: some fixes for vgic_irq Date: Wed, 26 Jan 2022 19:08:53 -0800 [thread overview] Message-ID: <20220127030858.3269036-1-ricarkol@google.com> (raw) Reiji discovered multiple issues with the vgic_irq series [0]: 1. there's an assert that needs fixing. 2. some guest arguments are not set correctly. 3. the failure test in kvm_set_gsi_routing_irqchip_check is wrong. 4. there are lots of comments that use the wrong formatting. 5. vgic_poke_irq() could use a tighter assert check. The first 3 issues above are critical, the last 2 would be nice to have. I haven't hit the failed assert (1.), but just by chance: my compiler is initializing the respective local variable to 0. The second issue (2.) leads to not testing one of the injection methods (irqfd). The third issue could be hit if we tested more intids. v1 -> v2: - adding 3 more fixes: 2, 3, 5 above. (Reiji) - corrected the comments in 4 above. (Andrew) - dded drjones@ reviewed-by tag. [0] https://lore.kernel.org/kvmarm/164072141023.1027791.3183483860602648119.b4-ty@kernel.org/ Ricardo Koller (5): kvm: selftests: aarch64: fix assert in gicv3_access_reg kvm: selftests: aarch64: pass vgic_irq guest args as a pointer kvm: selftests: aarch64: fix the failure check in kvm_set_gsi_routing_irqchip_check kvm: selftests: aarch64: fix some vgic related comments kvm: selftests: aarch64: use a tighter assert in vgic_poke_irq() .../testing/selftests/kvm/aarch64/vgic_irq.c | 45 +++++++++++-------- .../selftests/kvm/lib/aarch64/gic_v3.c | 12 ++--- .../testing/selftests/kvm/lib/aarch64/vgic.c | 9 ++-- 3 files changed, 38 insertions(+), 28 deletions(-) -- 2.35.0.rc0.227.g00780c9af4-goog
WARNING: multiple messages have this Message-ID (diff)
From: Ricardo Koller <ricarkol@google.com> To: kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu, drjones@redhat.com Cc: maz@kernel.org, Paolo Bonzini <pbonzini@redhat.com> Subject: [PATCH v2 0/5] kvm: selftests: aarch64: some fixes for vgic_irq Date: Wed, 26 Jan 2022 19:08:53 -0800 [thread overview] Message-ID: <20220127030858.3269036-1-ricarkol@google.com> (raw) Reiji discovered multiple issues with the vgic_irq series [0]: 1. there's an assert that needs fixing. 2. some guest arguments are not set correctly. 3. the failure test in kvm_set_gsi_routing_irqchip_check is wrong. 4. there are lots of comments that use the wrong formatting. 5. vgic_poke_irq() could use a tighter assert check. The first 3 issues above are critical, the last 2 would be nice to have. I haven't hit the failed assert (1.), but just by chance: my compiler is initializing the respective local variable to 0. The second issue (2.) leads to not testing one of the injection methods (irqfd). The third issue could be hit if we tested more intids. v1 -> v2: - adding 3 more fixes: 2, 3, 5 above. (Reiji) - corrected the comments in 4 above. (Andrew) - dded drjones@ reviewed-by tag. [0] https://lore.kernel.org/kvmarm/164072141023.1027791.3183483860602648119.b4-ty@kernel.org/ Ricardo Koller (5): kvm: selftests: aarch64: fix assert in gicv3_access_reg kvm: selftests: aarch64: pass vgic_irq guest args as a pointer kvm: selftests: aarch64: fix the failure check in kvm_set_gsi_routing_irqchip_check kvm: selftests: aarch64: fix some vgic related comments kvm: selftests: aarch64: use a tighter assert in vgic_poke_irq() .../testing/selftests/kvm/aarch64/vgic_irq.c | 45 +++++++++++-------- .../selftests/kvm/lib/aarch64/gic_v3.c | 12 ++--- .../testing/selftests/kvm/lib/aarch64/vgic.c | 9 ++-- 3 files changed, 38 insertions(+), 28 deletions(-) -- 2.35.0.rc0.227.g00780c9af4-goog _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
next reply other threads:[~2022-01-27 3:09 UTC|newest] Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-01-27 3:08 Ricardo Koller [this message] 2022-01-27 3:08 ` [PATCH v2 0/5] kvm: selftests: aarch64: some fixes for vgic_irq Ricardo Koller 2022-01-27 3:08 ` [PATCH v2 1/5] kvm: selftests: aarch64: fix assert in gicv3_access_reg Ricardo Koller 2022-01-27 3:08 ` Ricardo Koller 2022-01-27 7:34 ` Andrew Jones 2022-01-27 7:34 ` Andrew Jones 2022-01-27 3:08 ` [PATCH v2 2/5] kvm: selftests: aarch64: pass vgic_irq guest args as a pointer Ricardo Koller 2022-01-27 3:08 ` Ricardo Koller 2022-01-27 7:46 ` Andrew Jones 2022-01-27 7:46 ` Andrew Jones 2022-01-27 3:08 ` [PATCH v2 3/5] kvm: selftests: aarch64: fix the failure check in kvm_set_gsi_routing_irqchip_check Ricardo Koller 2022-01-27 3:08 ` Ricardo Koller 2022-01-27 3:08 ` [PATCH v2 4/5] kvm: selftests: aarch64: fix some vgic related comments Ricardo Koller 2022-01-27 3:08 ` Ricardo Koller 2022-01-27 7:49 ` Andrew Jones 2022-01-27 7:49 ` Andrew Jones 2022-01-27 15:06 ` Ricardo Koller 2022-01-27 15:06 ` Ricardo Koller 2022-01-27 3:08 ` [PATCH v2 5/5] kvm: selftests: aarch64: use a tighter assert in vgic_poke_irq() Ricardo Koller 2022-01-27 3:08 ` Ricardo Koller 2022-01-27 7:51 ` Andrew Jones 2022-01-27 7:51 ` Andrew Jones 2022-02-08 17:37 ` [PATCH v2 0/5] kvm: selftests: aarch64: some fixes for vgic_irq Marc Zyngier 2022-02-08 17:37 ` Marc Zyngier
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=20220127030858.3269036-1-ricarkol@google.com \ --to=ricarkol@google.com \ --cc=drjones@redhat.com \ --cc=kvm@vger.kernel.org \ --cc=kvmarm@lists.cs.columbia.edu \ --cc=maz@kernel.org \ --cc=oupton@google.com \ --cc=pbonzini@redhat.com \ --cc=reijiw@google.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.