On 2020/12/7 下午8:45, Xie XiuQi wrote:

Hi,

Thanks for your report.

The enum variable and function has the same name, we will fix soon. The simplest way is to change a name.

No compilation error is reported in my environment. (gcc 7.3) Maybe the two variables appear in the same scope due to "asm/machine_t.h".

I re-check the latest openEuler-1.0-LTS branch, it's ok.

You are right, machine_t.h was customized by us, not appear in original openEuler-1.0-LTS.

Thanks,

Anyway, we'll send a fix patch soon.

---

Xie XiuQi openEuler kernel sig

On 2020/12/7 17:49, Zhengyuan Liu wrote:

...

Building the latest 2003 LTS can trigger following compilation error:

         In file included from ./include/acpi/acpi_io.h:7:0,
                          from ./include/linux/acpi.h:47,
                          from ./arch/arm64/include/asm/machine_t.h:26,
                          from arch/arm64/kernel/process.c:64:
         ./arch/arm64/include/asm/acpi.h:82:20: error: ‘enable_acpi’ redeclared as different kind of symbol
          static inline void enable_acpi(void)
                             ^~~~~~~~~~~
         In file included from arch/arm64/kernel/process.c:61:0:
         ./arch/arm64/include/asm/mpam_sched.h:46:2: note: previous definition of ‘enable_acpi’ was here
           enable_acpi,

The redefinition was introduced by commit 7e66740ad725 ("MPAM / ACPI: Refactoring MPAM init process and set MPAM ACPI as entrance")