[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 04/19] bsd-user: Clean up includes
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH v4 04/19] bsd-user: Clean up includes |
Date: |
Sat, 28 Jan 2023 05:29:47 -0500 |
On Fri, Jan 27, 2023 at 10:01:57AM -0500, Michael S. Tsirkin wrote:
> On Fri, Jan 27, 2023 at 02:54:30PM +0000, Peter Maydell wrote:
> > On Thu, 19 Jan 2023 at 14:42, Warner Losh <imp@bsdimp.com> wrote:
> > >
> > > 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?
> >
> > The rough rule of thumb is that if some OS needs a compatibility
> > fixup or workaround for a system header (eg not every mmap.h
> > defines MAP_ANONYMOUS; on Windows unistd.h has to come before
> > time.h) then we put that header include and the compat workaround
> > into osdep.h. This avoids "only fails on obscure platform" issues
> > where somebody puts a header include into some specific .c file
> > but not the compat workaround, and it works on the Linux host
> > that most people develop and test on and we only find the
> > problem later.
> >
> > There's also no doubt some includes there for historical
> > reasons, and some which really are "everybody needs these"
> > convenience ones. But we should probably not add new
> > includes to osdep.h unless they fall into the "working around
> > system header issues" bucket.
> >
> > thanks
> > -- PMM
>
>
> BTW maybe we should teach checkpatch about that rule:
> if a header is in osdep do not include it directly.
To be more precise, make checkpatch run clean-includes somehow?
Or just make CI run clean-includes on the tree and verify result
is empty?
> --
> MST
- Re: [PATCH v4 16/19] Fix non-first inclusions of qemu/osdep.h, (continued)
- [PATCH v4 11/19] net: Clean up includes, Markus Armbruster, 2023/01/19
- [PATCH v4 04/19] bsd-user: Clean up includes, Markus Armbruster, 2023/01/19
- Re: [PATCH v4 04/19] bsd-user: Clean up includes, Peter Maydell, 2023/01/27
- Re: [PATCH v4 04/19] bsd-user: Clean up includes, Michael S. Tsirkin, 2023/01/27
- Re: [PATCH v4 04/19] bsd-user: Clean up includes,
Michael S. Tsirkin <=
- Re: [PATCH v4 04/19] bsd-user: Clean up includes, Markus Armbruster, 2023/01/30
[PATCH v4 17/19] Don't include headers already included by qemu/osdep.h, Markus Armbruster, 2023/01/19
[PATCH v4 08/19] hw/tricore: Clean up includes, Markus Armbruster, 2023/01/19
[PATCH v4 09/19] qga: Clean up includes, Markus Armbruster, 2023/01/19
[PATCH v4 13/19] riscv: Clean up includes, Markus Armbruster, 2023/01/19