diff mbox series

[v13,07/10] xen/common: fix build issue for common/trace.c

Message ID f14f2c5629a75856f4bafdbff3cc165c373f8dc2.1719319093.git.oleksii.kurochko@gmail.com (mailing list archive)
State New
Headers show
Series Enable build of full Xen for RISC-V | expand

Commit Message

Oleksii June 25, 2024, 1:51 p.m. UTC
During Gitlab CI randconfig job for RISC-V failed witn an error:
 common/trace.c:57:22: error: expected '=', ',', ';', 'asm' or
                              '__attribute__' before '__read_mostly'
   57 | static u32 data_size __read_mostly;

Signed-off-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>
---
 Changes in V13:
  - new patch
---
 xen/common/trace.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Jan Beulich June 25, 2024, 2:25 p.m. UTC | #1
On 25.06.2024 15:51, Oleksii Kurochko wrote:
> During Gitlab CI randconfig job for RISC-V failed witn an error:
>  common/trace.c:57:22: error: expected '=', ',', ';', 'asm' or
>                               '__attribute__' before '__read_mostly'
>    57 | static u32 data_size __read_mostly;
> 
> Signed-off-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>

Acked-by: Jan Beulich <jbeulich@suse.com>

If you give a release-ack, this can go in right away, I think.

Jan
Oleksii June 25, 2024, 4:23 p.m. UTC | #2
On Tue, 2024-06-25 at 16:25 +0200, Jan Beulich wrote:
> On 25.06.2024 15:51, Oleksii Kurochko wrote:
> > During Gitlab CI randconfig job for RISC-V failed witn an error:
> >  common/trace.c:57:22: error: expected '=', ',', ';', 'asm' or
> >                               '__attribute__' before
> > '__read_mostly'
> >    57 | static u32 data_size __read_mostly;
> > 
> > Signed-off-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>
> 
> Acked-by: Jan Beulich <jbeulich@suse.com>
> 
> If you give a release-ack, this can go in right away, I think.
Release-Acked-by: Oleksii Kurochko  <oleksii.kurochko@gmail.com>

~ Oleksii
Jan Beulich June 26, 2024, 10:46 a.m. UTC | #3
On 25.06.2024 18:23, Oleksii wrote:
> On Tue, 2024-06-25 at 16:25 +0200, Jan Beulich wrote:
>> On 25.06.2024 15:51, Oleksii Kurochko wrote:
>>> During Gitlab CI randconfig job for RISC-V failed witn an error:
>>>  common/trace.c:57:22: error: expected '=', ',', ';', 'asm' or
>>>                               '__attribute__' before
>>> '__read_mostly'
>>>    57 | static u32 data_size __read_mostly;
>>>
>>> Signed-off-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>
>>
>> Acked-by: Jan Beulich <jbeulich@suse.com>
>>
>> If you give a release-ack, this can go in right away, I think.
> Release-Acked-by: Oleksii Kurochko  <oleksii.kurochko@gmail.com>

Thanks, but actually I was misled by the subject prefix. From a formal
perspective this really wants an ack from George (and mine doesn't
count anything at all).

Jan
George Dunlap June 27, 2024, 3:27 p.m. UTC | #4
On Wed, Jun 26, 2024 at 11:46 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> On 25.06.2024 18:23, Oleksii wrote:
> > On Tue, 2024-06-25 at 16:25 +0200, Jan Beulich wrote:
> >> On 25.06.2024 15:51, Oleksii Kurochko wrote:
> >>> During Gitlab CI randconfig job for RISC-V failed witn an error:
> >>>  common/trace.c:57:22: error: expected '=', ',', ';', 'asm' or
> >>>                               '__attribute__' before
> >>> '__read_mostly'
> >>>    57 | static u32 data_size __read_mostly;
> >>>
> >>> Signed-off-by: Oleksii Kurochko <oleksii.kurochko@gmail.com>
> >>
> >> Acked-by: Jan Beulich <jbeulich@suse.com>
> >>
> >> If you give a release-ack, this can go in right away, I think.
> > Release-Acked-by: Oleksii Kurochko  <oleksii.kurochko@gmail.com>
>
> Thanks, but actually I was misled by the subject prefix. From a formal
> perspective this really wants an ack from George (and mine doesn't
> count anything at all).

Acked-by: George Dunlap <george.dunlap@cloud.com>
diff mbox series

Patch

diff --git a/xen/common/trace.c b/xen/common/trace.c
index c9c555094b..c33f115b6c 100644
--- a/xen/common/trace.c
+++ b/xen/common/trace.c
@@ -29,6 +29,7 @@ 
 #include <xen/mm.h>
 #include <xen/percpu.h>
 #include <xen/pfn.h>
+#include <xen/sections.h>
 #include <xen/cpu.h>
 #include <asm/atomic.h>
 #include <public/sysctl.h>