diff mbox series

hw/core/cpu: Add missing 'exec/cpu-common.h' include

Message ID 20200908123433.105706-1-philmd@redhat.com
State New
Headers show
Series hw/core/cpu: Add missing 'exec/cpu-common.h' include | expand

Commit Message

Philippe Mathieu-Daudé Sept. 8, 2020, 12:34 p.m. UTC
cpu_common_reset() uses tcg_flush_softmmu_tlb() which is
declared in "exec/cpu-common.h". Add the missing header
to avoid when refactoring other headers:

  hw/core/cpu.c: In function ‘cpu_common_reset’:
  hw/core/cpu.c:273:9: error: implicit declaration of function ‘tcg_flush_softmmu_tlb’ [-Werror=implicit-function-declaration]
    273 |         tcg_flush_softmmu_tlb(cpu);
        |         ^~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 hw/core/cpu.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Philippe Mathieu-Daudé Oct. 1, 2020, 5:44 p.m. UTC | #1
ping?

On 9/8/20 2:34 PM, Philippe Mathieu-Daudé wrote:
> cpu_common_reset() uses tcg_flush_softmmu_tlb() which is

> declared in "exec/cpu-common.h". Add the missing header

> to avoid when refactoring other headers:

> 

>   hw/core/cpu.c: In function ‘cpu_common_reset’:

>   hw/core/cpu.c:273:9: error: implicit declaration of function ‘tcg_flush_softmmu_tlb’ [-Werror=implicit-function-declaration]

>     273 |         tcg_flush_softmmu_tlb(cpu);

>         |         ^~~~~~~~~~~~~~~~~~~~~

> 

> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> ---

>  hw/core/cpu.c | 1 +

>  1 file changed, 1 insertion(+)

> 

> diff --git a/hw/core/cpu.c b/hw/core/cpu.c

> index 22bc3f974a2..63ece3662d6 100644

> --- a/hw/core/cpu.c

> +++ b/hw/core/cpu.c

> @@ -26,6 +26,7 @@

>  #include "qemu/log.h"

>  #include "qemu/main-loop.h"

>  #include "exec/log.h"

> +#include "exec/cpu-common.h"

>  #include "qemu/error-report.h"

>  #include "qemu/qemu-print.h"

>  #include "sysemu/tcg.h"

>
Eduardo Habkost Oct. 1, 2020, 5:56 p.m. UTC | #2
On Thu, Oct 01, 2020 at 07:44:57PM +0200, Philippe Mathieu-Daudé wrote:
> ping?

> 

> On 9/8/20 2:34 PM, Philippe Mathieu-Daudé wrote:

> > cpu_common_reset() uses tcg_flush_softmmu_tlb() which is

> > declared in "exec/cpu-common.h". Add the missing header

> > to avoid when refactoring other headers:

> > 

> >   hw/core/cpu.c: In function ‘cpu_common_reset’:

> >   hw/core/cpu.c:273:9: error: implicit declaration of function ‘tcg_flush_softmmu_tlb’ [-Werror=implicit-function-declaration]

> >     273 |         tcg_flush_softmmu_tlb(cpu);

> >         |         ^~~~~~~~~~~~~~~~~~~~~

> > 

> > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>


Qeueued, thanks!

-- 
Eduardo
diff mbox series

Patch

diff --git a/hw/core/cpu.c b/hw/core/cpu.c
index 22bc3f974a2..63ece3662d6 100644
--- a/hw/core/cpu.c
+++ b/hw/core/cpu.c
@@ -26,6 +26,7 @@ 
 #include "qemu/log.h"
 #include "qemu/main-loop.h"
 #include "exec/log.h"
+#include "exec/cpu-common.h"
 #include "qemu/error-report.h"
 #include "qemu/qemu-print.h"
 #include "sysemu/tcg.h"