[PATCH v2 2/5] scripts/coverity-scan/COMPONENTS.md: Fix 'char' component

Peter Maydell posted 5 patches 5 months, 3 weeks ago
Maintainers: Peter Maydell <peter.maydell@linaro.org>
[PATCH v2 2/5] scripts/coverity-scan/COMPONENTS.md: Fix 'char' component
Posted by Peter Maydell 5 months, 3 weeks ago
The 'char' component:
 * includes the no-longer-present qemu-char.c, which has been
   long since split into the chardev/ backend code
 * also includes the hw/char devices

Split it into two components:
 * char is the hw/char devices
 * chardev is the chardev backends
with regexes matching our current sources.

Signed-off-by: Peter Maydell <peter.maydell@linaro.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 98d4bcd6a50..fb081a59265 100644
--- a/scripts/coverity-scan/COMPONENTS.md
+++ b/scripts/coverity-scan/COMPONENTS.md
@@ -73,7 +73,10 @@ block
   ~ .*/qemu(/block.*|(/include?)/(block|storage-daemon)/.*|(/include)?/hw/(block|ide|nvme)/.*|/qemu-(img|io).*|/util/(aio|async|thread-pool).*)
 
 char
-  ~ .*/qemu(/qemu-char\.c|/include/sysemu/char\.h|(/include)?/hw/char/.*)
+  ~ .*/qemu((/include)?/hw/char/.*)
+
+chardev
+  ~ .*/qemu((/include)?/chardev/.*)
 
 crypto
   ~ .*/qemu((/include)?/crypto/.*|/hw/.*/.*crypto.*|(/include/sysemu|/backends)/cryptodev.*)
-- 
2.34.1
Re: [PATCH v2 2/5] scripts/coverity-scan/COMPONENTS.md: Fix 'char' component
Posted by Philippe Mathieu-Daudé 5 months, 3 weeks ago
On 4/6/24 16:59, Peter Maydell wrote:
> The 'char' component:
>   * includes the no-longer-present qemu-char.c, which has been
>     long since split into the chardev/ backend code
>   * also includes the hw/char devices
> 
> Split it into two components:
>   * char is the hw/char devices
>   * chardev is the chardev backends
> with regexes matching our current sources.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>   scripts/coverity-scan/COMPONENTS.md | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>