All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] coverity-scan: Remove lm32/unicore32 and add sysemu component
@ 2021-06-19  9:13 Philippe Mathieu-Daudé
  2021-06-19  9:13 ` [PATCH 1/2] coverity-scan: Remove lm32 / unicore32 targets Philippe Mathieu-Daudé
  2021-06-19  9:13 ` [PATCH 2/2] coverity-scan: Add 'sysemu' component Philippe Mathieu-Daudé
  0 siblings, 2 replies; 6+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-06-19  9:13 UTC (permalink / raw)
  To: qemu-devel; +Cc: Paolo Bonzini, Philippe Mathieu-Daudé, Peter Maydell

Add 'sysemu' component for issues not related to TCG.

Philippe Mathieu-Daudé (2):
  coverity-scan: Remove lm32 / unicore32 targets
  coverity-scan: Add 'sysemu' component

 scripts/coverity-scan/COMPONENTS.md | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

-- 
2.31.1



^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/2] coverity-scan: Remove lm32 / unicore32 targets
  2021-06-19  9:13 [PATCH 0/2] coverity-scan: Remove lm32/unicore32 and add sysemu component Philippe Mathieu-Daudé
@ 2021-06-19  9:13 ` Philippe Mathieu-Daudé
  2021-06-29 15:16   ` Peter Maydell
  2021-06-19  9:13 ` [PATCH 2/2] coverity-scan: Add 'sysemu' component Philippe Mathieu-Daudé
  1 sibling, 1 reply; 6+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-06-19  9:13 UTC (permalink / raw)
  To: qemu-devel; +Cc: Paolo Bonzini, Philippe Mathieu-Daudé, Peter Maydell

lm32 has been removed in commit 9d49bcf6992 ("Drop the deprecated
lm32 target"), and unicore32 in 4369223902a ("Drop the deprecated
unicore32 target").

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

diff --git a/scripts/coverity-scan/COMPONENTS.md b/scripts/coverity-scan/COMPONENTS.md
index 02a3447dab0..183f26a32c9 100644
--- a/scripts/coverity-scan/COMPONENTS.md
+++ b/scripts/coverity-scan/COMPONENTS.md
@@ -21,9 +21,6 @@ hppa
 i386
   ~ (/qemu)?((/include)?/hw/i386/.*|/target/i386/.*|/hw/intc/[^/]*apic[^/]*\.c)
 
-lm32
-  ~ (/qemu)?((/include)?/hw/lm32/.*|/target/lm32/.*|/hw/.*/(milkymist|lm32).*)
-
 m68k
   ~ (/qemu)?((/include)?/hw/m68k/.*|/target/m68k/.*|(/include)?/hw(/.*)?/mcf.*)
 
@@ -60,9 +57,6 @@ tilegx
 tricore
   ~ (/qemu)?((/include)?/hw/tricore/.*|/target/tricore/.*)
 
-unicore32
-  ~ (/qemu)?((/include)?/hw/unicore32/.*|/target/unicore32/.*)
-
 9pfs
   ~ (/qemu)?(/hw/9pfs/.*|/fsdev/.*)
 
-- 
2.31.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] coverity-scan: Add 'sysemu' component
  2021-06-19  9:13 [PATCH 0/2] coverity-scan: Remove lm32/unicore32 and add sysemu component Philippe Mathieu-Daudé
  2021-06-19  9:13 ` [PATCH 1/2] coverity-scan: Remove lm32 / unicore32 targets Philippe Mathieu-Daudé
@ 2021-06-19  9:13 ` Philippe Mathieu-Daudé
  2021-06-29 15:18   ` Peter Maydell
  1 sibling, 1 reply; 6+ messages in thread
From: Philippe Mathieu-Daudé @ 2021-06-19  9:13 UTC (permalink / raw)
  To: qemu-devel; +Cc: Paolo Bonzini, Philippe Mathieu-Daudé, Peter Maydell

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/*\.?)
 
 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/.*)
 
-- 
2.31.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] coverity-scan: Remove lm32 / unicore32 targets
  2021-06-19  9:13 ` [PATCH 1/2] coverity-scan: Remove lm32 / unicore32 targets Philippe Mathieu-Daudé
@ 2021-06-29 15:16   ` Peter Maydell
  2021-07-05 17:09     ` Paolo Bonzini
  0 siblings, 1 reply; 6+ messages in thread
From: Peter Maydell @ 2021-06-29 15:16 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé; +Cc: Paolo Bonzini, QEMU Developers

On Sat, 19 Jun 2021 at 10:13, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> lm32 has been removed in commit 9d49bcf6992 ("Drop the deprecated
> lm32 target"), and unicore32 in 4369223902a ("Drop the deprecated
> unicore32 target").
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  scripts/coverity-scan/COMPONENTS.md | 6 ------
>  1 file changed, 6 deletions(-)
>
> diff --git a/scripts/coverity-scan/COMPONENTS.md b/scripts/coverity-scan/COMPONENTS.md
> index 02a3447dab0..183f26a32c9 100644
> --- a/scripts/coverity-scan/COMPONENTS.md
> +++ b/scripts/coverity-scan/COMPONENTS.md
> @@ -21,9 +21,6 @@ hppa
>  i386
>    ~ (/qemu)?((/include)?/hw/i386/.*|/target/i386/.*|/hw/intc/[^/]*apic[^/]*\.c)
>
> -lm32
> -  ~ (/qemu)?((/include)?/hw/lm32/.*|/target/lm32/.*|/hw/.*/(milkymist|lm32).*)
> -
>  m68k
>    ~ (/qemu)?((/include)?/hw/m68k/.*|/target/m68k/.*|(/include)?/hw(/.*)?/mcf.*)
>
> @@ -60,9 +57,6 @@ tilegx
>  tricore
>    ~ (/qemu)?((/include)?/hw/tricore/.*|/target/tricore/.*)
>
> -unicore32
> -  ~ (/qemu)?((/include)?/hw/unicore32/.*|/target/unicore32/.*)
> -
>  9pfs
>    ~ (/qemu)?(/hw/9pfs/.*|/fsdev/.*)

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

