Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-12-06 Thread Nicola Vetrini
On 2023-12-05 00:18, Stefano Stabellini wrote: On Mon, 4 Dec 2023, Nicola Vetrini wrote: On 2023-12-04 08:44, Jan Beulich wrote: > On 02.12.2023 04:03, Stefano Stabellini wrote: > > On Fri, 1 Dec 2023, Jan Beulich wrote: > > > On 01.12.2023 03:47, Stefano Stabellini wrote: > > > > On Wed, 29 Nov

Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-12-04 Thread Stefano Stabellini
On Sat, 2 Dec 2023, Julien Grall wrote: > Hi, > > On 02/12/2023 04:03, Stefano Stabellini wrote: > > On Fri, 1 Dec 2023, Jan Beulich wrote: > > > On 01.12.2023 03:47, Stefano Stabellini wrote: > > > > On Wed, 29 Nov 2023, Nicola Vetrini wrote: > > > > > No functional change. > > > > > > > > > > S

Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-12-04 Thread Stefano Stabellini
On Mon, 4 Dec 2023, Nicola Vetrini wrote: > On 2023-12-04 08:44, Jan Beulich wrote: > > On 02.12.2023 04:03, Stefano Stabellini wrote: > > > On Fri, 1 Dec 2023, Jan Beulich wrote: > > > > On 01.12.2023 03:47, Stefano Stabellini wrote: > > > > > On Wed, 29 Nov 2023, Nicola Vetrini wrote: > > > > > >

Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-12-04 Thread Nicola Vetrini
On 2023-12-04 08:44, Jan Beulich wrote: On 02.12.2023 04:03, Stefano Stabellini wrote: On Fri, 1 Dec 2023, Jan Beulich wrote: On 01.12.2023 03:47, Stefano Stabellini wrote: On Wed, 29 Nov 2023, Nicola Vetrini wrote: No functional change. Signed-off-by: Nicola Vetrini --- The preferred way t

Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-12-03 Thread Jan Beulich
On 02.12.2023 04:03, Stefano Stabellini wrote: > On Fri, 1 Dec 2023, Jan Beulich wrote: >> On 01.12.2023 03:47, Stefano Stabellini wrote: >>> On Wed, 29 Nov 2023, Nicola Vetrini wrote: No functional change. Signed-off-by: Nicola Vetrini --- The preferred way to deviate is

Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-12-02 Thread Julien Grall
Hi, On 02/12/2023 04:03, Stefano Stabellini wrote: On Fri, 1 Dec 2023, Jan Beulich wrote: On 01.12.2023 03:47, Stefano Stabellini wrote: On Wed, 29 Nov 2023, Nicola Vetrini wrote: No functional change. Signed-off-by: Nicola Vetrini --- The preferred way to deviate is to use asmlinkage, but

Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-12-01 Thread Stefano Stabellini
On Fri, 1 Dec 2023, Jan Beulich wrote: > On 01.12.2023 03:47, Stefano Stabellini wrote: > > On Wed, 29 Nov 2023, Nicola Vetrini wrote: > >> No functional change. > >> > >> Signed-off-by: Nicola Vetrini > >> --- > >> The preferred way to deviate is to use asmlinkage, but this modification > >> is

Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-11-30 Thread Jan Beulich
On 01.12.2023 03:47, Stefano Stabellini wrote: > On Wed, 29 Nov 2023, Nicola Vetrini wrote: >> No functional change. >> >> Signed-off-by: Nicola Vetrini >> --- >> The preferred way to deviate is to use asmlinkage, but this modification is >> only >> the consequence of NUMA on ARM (and possibly PP

Re: [XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-11-30 Thread Stefano Stabellini
On Wed, 29 Nov 2023, Nicola Vetrini wrote: > No functional change. > > Signed-off-by: Nicola Vetrini > --- > The preferred way to deviate is to use asmlinkage, but this modification is > only > the consequence of NUMA on ARM (and possibly PPC) being a work in progress. > As stated in the comment

[XEN PATCH 7/7] xen/page_alloc: deviate first_valid_mfn for MISRA C Rule 8.4

2023-11-29 Thread Nicola Vetrini
No functional change. Signed-off-by: Nicola Vetrini --- The preferred way to deviate is to use asmlinkage, but this modification is only the consequence of NUMA on ARM (and possibly PPC) being a work in progress. As stated in the comment above the textual deviation, first_valid_mfn will likely th