From patchwork Thu Sep 16 15:12:35 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Laurent Vivier X-Patchwork-Id: 512938 Delivered-To: patch@linaro.org Received: by 2002:a02:c816:0:0:0:0:0 with SMTP id p22csp1315789jao; Thu, 16 Sep 2021 08:24:35 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzHv7jRqYaa4y5sfDz55sGldAsxZ8WC/B6345L8SwZB9QM2f+6XS+PEr1PgRMP5z1Aihgl1 X-Received: by 2002:a37:9cc8:: with SMTP id f191mr5488481qke.113.1631805875302; Thu, 16 Sep 2021 08:24:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1631805875; cv=none; d=google.com; s=arc-20160816; b=t/Km4um48x2Dmfy7hrpRn9FT24LDvRTmFVbZgqOvVYCXQjU4ZypjDkgOz2W9dz2Ffp 1YsrwH5Ti+1Ep9LfdElZSrS7dgcGIpHbl1AWBoTpQW2CzH/cZLOTTm2IwvBwrs6cdBaR nSP+MN65ZNXrPp6g5a6zS0kQga0NJlSgoXQR7ugpIgLRSP8X1ZARksF/yHoE0nW9E3a7 W4lLONP9D0wbv9qeqNLtc/vvKABOY5RuNhgK6LzVTq42Qm+YYKPYGU1+6wenHAhZATEs c0jtOvgea9CIQrQ9ib2utXCAN5c45PGagTkx8maEiRqBR7Tabe+Xtk6UE8aL4aroniq/ asYQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:cc:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:to:from; bh=NuccEDpT333n8yU4ZD3dwhgG+y+Zqhk9kUDG7Uy2GwA=; b=U4kA4eOozc4o89tY8PGuzV8V10HFZJxvt4CcP+/qY9hXE7S/fbqXEMghsLoZJeHJKp PYnxJSWf47u9p5CADtzUG61nnA2jd/rKcARWV7VSoulGKLkTNa/xdgIOTHz4FPd3szqA ij1XothzXu1fj78q18po/2KYJXTbejjvRM/X8oX6sBF7uSK+rPmsaUUBzGBJZ9mE1mXa jHDxh9uRq6jXrXIq4Zi8Go3EdPOme+ST9pN9rJwCxlvk4pEfjzTORoLohuHCv08I+829 ciNEjaQ3GWzPLikfyTFpQmjB86iFdNhpvHij8Z+8wBI0I1n+4FMKugkujr/0NDECv0Ln wSfg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org" Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id s21si1845265qkg.298.2021.09.16.08.24.35 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Thu, 16 Sep 2021 08:24:35 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org" Received: from localhost ([::1]:54140 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mQtFa-0003rQ-Rr for patch@linaro.org; Thu, 16 Sep 2021 11:24:34 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:36250) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mQt4L-00030T-QY for qemu-devel@nongnu.org; Thu, 16 Sep 2021 11:12:57 -0400 Received: from mout.kundenserver.de ([212.227.17.13]:40985) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mQt4I-0004GH-ML for qemu-devel@nongnu.org; Thu, 16 Sep 2021 11:12:57 -0400 Received: from quad ([82.142.27.6]) by mrelayeu.kundenserver.de (mreue107 [212.227.15.183]) with ESMTPSA (Nemesis) id 1MJEpp-1mAaIa47Q3-00KfbI; Thu, 16 Sep 2021 17:12:44 +0200 From: Laurent Vivier To: qemu-devel@nongnu.org Subject: [PULL 08/10] linux-user: Don't include gdbstub.h in qemu.h Date: Thu, 16 Sep 2021 17:12:35 +0200 Message-Id: <20210916151237.1188301-9-laurent@vivier.eu> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210916151237.1188301-1-laurent@vivier.eu> References: <20210916151237.1188301-1-laurent@vivier.eu> MIME-Version: 1.0 X-Provags-ID: V03:K1:k9SQK62dp9bppCOtgR0jfzRyhPojCs7c9R3310/PyqJk6F0QaMZ bFzFAgEahZqEns+gkn5Y+cZVxUgeveV5AsmX7sORn+Yn3+gucbEMo9yl2zqtWpQ8L5ATKTo f4o55ZmXHCvOo0PxvyWnTFIucoHDJRg5bznHK2OSPvwZiW/VEj5Cxp0NVLKEAWBe8Gtxu0h +vQGu5HSdnsaKzJonBxmQ== X-UI-Out-Filterresults: notjunk:1; V03:K0:Z1wVyVC1350=:CLxW+wuigxGv2tLULeAsJs nMEYU05IZN6opAzxrgjF6S2K4ECq6qKIAeOTD+QuCssvD3GhaZzc74gvn+BM6sBJySPm5mYH+ ZHZuIA+bs1MyuYqAAA2xnpwEmHcAfvv/WNBZqsTEYn6/Je36htDwlqBCEt1iAaoUOO/D53/du KKmqvSG+M7q2PKkDJYt/2+FLNhxqc9NM4YQbrx37qShERqg9RcCDsI45OfV+jE2FUBvh5Vj3f HLFGLaRtYEqhcbOMeytR+KbmOrzJM5tXGVyNrQZ7iODPwJad5xzW0WTbSlpAxyu2+q2+otxd6 IXdPTem9MXjeLCgi6JEX7ryjRJ2UMvxCod5kFh9k2luOdd1wCCjJ9OZFDUXXg9nUuZqZoViCB SqofC7bynK6gHCLDrHriihJwTMYNSE4ntCDWNdnbLb8F5V7Y1baNgC6HGP61kibpgO3vPV9Xc FpAwlz83rkejevgh6CliczyXKKA5zLFgmgCH3V6u33a89JtRHhlG2T+ExgcGf7RbW1ThUy1Dx KxWh+pHQHZ53o5Gf8SE5g80hF62TYYQfb17zlVICTtOb6Q3U+0jD5fDqcM/xwTC2iDYzGxLsk W+P9kRoH3eJOYiEY3Q3MvUUcCiwJi/+z27sqICVxd/hRRyL2OQrE60NpuGpouPooZB6i8bXYI RwoapO5NACuxw2c/4v9j4v10WEGrxAkaTNHiTq4cReyIQSU/mPX3QIWj6tnpgf+dRIPtQXCS3 jiRLv9FwWAk0+uFwci9LEWV2vgAs+a2wH67Ohg== Received-SPF: none client-ip=212.227.17.13; envelope-from=laurent@vivier.eu; helo=mout.kundenserver.de X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , Richard Henderson , =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= , Laurent Vivier Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" From: Peter Maydell Currently the linux-user qemu.h pulls in gdbstub.h. There's no real reason why it should do this; include it directly from the C files which require it, and drop the include line in qemu.h. (Note that several of the C files previously relying on this indirect include were going out of their way to only include gdbstub.h conditionally on not CONFIG_USER_ONLY!) Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210908154405.15417-9-peter.maydell@linaro.org> Signed-off-by: Laurent Vivier --- gdbstub.c | 2 +- linux-user/exit.c | 1 + linux-user/main.c | 1 + linux-user/qemu.h | 1 - linux-user/signal.c | 2 ++ semihosting/arm-compat-semi.c | 2 +- target/m68k/m68k-semi.c | 2 +- target/nios2/nios2-semi.c | 2 +- 8 files changed, 8 insertions(+), 5 deletions(-) -- 2.31.1 diff --git a/gdbstub.c b/gdbstub.c index 5d8e6ae3cd9b..36b85aa50e24 100644 --- a/gdbstub.c +++ b/gdbstub.c @@ -31,13 +31,13 @@ #include "qemu/cutils.h" #include "qemu/module.h" #include "trace/trace-root.h" +#include "exec/gdbstub.h" #ifdef CONFIG_USER_ONLY #include "qemu.h" #else #include "monitor/monitor.h" #include "chardev/char.h" #include "chardev/char-fe.h" -#include "exec/gdbstub.h" #include "hw/cpu/cluster.h" #include "hw/boards.h" #endif diff --git a/linux-user/exit.c b/linux-user/exit.c index 68a3a6f9df01..fa6ef0b9b44c 100644 --- a/linux-user/exit.c +++ b/linux-user/exit.c @@ -17,6 +17,7 @@ * along with this program; if not, see . */ #include "qemu/osdep.h" +#include "exec/gdbstub.h" #include "qemu.h" #include "user-internals.h" #ifdef CONFIG_GPROF diff --git a/linux-user/main.c b/linux-user/main.c index 9edc0b222076..5ce17e423db1 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@ -40,6 +40,7 @@ #include "qemu/module.h" #include "qemu/plugin.h" #include "exec/exec-all.h" +#include "exec/gdbstub.h" #include "tcg/tcg.h" #include "qemu/timer.h" #include "qemu/envlist.h" diff --git a/linux-user/qemu.h b/linux-user/qemu.h index 92290a55c0de..fda90fc28d68 100644 --- a/linux-user/qemu.h +++ b/linux-user/qemu.h @@ -11,7 +11,6 @@ #include "syscall_defs.h" #include "target_syscall.h" -#include "exec/gdbstub.h" /* * This is the size of the host kernel's sigset_t, needed where we make diff --git a/linux-user/signal.c b/linux-user/signal.c index 6af66123d0f7..f8346f5ec5fd 100644 --- a/linux-user/signal.c +++ b/linux-user/signal.c @@ -18,6 +18,8 @@ */ #include "qemu/osdep.h" #include "qemu/bitops.h" +#include "exec/gdbstub.h" + #include #include diff --git a/semihosting/arm-compat-semi.c b/semihosting/arm-compat-semi.c index 1c29146dcfa6..01badea99c81 100644 --- a/semihosting/arm-compat-semi.c +++ b/semihosting/arm-compat-semi.c @@ -37,12 +37,12 @@ #include "semihosting/console.h" #include "semihosting/common-semi.h" #include "qemu/timer.h" +#include "exec/gdbstub.h" #ifdef CONFIG_USER_ONLY #include "qemu.h" #define COMMON_SEMI_HEAP_SIZE (128 * 1024 * 1024) #else -#include "exec/gdbstub.h" #include "qemu/cutils.h" #ifdef TARGET_ARM #include "hw/arm/boot.h" diff --git a/target/m68k/m68k-semi.c b/target/m68k/m68k-semi.c index d919245e4f85..44ec7e4612c6 100644 --- a/target/m68k/m68k-semi.c +++ b/target/m68k/m68k-semi.c @@ -20,11 +20,11 @@ #include "qemu/osdep.h" #include "cpu.h" +#include "exec/gdbstub.h" #if defined(CONFIG_USER_ONLY) #include "qemu.h" #define SEMIHOSTING_HEAP_SIZE (128 * 1024 * 1024) #else -#include "exec/gdbstub.h" #include "exec/softmmu-semi.h" #include "hw/boards.h" #endif diff --git a/target/nios2/nios2-semi.c b/target/nios2/nios2-semi.c index e508b2fafceb..fe5598bae4d7 100644 --- a/target/nios2/nios2-semi.c +++ b/target/nios2/nios2-semi.c @@ -24,11 +24,11 @@ #include "qemu/osdep.h" #include "cpu.h" +#include "exec/gdbstub.h" #if defined(CONFIG_USER_ONLY) #include "qemu.h" #else #include "qemu-common.h" -#include "exec/gdbstub.h" #include "exec/softmmu-semi.h" #endif #include "qemu/log.h"