On Sat, 19 Jun 2021 at 10:13, Philippe Mathieu-Daudé <f4...@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é <f4...@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