[XEN PATCH] automation/eclair: add more guidelines to the monitored set

Federico Serafini posted 1 patch 5 months ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/f03398504405689413521de1675a33e50cdbc30b.1718983858.git.federico.serafini@bugseng.com
automation/eclair_analysis/ECLAIR/monitored.ecl | 5 +++++
1 file changed, 5 insertions(+)
[XEN PATCH] automation/eclair: add more guidelines to the monitored set
Posted by Federico Serafini 5 months ago
Add more accepted guidelines to the monitored set to check them at each
commit.

Signed-off-by: Federico Serafini <federico.serafini@bugseng.com>
---
 automation/eclair_analysis/ECLAIR/monitored.ecl | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/automation/eclair_analysis/ECLAIR/monitored.ecl b/automation/eclair_analysis/ECLAIR/monitored.ecl
index 4daecb0c83..9ffaebbdc3 100644
--- a/automation/eclair_analysis/ECLAIR/monitored.ecl
+++ b/automation/eclair_analysis/ECLAIR/monitored.ecl
@@ -18,10 +18,13 @@
 -enable=MC3R1.R12.5
 -enable=MC3R1.R1.3
 -enable=MC3R1.R13.6
+-enable=MC3R1.R13.1
 -enable=MC3R1.R1.4
 -enable=MC3R1.R14.1
 -enable=MC3R1.R14.4
 -enable=MC3R1.R16.2
+-enable=MC3R1.R16.3
+-enable=MC3R1.R16.4
 -enable=MC3R1.R16.6
 -enable=MC3R1.R16.7
 -enable=MC3R1.R17.1
@@ -34,6 +37,7 @@
 -enable=MC3R1.R20.13
 -enable=MC3R1.R20.14
 -enable=MC3R1.R20.4
+-enable=MC3R1.R20.7
 -enable=MC3R1.R20.9
 -enable=MC3R1.R2.1
 -enable=MC3R1.R21.10
@@ -58,6 +62,7 @@
 -enable=MC3R1.R5.2
 -enable=MC3R1.R5.3
 -enable=MC3R1.R5.4
+-enable=MC3R1.R5.5
 -enable=MC3R1.R5.6
 -enable=MC3R1.R6.1
 -enable=MC3R1.R6.2
-- 
2.34.1
Re: [XEN PATCH] automation/eclair: add more guidelines to the monitored set
Posted by Stefano Stabellini 5 months ago
On Fri, 21 Jun 2024, Federico Serafini wrote:
> Add more accepted guidelines to the monitored set to check them at each
> commit.
> 
> Signed-off-by: Federico Serafini <federico.serafini@bugseng.com>

Acked-by: Stefano Stabellini <sstabellini@kernel.org>

Asking for a release ack: this allows us to see more violations in the
regular ECLAIR scanning results. But they are not blocking, so they
won't cause additional new failures in the pipeline. It is just
informative.


> ---
>  automation/eclair_analysis/ECLAIR/monitored.ecl | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/automation/eclair_analysis/ECLAIR/monitored.ecl b/automation/eclair_analysis/ECLAIR/monitored.ecl
> index 4daecb0c83..9ffaebbdc3 100644
> --- a/automation/eclair_analysis/ECLAIR/monitored.ecl
> +++ b/automation/eclair_analysis/ECLAIR/monitored.ecl
> @@ -18,10 +18,13 @@
>  -enable=MC3R1.R12.5
>  -enable=MC3R1.R1.3
>  -enable=MC3R1.R13.6
> +-enable=MC3R1.R13.1
>  -enable=MC3R1.R1.4
>  -enable=MC3R1.R14.1
>  -enable=MC3R1.R14.4
>  -enable=MC3R1.R16.2
> +-enable=MC3R1.R16.3
> +-enable=MC3R1.R16.4
>  -enable=MC3R1.R16.6
>  -enable=MC3R1.R16.7
>  -enable=MC3R1.R17.1
> @@ -34,6 +37,7 @@
>  -enable=MC3R1.R20.13
>  -enable=MC3R1.R20.14
>  -enable=MC3R1.R20.4
> +-enable=MC3R1.R20.7
>  -enable=MC3R1.R20.9
>  -enable=MC3R1.R2.1
>  -enable=MC3R1.R21.10
> @@ -58,6 +62,7 @@
>  -enable=MC3R1.R5.2
>  -enable=MC3R1.R5.3
>  -enable=MC3R1.R5.4
> +-enable=MC3R1.R5.5
>  -enable=MC3R1.R5.6
>  -enable=MC3R1.R6.1
>  -enable=MC3R1.R6.2
> -- 
> 2.34.1
>
Re: [XEN PATCH] automation/eclair: add more guidelines to the monitored set
Posted by Oleksii 5 months ago
On Fri, 2024-06-21 at 14:33 -0700, Stefano Stabellini wrote:
> On Fri, 21 Jun 2024, Federico Serafini wrote:
> > Add more accepted guidelines to the monitored set to check them at
> > each
> > commit.
> > 
> > Signed-off-by: Federico Serafini <federico.serafini@bugseng.com>
> 
> Acked-by: Stefano Stabellini <sstabellini@kernel.org>
> 
> Asking for a release ack: this allows us to see more violations in
> the
> regular ECLAIR scanning results. But they are not blocking, so they
> won't cause additional new failures in the pipeline. It is just
> informative.
Release-Acked-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>

~ Oleksii

> 
> 
> > ---
> >  automation/eclair_analysis/ECLAIR/monitored.ecl | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/automation/eclair_analysis/ECLAIR/monitored.ecl
> > b/automation/eclair_analysis/ECLAIR/monitored.ecl
> > index 4daecb0c83..9ffaebbdc3 100644
> > --- a/automation/eclair_analysis/ECLAIR/monitored.ecl
> > +++ b/automation/eclair_analysis/ECLAIR/monitored.ecl
> > @@ -18,10 +18,13 @@
> >  -enable=MC3R1.R12.5
> >  -enable=MC3R1.R1.3
> >  -enable=MC3R1.R13.6
> > +-enable=MC3R1.R13.1
> >  -enable=MC3R1.R1.4
> >  -enable=MC3R1.R14.1
> >  -enable=MC3R1.R14.4
> >  -enable=MC3R1.R16.2
> > +-enable=MC3R1.R16.3
> > +-enable=MC3R1.R16.4
> >  -enable=MC3R1.R16.6
> >  -enable=MC3R1.R16.7
> >  -enable=MC3R1.R17.1
> > @@ -34,6 +37,7 @@
> >  -enable=MC3R1.R20.13
> >  -enable=MC3R1.R20.14
> >  -enable=MC3R1.R20.4
> > +-enable=MC3R1.R20.7
> >  -enable=MC3R1.R20.9
> >  -enable=MC3R1.R2.1
> >  -enable=MC3R1.R21.10
> > @@ -58,6 +62,7 @@
> >  -enable=MC3R1.R5.2
> >  -enable=MC3R1.R5.3
> >  -enable=MC3R1.R5.4
> > +-enable=MC3R1.R5.5
> >  -enable=MC3R1.R5.6
> >  -enable=MC3R1.R6.1
> >  -enable=MC3R1.R6.2
> > -- 
> > 2.34.1
> >