Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-16 Thread Jan Beulich
On 05.10.2023 01:32, Stefano Stabellini wrote: > On Wed, 4 Oct 2023, Luca Fancellu wrote: >>> On 4 Oct 2023, at 11:29, Nicola Vetrini wrote: >>> On 04/10/2023 12:06, Luca Fancellu wrote: Hi Nicola, > On 4 Oct 2023, at 10:56, andrew.coop...@citrix.com wrote: > On 03/10/2023 9:46 pm, St

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-12 Thread Stefano Stabellini
On Wed, 11 Oct 2023, Julien Grall wrote: > On 11/10/2023 11:53, Luca Fancellu wrote: > > > > > > > > > > > > Luca answered to a similar, more generic, question a few days ago > > > > > about > > > > > Coverity: https://marc.info/?l=xen-devel&m=169657904027210 > > > > Interesting. > > > > > > > >

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-11 Thread Julien Grall
On 11/10/2023 11:53, Luca Fancellu wrote: Luca answered to a similar, more generic, question a few days ago about Coverity: https://marc.info/?l=xen-devel&m=169657904027210 Interesting. So even if we get cppcheck to work like that, we would lose Coverity support. I think my suggestion w

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-11 Thread Luca Fancellu
>>> >>> Luca answered to a similar, more generic, question a few days ago about >>> Coverity: https://marc.info/?l=xen-devel&m=169657904027210 >> Interesting. >>> >>> So even if we get cppcheck to work like that, we would lose Coverity >>> support. >> I think my suggestion was probably misunders

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-11 Thread Julien Grall
On 11/10/2023 10:45, Julien Grall wrote: Hi Stefano, On 11/10/2023 00:39, Stefano Stabellini wrote: On Tue, 10 Oct 2023, Julien Grall wrote: Hi Stefano, On 09/10/2023 23:19, Stefano Stabellini wrote: I am not sure this is better. This is a long line to read. But this is a personal opin

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-11 Thread Julien Grall
Hi Stefano, On 11/10/2023 00:39, Stefano Stabellini wrote: On Tue, 10 Oct 2023, Julien Grall wrote: Hi Stefano, On 09/10/2023 23:19, Stefano Stabellini wrote: I am not sure this is better. This is a long line to read. But this is a personal opinion. On the technical side, can we easily teac

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-10 Thread Stefano Stabellini
On Tue, 10 Oct 2023, Julien Grall wrote: > Hi Stefano, > > On 09/10/2023 23:19, Stefano Stabellini wrote: > > > > > > I am not sure this is better. This is a long line to read. But this is a > > > personal opinion. > > > > > > On the technical side, can we easily teach a tool to format this kind

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-10 Thread Julien Grall
Hi Stefano, On 09/10/2023 23:19, Stefano Stabellini wrote: I am not sure this is better. This is a long line to read. But this is a personal opinion. On the technical side, can we easily teach a tool to format this kind of exception? If not, then this should not be an exception we should imple

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-10 Thread Nicola Vetrini
On 10/10/2023 09:29, Luca Fancellu wrote: /* SAF-x-safe */ if ( modrm_mod == MASK_EXTR(instr_modrm, 0300) && (modrm_reg & 7) == MASK_EXTR(instr_modrm, 0070) && (modrm_rm & 7) == MASK_EXTR(instr_modrm, 0007) ) Any other ideas? Could we have a number in the comment to indicate the number of

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-10 Thread Luca Fancellu
> >>> /* SAF-x-safe */ >>> if ( modrm_mod == MASK_EXTR(instr_modrm, 0300) && (modrm_reg & 7) == >>> MASK_EXTR(instr_modrm, 0070) && (modrm_rm & 7) == MASK_EXTR(instr_modrm, >>> 0007) ) >>> >>> Any other ideas? >> >> Could we have a number in the comment to indicate the number of lines the >> co

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-09 Thread Stefano Stabellini
On Mon, 9 Oct 2023, Julien Grall wrote: > On 07/10/2023 01:43, Stefano Stabellini wrote: > > On Fri, 6 Oct 2023, Luca Fancellu wrote: > > > > On 6 Oct 2023, at 02:02, Stefano Stabellini > > > > wrote: > > > > > > > > On Thu, 5 Oct 2023, Luca Fancellu wrote: > > > > > > On 5 Oct 2023, at 00:32, St

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-09 Thread Julien Grall
Hi, On 07/10/2023 01:43, Stefano Stabellini wrote: On Fri, 6 Oct 2023, Luca Fancellu wrote: On 6 Oct 2023, at 02:02, Stefano Stabellini wrote: On Thu, 5 Oct 2023, Luca Fancellu wrote: On 5 Oct 2023, at 00:32, Stefano Stabellini wrote: On Wed, 4 Oct 2023, Luca Fancellu wrote: On 4 Oct 2023

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-09 Thread Luca Fancellu
>>> >>> Right so the results would be all off by a few lines of code so when >>> you go to read the report generated by cppcheck, the references >>> wouldn't match anymore. >>> >>> Before giving up and accepting that we are constrained to only formats >>> that don't change the LOC numbers, can we

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-06 Thread Stefano Stabellini
On Fri, 6 Oct 2023, Luca Fancellu wrote: > > On 6 Oct 2023, at 02:02, Stefano Stabellini wrote: > > > > On Thu, 5 Oct 2023, Luca Fancellu wrote: > >>> On 5 Oct 2023, at 00:32, Stefano Stabellini > >>> wrote: > >>> > >>> On Wed, 4 Oct 2023, Luca Fancellu wrote: > > On 4 Oct 2023, at 11:29,

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-06 Thread Luca Fancellu
> On 6 Oct 2023, at 02:02, Stefano Stabellini wrote: > > On Thu, 5 Oct 2023, Luca Fancellu wrote: >>> On 5 Oct 2023, at 00:32, Stefano Stabellini wrote: >>> >>> On Wed, 4 Oct 2023, Luca Fancellu wrote: > On 4 Oct 2023, at 11:29, Nicola Vetrini > wrote: > On 04/10/2023 12:06, Luc

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-05 Thread Stefano Stabellini
On Thu, 5 Oct 2023, Luca Fancellu wrote: > > On 5 Oct 2023, at 00:32, Stefano Stabellini wrote: > > > > On Wed, 4 Oct 2023, Luca Fancellu wrote: > >>> On 4 Oct 2023, at 11:29, Nicola Vetrini > >>> wrote: > >>> On 04/10/2023 12:06, Luca Fancellu wrote: > Hi Nicola, > > On 4 Oct 2023, at

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-05 Thread Luca Fancellu
> On 5 Oct 2023, at 00:32, Stefano Stabellini wrote: > > On Wed, 4 Oct 2023, Luca Fancellu wrote: >>> On 4 Oct 2023, at 11:29, Nicola Vetrini wrote: >>> On 04/10/2023 12:06, Luca Fancellu wrote: Hi Nicola, > On 4 Oct 2023, at 10:56, andrew.coop...@citrix.com wrote: > On 03/10/2023

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-04 Thread Stefano Stabellini
On Wed, 4 Oct 2023, Luca Fancellu wrote: > > On 4 Oct 2023, at 11:29, Nicola Vetrini wrote: > > On 04/10/2023 12:06, Luca Fancellu wrote: > >> Hi Nicola, > >>> On 4 Oct 2023, at 10:56, andrew.coop...@citrix.com wrote: > >>> On 03/10/2023 9:46 pm, Stefano Stabellini wrote: > On Tue, 3 Oct 2023

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-04 Thread Nicola Vetrini
On 04/10/2023 12:52, Luca Fancellu wrote: On 4 Oct 2023, at 11:29, Nicola Vetrini wrote: On 04/10/2023 12:06, Luca Fancellu wrote: Hi Nicola, On 4 Oct 2023, at 10:56, andrew.coop...@citrix.com wrote: On 03/10/2023 9:46 pm, Stefano Stabellini wrote: On Tue, 3 Oct 2023, Nicola Vetrini wrote:

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-04 Thread Luca Fancellu
> On 4 Oct 2023, at 12:17, Andrew Cooper wrote: > > On 04/10/2023 11:52 am, Luca Fancellu wrote: >> >> From the documentation: >> >> In the Xen codebase, these tags will be used to document and suppress >> findings: >> >> - SAF-X-safe: This tag means that the next line of code contains a fi

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-04 Thread Andrew Cooper
On 04/10/2023 11:52 am, Luca Fancellu wrote: > From the documentation: > > In the Xen codebase, these tags will be used to document and suppress > findings: > > - SAF-X-safe: This tag means that the next line of code contains a > finding, but > the non compliance to the checker is a

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-04 Thread Luca Fancellu
> On 4 Oct 2023, at 11:29, Nicola Vetrini wrote: > > On 04/10/2023 12:06, Luca Fancellu wrote: >> Hi Nicola, >>> On 4 Oct 2023, at 10:56, andrew.coop...@citrix.com wrote: >>> On 03/10/2023 9:46 pm, Stefano Stabellini wrote: On Tue, 3 Oct 2023, Nicola Vetrini wrote: > As specified in ru

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-04 Thread Nicola Vetrini
On 04/10/2023 12:06, Luca Fancellu wrote: Hi Nicola, On 4 Oct 2023, at 10:56, andrew.coop...@citrix.com wrote: On 03/10/2023 9:46 pm, Stefano Stabellini wrote: On Tue, 3 Oct 2023, Nicola Vetrini wrote: As specified in rules.rst, these constants can be used in the code. Their deviation is no

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-04 Thread Luca Fancellu
Hi Nicola, > On 4 Oct 2023, at 10:56, andrew.coop...@citrix.com wrote: > > On 03/10/2023 9:46 pm, Stefano Stabellini wrote: >> On Tue, 3 Oct 2023, Nicola Vetrini wrote: >> >>> As specified in rules.rst, these constants can be used >>> in the code. >>> Their deviation is now accomplished by using

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-04 Thread andrew . cooper3
On 03/10/2023 9:46 pm, Stefano Stabellini wrote: > On Tue, 3 Oct 2023, Nicola Vetrini wrote: >> As specified in rules.rst, these constants can be used >> in the code. >> Their deviation is now accomplished by using a SAF comment, >> rather than an ECLAIR configuration. >> >> Signed-off-by: Nicola V

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-03 Thread Stefano Stabellini
On Tue, 3 Oct 2023, Nicola Vetrini wrote: > As specified in rules.rst, these constants can be used > in the code. > Their deviation is now accomplished by using a SAF comment, > rather than an ECLAIR configuration. > > Signed-off-by: Nicola Vetrini "SAF" discussion aside that can be resolved els

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-03 Thread Stefano Stabellini
On Tue, 3 Oct 2023, Andrew Cooper wrote: > On 03/10/2023 6:14 pm, Luca Fancellu wrote: > > > >> On 3 Oct 2023, at 17:17, andrew.coop...@citrix.com wrote: > >> > >> On 03/10/2023 4:37 pm, Nicola Vetrini wrote: > >>> As specified in rules.rst, these constants can be used > >>> in the code. > >>> Thei

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-03 Thread Andrew Cooper
On 03/10/2023 6:14 pm, Luca Fancellu wrote: > >> On 3 Oct 2023, at 17:17, andrew.coop...@citrix.com wrote: >> >> On 03/10/2023 4:37 pm, Nicola Vetrini wrote: >>> As specified in rules.rst, these constants can be used >>> in the code. >>> Their deviation is now accomplished by using a SAF comment, >

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-03 Thread Luca Fancellu
> On 3 Oct 2023, at 17:17, andrew.coop...@citrix.com wrote: > > On 03/10/2023 4:37 pm, Nicola Vetrini wrote: >> As specified in rules.rst, these constants can be used >> in the code. >> Their deviation is now accomplished by using a SAF comment, >> rather than an ECLAIR configuration. >> >> Si

Re: [XEN PATCH] xen: Add SAF deviations for MISRA C:2012 Rule 7.1

2023-10-03 Thread andrew . cooper3
On 03/10/2023 4:37 pm, Nicola Vetrini wrote: > As specified in rules.rst, these constants can be used > in the code. > Their deviation is now accomplished by using a SAF comment, > rather than an ECLAIR configuration. > > Signed-off-by: Nicola Vetrini > --- > automation/eclair_analysis/ECLAIR/dev