[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/38] ivshmem-test: Improve test cases /ivshmem
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH 07/38] ivshmem-test: Improve test cases /ivshmem/server-* |
Date: |
Tue, 1 Mar 2016 12:13:07 +0100 |
On Mon, Feb 29, 2016 at 7:40 PM, Markus Armbruster <address@hidden> wrote:
> Document missing test: behavior with MSI-X present but not enabled.
>
> For MSI-X, we test and clear the interrupt pending bit before testing
> the interrupt. For INTx, we only clear. Change to test and clear for
> consistency.
>
> Test MSI-X vector 1 in addition to vector 0.
>
> Improve comments.
>
> Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
> ---
> tests/ivshmem-test.c | 17 ++++++++++-------
> 1 file changed, 10 insertions(+), 7 deletions(-)
>
> diff --git a/tests/ivshmem-test.c b/tests/ivshmem-test.c
> index f40c3497..c1dd7bb 100644
> --- a/tests/ivshmem-test.c
> +++ b/tests/ivshmem-test.c
> @@ -339,18 +339,21 @@ static void test_ivshmem_server(bool msi)
> vm2 = in_reg(s2, IVPOSITION);
> g_assert_cmpuint(vm1, !=, vm2);
>
> + /* check number of MSI-X vectors */
> global_qtest = s1->qtest;
> if (msi) {
> ret = qpci_msix_table_size(s1->dev);
> g_assert_cmpuint(ret, ==, nvectors);
> }
>
> - /* ping vm2 -> vm1 */
> + /* TODO test behavior before MSI-X is enabled */
> +
> + /* ping vm2 -> vm1 on vector 0 */
> if (msi) {
> ret = qpci_msix_pending(s1->dev, 0);
> g_assert_cmpuint(ret, ==, 0);
> } else {
> - out_reg(s1, INTRSTATUS, 0);
> + g_assert_cmpuint(in_reg(s1, INTRSTATUS), ==, 0);
> }
> out_reg(s2, DOORBELL, vm1 << 16);
> do {
> @@ -359,18 +362,18 @@ static void test_ivshmem_server(bool msi)
> } while (ret == 0 && g_get_monotonic_time() < end_time);
> g_assert_cmpuint(ret, !=, 0);
>
> - /* ping vm1 -> vm2 */
> + /* ping vm1 -> vm2 on vector 1 */
> global_qtest = s2->qtest;
> if (msi) {
> - ret = qpci_msix_pending(s2->dev, 0);
> + ret = qpci_msix_pending(s2->dev, 1);
> g_assert_cmpuint(ret, ==, 0);
> } else {
> - out_reg(s2, INTRSTATUS, 0);
> + g_assert_cmpuint(in_reg(s2, INTRSTATUS), ==, 0);
> }
> - out_reg(s1, DOORBELL, vm2 << 16);
> + out_reg(s1, DOORBELL, vm2 << 16 | 1);
> do {
> g_usleep(10000);
> - ret = msi ? qpci_msix_pending(s2->dev, 0) : in_reg(s2, INTRSTATUS);
> + ret = msi ? qpci_msix_pending(s2->dev, 1) : in_reg(s2, INTRSTATUS);
> } while (ret == 0 && g_get_monotonic_time() < end_time);
> g_assert_cmpuint(ret, !=, 0);
>
> --
> 2.4.3
>
>
--
Marc-André Lureau
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 07/38] ivshmem-test: Improve test cases /ivshmem/server-*,
Marc-André Lureau <=