NB that somebody will have to manually make the changes to the Coverity config
on the website as well.

thanks
-- PMM


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/2] coverity-scan: Add 'sysemu' component
  2021-06-19  9:13 ` [PATCH 2/2] coverity-scan: Add 'sysemu' component Philippe Mathieu-Daudé
@ 2021-06-29 15:18   ` Peter Maydell
  0 siblings, 0 replies; 6+ messages in thread
From: Peter Maydell @ 2021-06-29 15:18 UTC (permalink / raw)
  To: Philippe Mathieu-Daudé; +Cc: Paolo Bonzini, QEMU Developers

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] coverity-scan: Remove lm32 / unicore32 targets
  2021-06-29 15:16   ` Peter Maydell
@ 2021-07-05 17:09     ` Paolo Bonzini
  0 siblings, 0 replies; 6+ messages in thread
From: Paolo Bonzini @ 2021-07-05 17:09 UTC (permalink / raw)
  To: Peter Maydell, Philippe Mathieu-Daudé; +Cc: QEMU Developers

On 29/06/21 17:16, Peter Maydell wrote:
> On Sat, 19 Jun 2021 at 10:13, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>>
>> lm32 has been removed in commit 9d49bcf6992 ("Drop the deprecated
>> lm32 target"), and unicore32 in 4369223902a ("Drop the deprecated
>> unicore32 target").
>>
>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>> ---
>>   scripts/coverity-scan/COMPONENTS.md | 6 ------
>>   1 file changed, 6 deletions(-)
>>
>> diff --git a/scripts/coverity-scan/COMPONENTS.md b/scripts/coverity-scan/COMPONENTS.md
>> index 02a3447dab0..183f26a32c9 100644
>> --- a/scripts/coverity-scan/COMPONENTS.md
>> +++ b/scripts/coverity-scan/COMPONENTS.md
>> @@ -21,9 +21,6 @@ hppa
>>   i386
>>     ~ (/qemu)?((/include)?/hw/i386/.*|/target/i386/.*|/hw/intc/[^/]*apic[^/]*\.c)
>>
>> -lm32
>> -  ~ (/qemu)?((/include)?/hw/lm32/.*|/target/lm32/.*|/hw/.*/(milkymist|lm32).*)
>> -
>>   m68k
>>     ~ (/qemu)?((/include)?/hw/m68k/.*|/target/m68k/.*|(/include)?/hw(/.*)?/mcf.*)
>>
>> @@ -60,9 +57,6 @@ tilegx
>>   tricore
>>     ~ (/qemu)?((/include)?/hw/tricore/.*|/target/tricore/.*)
>>
>> -unicore32
>> -  ~ (/qemu)?((/include)?/hw/unicore32/.*|/target/unicore32/.*)
>> -
>>   9pfs
>>     ~ (/qemu)?(/hw/9pfs/.*|/fsdev/.*)
> 
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
> 
> NB that somebody will have to manually make the changes to the Coverity config
> on the website as well.
> 
> thanks
> -- PMM
> 

Queued this one (and made the manual changes).

Paolo



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2021-07-05 17:11 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-19  9:13 [PATCH 0/2] coverity-scan: Remove lm32/unicore32 and add sysemu component Philippe Mathieu-Daudé
2021-06-19  9:13 ` [PATCH 1/2] coverity-scan: Remove lm32 / unicore32 targets Philippe Mathieu-Daudé
2021-06-29 15:16   ` Peter Maydell
2021-07-05 17:09     ` Paolo Bonzini
2021-06-19  9:13 ` [PATCH 2/2] coverity-scan: Add 'sysemu' component Philippe Mathieu-Daudé
2021-06-29 15:18   ` Peter Maydell

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.