From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id ACAD3C004D4 for ; Thu, 19 Jan 2023 14:43:12 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pIW7k-0003x5-Jb; Thu, 19 Jan 2023 09:42:40 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pIW7i-0003uo-UQ for qemu-devel@nongnu.org; Thu, 19 Jan 2023 09:42:38 -0500 Received: from mail-ej1-x632.google.com ([2a00:1450:4864:20::632]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1pIW7f-0000BJ-D0 for qemu-devel@nongnu.org; Thu, 19 Jan 2023 09:42:38 -0500 Received: by mail-ej1-x632.google.com with SMTP id u19so6190747ejm.8 for ; Thu, 19 Jan 2023 06:42:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20210112.gappssmtp.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=WLm/26TDzMxR9AjqcQLVzx0EJOKTTp2Q5NhztMFa29c=; b=TBMCPeFMINvlR6sLfqmBm9/vaLQjMVVaZZNdyj3wfzPAho/+69TQfPAGmXBfcQgvE2 3MvTFGEuGGhy10YgDigUsENiJ0WQ9QHP+FCn+M4x9x1Gv+33h+zvfPdAz3VQg5YfJ15C RBq8K4q5oaxmGElBG2xYzudAxiovhsDns7/JEomVxBoexfkvAbuMk7v7jiLFWlwH6Lf+ v6NVPChBDRjuviN5jqAQLygyfD4s3RMzQF08cYUy38D0qCHIYTBopdBA07F+x7d87w49 JejKb5gPur7f5jPjMFbXfHrzmRW0tlsWHSQz65j4IKCwbpU5xvBVjNpUzinQJltxSBAM px9w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=WLm/26TDzMxR9AjqcQLVzx0EJOKTTp2Q5NhztMFa29c=; b=Fzko+jkYF0X5k+aXNVm9sRr+OYI0d/E4Viyket08HRv11zPHbX9KOsoMKXGPPOS92E 1QHd5LaeuQUxlMrTB+nSTtYgzB6hrcegVFQfWEPjfVUMO+kLLqCEzzhK7wNJjc2QRD8E Xj3aqyeELU1/k7x+Ey0I/eFYEifv5BwWZ2dKteaC0IzIK+sQ2MUcMXWbzEOg4Qh8k0oD 5ecYc86EkI+cuhcJsIi6FIZLJgxpiAp90czdGRdbOMfoi+EgtM8vLxWf46EUFuIs9X+I ySg2izFNjoYdp0b+IxOBDuhxWEBiEDnulVT1UQvDzRI8HXS6oBJG4MFRg8hqno4hDIhR 91aw== X-Gm-Message-State: AFqh2kqZnGCZbc7wVO3gIRU0tCphhXIucOlnNQoIW6uRsiBt1ijcKVCA b4SB78j9v+aLWc6aqJhtvqDdzvfV9S7xzPr8GrJ6Nw== X-Google-Smtp-Source: AMrXdXv3voXsg9/2Y7jvMhjyvqkA6fh7HJ4k9HF9m+X+9XeI2roUUcwozH3kKKuRBWbh7Eo3g8KfVZP0g0ezvWD0IIA= X-Received: by 2002:a17:906:a3c3:b0:870:7c88:466e with SMTP id ca3-20020a170906a3c300b008707c88466emr788492ejb.140.1674139353359; Thu, 19 Jan 2023 06:42:33 -0800 (PST) MIME-Version: 1.0 References: <20230119065959.3104012-1-armbru@redhat.com> <20230119065959.3104012-5-armbru@redhat.com> In-Reply-To: <20230119065959.3104012-5-armbru@redhat.com> From: Warner Losh Date: Thu, 19 Jan 2023 07:42:46 -0700 Message-ID: Subject: Re: [PATCH v4 04/19] bsd-user: Clean up includes To: Markus Armbruster Cc: qemu-devel@nongnu.org, richard.henderson@linaro.org, pbonzini@redhat.com, kwolf@redhat.com, hreitz@redhat.com, kevans@freebsd.org, berrange@redhat.com, groug@kaod.org, qemu_oss@crudebyte.com, mst@redhat.com, philmd@linaro.org, peter.maydell@linaro.org, alistair@alistair23.me, jasowang@redhat.com, jonathan.cameron@huawei.com, kbastian@mail.uni-paderborn.de, quintela@redhat.com, dgilbert@redhat.com, michael.roth@amd.com, kkostiuk@redhat.com, tsimpson@quicinc.com, palmer@dabbelt.com, bin.meng@windriver.com, qemu-block@nongnu.org, qemu-arm@nongnu.org, qemu-riscv@nongnu.org Content-Type: multipart/alternative; boundary="000000000000763fdf05f29ef11f" Received-SPF: none client-ip=2a00:1450:4864:20::632; envelope-from=wlosh@bsdimp.com; helo=mail-ej1-x632.google.com 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, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org --000000000000763fdf05f29ef11f Content-Type: text/plain; charset="UTF-8" On Thu, Jan 19, 2023 at 12:00 AM Markus Armbruster wrote: > Clean up includes so that osdep.h is included first and headers > which it implies are not included manually. > > This commit was created with scripts/clean-includes. > > Signed-off-by: Markus Armbruster > --- > bsd-user/bsd-proc.h | 4 ---- > bsd-user/qemu.h | 1 - > bsd-user/arm/signal.c | 1 + > bsd-user/arm/target_arch_cpu.c | 2 ++ > bsd-user/freebsd/os-sys.c | 1 + > bsd-user/i386/signal.c | 1 + > bsd-user/i386/target_arch_cpu.c | 3 +-- > bsd-user/main.c | 4 +--- > bsd-user/strace.c | 1 - > bsd-user/x86_64/signal.c | 1 + > bsd-user/x86_64/target_arch_cpu.c | 3 +-- > 11 files changed, 9 insertions(+), 13 deletions(-) > > diff --git a/bsd-user/bsd-proc.h b/bsd-user/bsd-proc.h > index 68b66e571d..a1061bffb8 100644 > --- a/bsd-user/bsd-proc.h > +++ b/bsd-user/bsd-proc.h > @@ -20,11 +20,7 @@ > #ifndef BSD_PROC_H_ > #define BSD_PROC_H_ > > -#include > -#include > -#include > #include > Did you test this on FreeBSD 12? FreeBSD 13 has started to climb the hill where all includes are independent, but much of that hasn't been merged to 12 yet. sys/types.h, at least, is documented as a prereq for both sys/stat.h and sys/resource.h. I know many of these are in os-dep.h, and I've had trouble in the bsd-user fork with that and the layout of the code which has arguably way too much in the .h files. Also, why didn't you move sys/resource.h and other such files to os-dep.h? I'm struggling to understand the rules around what is or isn't included where? > -#include > > /* exit(2) */ > static inline abi_long do_bsd_exit(void *cpu_env, abi_long arg1) > diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h > index be6105385e..0ceecfb6df 100644 > --- a/bsd-user/qemu.h > +++ b/bsd-user/qemu.h > @@ -17,7 +17,6 @@ > #ifndef QEMU_H > #define QEMU_H > > -#include "qemu/osdep.h" > #include "cpu.h" > #include "qemu/units.h" > #include "exec/cpu_ldst.h" > diff --git a/bsd-user/arm/signal.c b/bsd-user/arm/signal.c > index 2b1dd745d1..9734407543 100644 > --- a/bsd-user/arm/signal.c > +++ b/bsd-user/arm/signal.c > @@ -17,6 +17,7 @@ > * along with this program; if not, see . > */ > > +#include "qemu/osdep.h" > #include "qemu.h" > > /* > diff --git a/bsd-user/arm/target_arch_cpu.c > b/bsd-user/arm/target_arch_cpu.c > index 02bf9149d5..fe38ae2210 100644 > --- a/bsd-user/arm/target_arch_cpu.c > +++ b/bsd-user/arm/target_arch_cpu.c > @@ -16,6 +16,8 @@ > * You should have received a copy of the GNU General Public License > * along with this program; if not, see . > */ > + > +#include "qemu/osdep.h" > #include "target_arch.h" > > void target_cpu_set_tls(CPUARMState *env, target_ulong newtls) > diff --git a/bsd-user/freebsd/os-sys.c b/bsd-user/freebsd/os-sys.c > index 309e27b9d6..1676ec10f8 100644 > --- a/bsd-user/freebsd/os-sys.c > +++ b/bsd-user/freebsd/os-sys.c > @@ -17,6 +17,7 @@ > * along with this program; if not, see . > */ > > +#include "qemu/osdep.h" > #include "qemu.h" > #include "target_arch_sysarch.h" > > diff --git a/bsd-user/i386/signal.c b/bsd-user/i386/signal.c > index 5dd975ce56..a3131047b8 100644 > --- a/bsd-user/i386/signal.c > +++ b/bsd-user/i386/signal.c > @@ -17,6 +17,7 @@ > * along with this program; if not, see . > */ > > +#include "qemu/osdep.h" > #include "qemu.h" > > /* > diff --git a/bsd-user/i386/target_arch_cpu.c > b/bsd-user/i386/target_arch_cpu.c > index d349e45299..2a3af2ddef 100644 > --- a/bsd-user/i386/target_arch_cpu.c > +++ b/bsd-user/i386/target_arch_cpu.c > @@ -17,9 +17,8 @@ > * along with this program; if not, see . > */ > > -#include > - > #include "qemu/osdep.h" > + > #include "cpu.h" > #include "qemu.h" > #include "qemu/timer.h" > diff --git a/bsd-user/main.c b/bsd-user/main.c > index 6f09180d65..41290e16f9 100644 > --- a/bsd-user/main.c > +++ b/bsd-user/main.c > @@ -18,12 +18,10 @@ > * along with this program; if not, see . > */ > > -#include > -#include > +#include "qemu/osdep.h" > #include > #include > > -#include "qemu/osdep.h" > #include "qemu/help-texts.h" > #include "qemu/units.h" > #include "qemu/accel.h" > diff --git a/bsd-user/strace.c b/bsd-user/strace.c > index a77d10dd6b..96499751eb 100644 > --- a/bsd-user/strace.c > +++ b/bsd-user/strace.c > @@ -20,7 +20,6 @@ > #include > #include > #include > -#include > > #include "qemu.h" > > diff --git a/bsd-user/x86_64/signal.c b/bsd-user/x86_64/signal.c > index c3875bc4c6..46cb865180 100644 > --- a/bsd-user/x86_64/signal.c > +++ b/bsd-user/x86_64/signal.c > @@ -16,6 +16,7 @@ > * along with this program; if not, see . > */ > > +#include "qemu/osdep.h" > #include "qemu.h" > > /* > diff --git a/bsd-user/x86_64/target_arch_cpu.c > b/bsd-user/x86_64/target_arch_cpu.c > index be7bd10720..1d32f18907 100644 > --- a/bsd-user/x86_64/target_arch_cpu.c > +++ b/bsd-user/x86_64/target_arch_cpu.c > @@ -17,9 +17,8 @@ > * along with this program; if not, see . > */ > > -#include > - > #include "qemu/osdep.h" > + > #include "cpu.h" > #include "qemu.h" > #include "qemu/timer.h" > I suppose these are fine. How do I run the cleanup script? I have 2x the number of files not upstream in the bsd-user fork that I'd like to cleanup as well and they are likely a bigger mess and I'll just upstream them in the messy state unless I understand how to keep the neat :). Warner --000000000000763fdf05f29ef11f Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable


=
On Thu, Jan 19, 2023 at 12:00 AM Mark= us Armbruster <armbru@redhat.com> wrote:
Cle= an up includes so that osdep.h is included first and headers
which it implies are not included manually.

This commit was created with scripts/clean-includes.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
=C2=A0bsd-user/bsd-proc.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0| 4 ----
=C2=A0bsd-user/qemu.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0| 1 -
=C2=A0bsd-user/arm/signal.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0= | 1 +
=C2=A0bsd-user/arm/target_arch_cpu.c=C2=A0 =C2=A0 | 2 ++
=C2=A0bsd-user/freebsd/os-sys.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 1 +
=C2=A0bsd-user/i386/signal.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 | 1 +=
=C2=A0bsd-user/i386/target_arch_cpu.c=C2=A0 =C2=A0| 3 +--
=C2=A0bsd-user/main.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0| 4 +---
=C2=A0bsd-user/strace.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0| 1 -
=C2=A0bsd-user/x86_64/signal.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 | 1 +
=C2=A0bsd-user/x86_64/target_arch_cpu.c | 3 +--
=C2=A011 files changed, 9 insertions(+), 13 deletions(-)

diff --git a/bsd-user/bsd-proc.h b/bsd-user/bsd-proc.h
index 68b66e571d..a1061bffb8 100644
--- a/bsd-user/bsd-proc.h
+++ b/bsd-user/bsd-proc.h
@@ -20,11 +20,7 @@
=C2=A0#ifndef BSD_PROC_H_
=C2=A0#define BSD_PROC_H_

-#include <sys/types.h>
-#include <sys/stat.h>
-#include <sys/time.h>
=C2=A0#include <sys/resource.h>

D= id you test this on FreeBSD 12? FreeBSD 13 has started to climb the hill wh= ere all includes are independent, but much of that hasn't been merged t= o 12 yet. sys/types.h, at least, is documented as a prereq for both sys/sta= t.h and sys/resource.h.

I know many of these are i= n os-dep.h, and I've had trouble in the bsd-user fork with that and the= layout of the code which has arguably way too much in the .h files.
<= div>
Also, why didn't you move sys/resource.h and other s= uch files to os-dep.h? I'm struggling to understand the rules around wh= at is or isn't included where?
=C2=A0
-#include <unistd.h>

=C2=A0/* exit(2) */
=C2=A0static inline abi_long do_bsd_exit(void *cpu_env, abi_long arg1)
diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h
index be6105385e..0ceecfb6df 100644
--- a/bsd-user/qemu.h
+++ b/bsd-user/qemu.h
@@ -17,7 +17,6 @@
=C2=A0#ifndef QEMU_H
=C2=A0#define QEMU_H

-#include "qemu/osdep.h"
=C2=A0#include "cpu.h"
=C2=A0#include "qemu/units.h"
=C2=A0#include "exec/cpu_ldst.h"
diff --git a/bsd-user/arm/signal.c b/bsd-user/arm/signal.c
index 2b1dd745d1..9734407543 100644
--- a/bsd-user/arm/signal.c
+++ b/bsd-user/arm/signal.c
@@ -17,6 +17,7 @@
=C2=A0 *=C2=A0 along with this program; if not, see <http://www.gnu.o= rg/licenses/>.
=C2=A0 */

+#include "qemu/osdep.h"
=C2=A0#include "qemu.h"

=C2=A0/*
diff --git a/bsd-user/arm/target_arch_cpu.c b/bsd-user/arm/target_arch_cpu.= c
index 02bf9149d5..fe38ae2210 100644
--- a/bsd-user/arm/target_arch_cpu.c
+++ b/bsd-user/arm/target_arch_cpu.c
@@ -16,6 +16,8 @@
=C2=A0 *=C2=A0 You should have received a copy of the GNU General Public Li= cense
=C2=A0 *=C2=A0 along with this program; if not, see <http://www.gnu.o= rg/licenses/>.
=C2=A0 */
+
+#include "qemu/osdep.h"
=C2=A0#include "target_arch.h"

=C2=A0void target_cpu_set_tls(CPUARMState *env, target_ulong newtls)
diff --git a/bsd-user/freebsd/os-sys.c b/bsd-user/freebsd/os-sys.c
index 309e27b9d6..1676ec10f8 100644
--- a/bsd-user/freebsd/os-sys.c
+++ b/bsd-user/freebsd/os-sys.c
@@ -17,6 +17,7 @@
=C2=A0 *=C2=A0 along with this program; if not, see <http://www.gnu.o= rg/licenses/>.
=C2=A0 */

+#include "qemu/osdep.h"
=C2=A0#include "qemu.h"
=C2=A0#include "target_arch_sysarch.h"

diff --git a/bsd-user/i386/signal.c b/bsd-user/i386/signal.c
index 5dd975ce56..a3131047b8 100644
--- a/bsd-user/i386/signal.c
+++ b/bsd-user/i386/signal.c
@@ -17,6 +17,7 @@
=C2=A0 *=C2=A0 along with this program; if not, see <http://www.gnu.o= rg/licenses/>.
=C2=A0 */

+#include "qemu/osdep.h"
=C2=A0#include "qemu.h"

=C2=A0/*
diff --git a/bsd-user/i386/target_arch_cpu.c b/bsd-user/i386/target_arch_cp= u.c
index d349e45299..2a3af2ddef 100644
--- a/bsd-user/i386/target_arch_cpu.c
+++ b/bsd-user/i386/target_arch_cpu.c
@@ -17,9 +17,8 @@
=C2=A0 *=C2=A0 along with this program; if not, see <http://www.gnu.o= rg/licenses/>.
=C2=A0 */

-#include <sys/types.h>
-
=C2=A0#include "qemu/osdep.h"
+
=C2=A0#include "cpu.h"
=C2=A0#include "qemu.h"
=C2=A0#include "qemu/timer.h"
diff --git a/bsd-user/main.c b/bsd-user/main.c
index 6f09180d65..41290e16f9 100644
--- a/bsd-user/main.c
+++ b/bsd-user/main.c
@@ -18,12 +18,10 @@
=C2=A0 *=C2=A0 along with this program; if not, see <http://www.gnu.o= rg/licenses/>.
=C2=A0 */

-#include <sys/types.h>
-#include <sys/time.h>
+#include "qemu/osdep.h"
=C2=A0#include <sys/resource.h>
=C2=A0#include <sys/sysctl.h>

-#include "qemu/osdep.h"
=C2=A0#include "qemu/help-texts.h"
=C2=A0#include "qemu/units.h"
=C2=A0#include "qemu/accel.h"
diff --git a/bsd-user/strace.c b/bsd-user/strace.c
index a77d10dd6b..96499751eb 100644
--- a/bsd-user/strace.c
+++ b/bsd-user/strace.c
@@ -20,7 +20,6 @@
=C2=A0#include <sys/select.h>
=C2=A0#include <sys/syscall.h>
=C2=A0#include <sys/ioccom.h>
-#include <ctype.h>

=C2=A0#include "qemu.h"

diff --git a/bsd-user/x86_64/signal.c b/bsd-user/x86_64/signal.c
index c3875bc4c6..46cb865180 100644
--- a/bsd-user/x86_64/signal.c
+++ b/bsd-user/x86_64/signal.c
@@ -16,6 +16,7 @@
=C2=A0 *=C2=A0 along with this program; if not, see <http://www.gnu.o= rg/licenses/>.
=C2=A0 */

+#include "qemu/osdep.h"
=C2=A0#include "qemu.h"

=C2=A0/*
diff --git a/bsd-user/x86_64/target_arch_cpu.c b/bsd-user/x86_64/target_arc= h_cpu.c
index be7bd10720..1d32f18907 100644
--- a/bsd-user/x86_64/target_arch_cpu.c
+++ b/bsd-user/x86_64/target_arch_cpu.c
@@ -17,9 +17,8 @@
=C2=A0 *=C2=A0 along with this program; if not, see <http://www.gnu.o= rg/licenses/>.
=C2=A0 */

-#include <sys/types.h>
-
=C2=A0#include "qemu/osdep.h"
+
=C2=A0#include "cpu.h"
=C2=A0#include "qemu.h"
=C2=A0#include "qemu/timer.h"

I suppose these are fine. How do I run the cleanup script? I have 2x the n= umber of files not upstream in the bsd-user fork that I'd like to clean= up as well and they are likely a bigger mess and I'll just upstream the= m in the messy state unless I understand how to keep the neat :).
=

Warner
--000000000000763fdf05f29ef11f--