Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-09 Thread Paolo Bonzini
On 09/07/20 08:59, Claudio Fontana wrote: > anything else that we could use to find the real problem? I'm using $ gcc -v ... gcc version 8.3.1 20191121 (Red Hat 8.3.1-5) (GCC) $ ld -v GNU ld version 2.30-68.el8 > Of course I can try a blind fix, where I suggest to explicitly provide the >

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-09 Thread Claudio Fontana
On 7/8/20 8:41 PM, Paolo Bonzini wrote: > > > Il mer 8 lug 2020, 20:25 Claudio Fontana > ha scritto: > > What I did notice is that all the code that directly or indirectly uses > the functions is under an > > if (0) ( > ) > > since tcg_enabled is the

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Philippe Mathieu-Daudé
On 7/7/20 8:37 PM, Peter Maydell wrote: > On Mon, 6 Jul 2020 at 17:48, Paolo Bonzini wrote: >> >> The following changes since commit fc1bff958998910ec8d25db86cd2f53ff125f7ab: >> >> hw/misc/pca9552: Add missing TypeInfo::class_size field (2020-06-29 >> 21:16:10 +0100) >> >> are available in the

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Paolo Bonzini
On 08/07/20 18:13, Claudio Fontana wrote: > so I don't get any tcg_get_icount_limit() compiled in, and no > errors. > > I think that having comparable configure command line and compiler > version/flags would help me pin down any related issue. Are you using link-time optimization by chance?

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Paolo Bonzini
On 08/07/20 18:45, Claudio Fontana wrote: > C++ is used to link the final qemu-system binary and on my system c++ has LTO: > > c++ -v > Using built-in specs. > COLLECT_GCC=c++ > COLLECT_LTO_WRAPPER=/usr/lib64/gcc/x86_64-suse-linux/7/lto-wrapper > OFFLOAD_TARGET_NAMES=hsa:nvptx-none > Target:

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Paolo Bonzini
Il mer 8 lug 2020, 20:25 Claudio Fontana ha scritto: > What I did notice is that all the code that directly or indirectly uses > the functions is under an > > if (0) ( > ) > > since tcg_enabled is the constant 0. > > By "indirectly" I mean that the static void qemu_tcg_cpu_thread_fn() > function

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Claudio Fontana
On 7/8/20 8:25 PM, Claudio Fontana wrote: > On 7/8/20 7:03 PM, Claudio Fontana wrote: >> On 7/8/20 6:55 PM, Paolo Bonzini wrote: >>> On 08/07/20 18:45, Claudio Fontana wrote: C++ is used to link the final qemu-system binary and on my system c++ has LTO: c++ -v Using

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Claudio Fontana
On 7/8/20 7:03 PM, Claudio Fontana wrote: > On 7/8/20 6:55 PM, Paolo Bonzini wrote: >> On 08/07/20 18:45, Claudio Fontana wrote: >>> C++ is used to link the final qemu-system binary and on my system c++ has >>> LTO: >>> >>> c++ -v >>> Using built-in specs. >>> COLLECT_GCC=c++ >>>

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Claudio Fontana
On 7/8/20 6:55 PM, Paolo Bonzini wrote: > On 08/07/20 18:45, Claudio Fontana wrote: >> C++ is used to link the final qemu-system binary and on my system c++ has >> LTO: >> >> c++ -v >> Using built-in specs. >> COLLECT_GCC=c++ >> COLLECT_LTO_WRAPPER=/usr/lib64/gcc/x86_64-suse-linux/7/lto-wrapper

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Claudio Fontana
On 7/8/20 6:16 PM, Paolo Bonzini wrote: > On 08/07/20 18:13, Claudio Fontana wrote: >> so I don't get any tcg_get_icount_limit() compiled in, and no >> errors. >> >> I think that having comparable configure command line and compiler >> version/flags would help me pin down any related issue. > >

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-08 Thread Claudio Fontana
On 7/7/20 8:37 PM, Peter Maydell wrote: > On Mon, 6 Jul 2020 at 17:48, Paolo Bonzini wrote: >> >> The following changes since commit fc1bff958998910ec8d25db86cd2f53ff125f7ab: >> >> hw/misc/pca9552: Add missing TypeInfo::class_size field (2020-06-29 >> 21:16:10 +0100) >> >> are available in the

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-07 Thread Paolo Bonzini
Il mar 7 lug 2020, 20:42 Peter Maydell ha scritto: > Also it broke my working tree, in the sense that when I > rolled back to current master incremental-rebuild didn't > work but instead failed with: > > make: *** No rule to make target '/home/ubuntu/qemu/Kconfig', needed > by

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-07 Thread Peter Maydell
On Tue, 7 Jul 2020 at 19:37, Peter Maydell wrote: > > On Mon, 6 Jul 2020 at 17:48, Paolo Bonzini wrote: > > > > The following changes since commit fc1bff958998910ec8d25db86cd2f53ff125f7ab: > > > > hw/misc/pca9552: Add missing TypeInfo::class_size field (2020-06-29 > > 21:16:10 +0100) > > > >

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-07 Thread Peter Maydell
On Mon, 6 Jul 2020 at 17:48, Paolo Bonzini wrote: > > The following changes since commit fc1bff958998910ec8d25db86cd2f53ff125f7ab: > > hw/misc/pca9552: Add missing TypeInfo::class_size field (2020-06-29 > 21:16:10 +0100) > > are available in the Git repository at: > >

Re: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze

2020-07-06 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200706164155.24696-1-pbonz...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PULL 00/53] Misc patches for QEMU 5.1 soft freeze Type: series Message-id: