[PATCH] device-crash-test: Ignore errors about a bus not being available

Thomas Huth posted 1 patch 3 years ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210323164718.1393792-1-thuth@redhat.com
scripts/device-crash-test | 1 +
1 file changed, 1 insertion(+)
[PATCH] device-crash-test: Ignore errors about a bus not being available
Posted by Thomas Huth 3 years ago
Recent QEMU versions now sometimes exit cleanly with an error message
that a bus is not available for a specified device. Don't flag those
as an error in the device-crash-test script.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 scripts/device-crash-test | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scripts/device-crash-test b/scripts/device-crash-test
index 6d809ac711..8331c057b8 100755
--- a/scripts/device-crash-test
+++ b/scripts/device-crash-test
@@ -175,6 +175,7 @@ ERROR_RULE_LIST = [
     {'log':r"Multiple VT220 operator consoles are not supported"},
     {'log':r"core 0 already populated"},
     {'log':r"could not find stage1 bootloader"},
+    {'log':r"No '.*' bus found for device"},
 
     # other exitcode=1 failures not listed above will just generate INFO messages:
     {'exitcode':1, 'loglevel':logging.INFO},
-- 
2.27.0


Re: [PATCH] device-crash-test: Ignore errors about a bus not being available
Posted by John Snow 3 years ago
On 3/23/21 12:47 PM, Thomas Huth wrote:
> Recent QEMU versions now sometimes exit cleanly with an error message
> that a bus is not available for a specified device. Don't flag those
> as an error in the device-crash-test script.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   scripts/device-crash-test | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/scripts/device-crash-test b/scripts/device-crash-test
> index 6d809ac711..8331c057b8 100755
> --- a/scripts/device-crash-test
> +++ b/scripts/device-crash-test
> @@ -175,6 +175,7 @@ ERROR_RULE_LIST = [
>       {'log':r"Multiple VT220 operator consoles are not supported"},
>       {'log':r"core 0 already populated"},
>       {'log':r"could not find stage1 bootloader"},
> +    {'log':r"No '.*' bus found for device"},
>   
>       # other exitcode=1 failures not listed above will just generate INFO messages:
>       {'exitcode':1, 'loglevel':logging.INFO},
> 

ACK, though I don't consider myself a maintainer of this file. Whose 
tree does it go in? (Trivial, if nobody's?)

--js