On 22/10/2020 11:13, Jan Kiszka wrote: > On 22.10.20 10:13, Andrea Bastoni wrote: >> On 22/10/2020 07:21, Jan Kiszka wrote: >>> On 21.10.20 21:23, Andrea Bastoni wrote: >>>>>> The patch series fixes the warnings generated by "-Wextra -Wundef >>>>>> -Wnested-externs -Wshadow -Wredundant-decls -Wdeprecated", and two small >>>>>> bugs: >>>>>> >>>>>> x86: bitops: only x86_64 is supported, avoid picking the wrong default >>>>>> suffix >>>>> >>>>> How did you come across that one? Luckily, we only ever set bit 0... >>>> >>>> Saw this warning and took a look at the disassembly. Panic is probably not >>>> so >>>> common anyway. >>>> >>>> hypervisor/arch/x86/include/asm/bitops.h:54: Warning: no instruction >>>> mnemonic >>>> suffix given and no register operands; using default for `bts' >>>> >>> >>> gcc-10, right? I'm not getting it with 9.3.1 which I have here. >> >> Yes. I also tried gcc-8 (gcc-8 (Debian 8.4.0-4) 8.4.0) and I was getting it >> as well. >> > > Maybe rather a binutils version thing? I'm on 2.32.0.
Looks like it. I see the warning with 2.35.1, I don't see it with 2.34. -- Thanks, Andrea Bastoni -- You received this message because you are subscribed to the Google Groups "Jailhouse" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jailhouse-dev/359e5ab7-1d29-8ea0-66c9-3bb6644a7757%40tum.de.
