Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
qtest: irq_intercept_[out/in]: return FAIL if no intercepts are insta…
…lled

This is much better than just silently failing with OK.

Signed-off-by: Chris Laplante <chris@laplante.io>
Message-id: 20230728160324.1159090-6-chris@laplante.io
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
  • Loading branch information
mostthingsweb authored and pm215 committed Aug 22, 2023
1 parent fe692f7 commit c7bb6fa
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions softmmu/qtest.c
Expand Up @@ -399,6 +399,7 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
NamedGPIOList *ngl;
bool is_named;
bool is_outbound;
bool interception_succeeded = false;

g_assert(words[1]);
is_named = words[2] != NULL;
Expand Down Expand Up @@ -435,15 +436,22 @@ static void qtest_process_command(CharBackend *chr, gchar **words)
for (i = 0; i < ngl->num_out; ++i) {
qtest_install_gpio_out_intercept(dev, ngl->name, i);
}
interception_succeeded = true;
}
} else {
qemu_irq_intercept_in(ngl->in, qtest_irq_handler,
ngl->num_in);
interception_succeeded = true;
}
}
irq_intercept_dev = dev;

qtest_send_prefix(chr);
qtest_send(chr, "OK\n");
if (interception_succeeded) {
irq_intercept_dev = dev;
qtest_send(chr, "OK\n");
} else {
qtest_send(chr, "FAIL No intercepts installed\n");
}
} else if (strcmp(words[0], "set_irq_in") == 0) {
DeviceState *dev;
qemu_irq irq;
Expand Down

0 comments on commit c7bb6fa

Please sign in to comment.