diff mbox series

[2/2] coverity-scan: Add 'sysemu' component

Message ID 20210619091342.3660495-3-f4bug@amsat.org
State New
Headers show
Series coverity-scan: Remove lm32/unicore32 and add sysemu component | expand

Commit Message

Philippe Mathieu-Daudé June 19, 2021, 9:13 a.m. UTC
Issues related to system emulation currently appear under the 'tcg'
component. We commonly refer to system emulation as 'sysemu', so
add it as a new Coverity component to cover files that have nothing
to do with TCG.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
 scripts/coverity-scan/COMPONENTS.md | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Peter Maydell June 29, 2021, 3:18 p.m. UTC | #1
On Sat, 19 Jun 2021 at 10:13, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> Issues related to system emulation currently appear under the 'tcg'
> component. We commonly refer to system emulation as 'sysemu', so
> add it as a new Coverity component to cover files that have nothing
> to do with TCG.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  scripts/coverity-scan/COMPONENTS.md | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/coverity-scan/COMPONENTS.md b/scripts/coverity-scan/COMPONENTS.md
> index 183f26a32c9..7ed3f0a0077 100644
> --- a/scripts/coverity-scan/COMPONENTS.md
> +++ b/scripts/coverity-scan/COMPONENTS.md
> @@ -115,7 +115,7 @@ slirp
>    ~ (/qemu)?(/.*slirp.*)
>
>  tcg
> -  ~ (/qemu)?(/accel/tcg/.*|/replay/.*|/(.*/)?softmmu.*)
> +  ~ (/qemu)?(/accel/tcg/.*|/replay/.*|tcg/*\.?)

Makes sense not to call softmmu/ part of "TCG"...

>
>  trace
>    ~ (/qemu)?(/.*trace.*\.[ch])
> @@ -129,6 +129,9 @@ usb
>  user
>    ~ (/qemu)?(/linux-user/.*|/bsd-user/.*|/user-exec\.c|/thunk\.c|/include/exec/user/.*)
>
> +sysemu
> +  ~ (/qemu)?(/softmmu/.*|/accel/(hvf|kvm)/.*)
> +

...but maybe we should have a component for each accelerator
instead of lumping hvf and kvm under here ? We already have
components for xen and tcg.

>  util
>    ~ (/qemu)?(/util/.*|/include/qemu/.*)
>
> --
> 2.31.1

thanks
-- PMM
diff mbox series

Patch

diff --git a/scripts/coverity-scan/COMPONENTS.md b/scripts/coverity-scan/COMPONENTS.md
index 183f26a32c9..7ed3f0a0077 100644
--- a/scripts/coverity-scan/COMPONENTS.md
+++ b/scripts/coverity-scan/COMPONENTS.md
@@ -115,7 +115,7 @@  slirp
   ~ (/qemu)?(/.*slirp.*)
 
 tcg
-  ~ (/qemu)?(/accel/tcg/.*|/replay/.*|/(.*/)?softmmu.*)
+  ~ (/qemu)?(/accel/tcg/.*|/replay/.*|tcg/*\.?)
 
 trace
   ~ (/qemu)?(/.*trace.*\.[ch])
@@ -129,6 +129,9 @@  usb
 user
   ~ (/qemu)?(/linux-user/.*|/bsd-user/.*|/user-exec\.c|/thunk\.c|/include/exec/user/.*)
 
+sysemu
+  ~ (/qemu)?(/softmmu/.*|/accel/(hvf|kvm)/.*)
+
 util
   ~ (/qemu)?(/util/.*|/include/qemu/.*)