Re: [PULL 00/14] Bsd user for 9.1 patches

[Richard Henderson]

On 7/24/24 04:07, Warner Losh wrote:
> The following changes since commit 3cce8bd4d737f2ca688bbdcb92cd5cc683245bbd:
>
>    Merge tag 'ui-pull-request' ofhttps://gitlab.com/marcandre.lureau/qemu into 
> staging (2024-07-23 15:23:05 +1000)
>
> are available in the Git repository at:
>
>    git@gitlab.com:bsdimp/qemu.git tags/bsd-user-for-9.1-pull-request
>
> for you to fetch changes up to afdb6be1bd8528395af65a087bd668bf7a42ab99:
>
>    bsd-user: Add aarch64 build to tree (2024-07-23 10:56:30 -0600)
>
> ----------------------------------------------------------------
> bsd-user: Misc changes for 9.1 (I hope)
>
> This patch series includes two main sets of patches. To make it simple to
> review, I've included the changes from my student which the later changes depend
> on. I've included a change from Jessica and Doug as well. I've reviewed them,
> but more eyes never hurt.
>
> I've also included a number of 'touch up' patches needed either to get the
> aarch64 building, or to implmement suggestions from prior review cycles. The
> main one is what's charitably described as a kludge: force aarch64 to use 4k
> pages. The qemu-project (and blitz branch) hasn't had the necessary changes to
> bsd-user needed to support variable page size.
>
> Sorry this is so late... Live has conspired to delay me.

Something didn't get committed properly, as it doesn't build:

$ ninja
[343/1144] Compiling C object 
libqemu-aarch64-bsd-user.a.p/bsd-user_aarch64_target_arch_cpu.c.o
FAILED: libqemu-aarch64-bsd-user.a.p/bsd-user_aarch64_target_arch_cpu.c.o
cc -m64 -Ilibqemu-aarch64-bsd-user.a.p -I. -I../src -Itarget/arm -I../src/target/arm 
-I../src/common-user/host/x86_64 -I../src/bsd-user/include -Ibsd-user/freebsd 
-I../src/bsd-user/freebsd -I../src/bsd-user/host/x86_64 -Ibsd-user -I../src/bsd-user 
-I../src/bsd-user/aarch64 -Iqapi -Itrace -Iui/shader -I/usr/local/include/capstone 
-I/usr/local/include/glib-2.0 -I/usr/local/lib/glib-2.0/include -I/usr/local/include 
-fdiagnostics-color=auto -Wall -Winvalid-pch -std=gnu11 -O2 -g -fstack-protector-strong 
-Wempty-body -Wendif-labels -Wexpansion-to-defined -Wformat-security -Wformat-y2k 
-Wignored-qualifiers -Winit-self -Wmissing-format-attribute -Wmissing-prototypes 
-Wnested-externs -Wold-style-definition -Wredundant-decls -Wstrict-prototypes 
-Wtype-limits -Wundef -Wvla -Wwrite-strings -Wno-gnu-variable-sized-type-not-at-end 
-Wno-initializer-overrides -Wno-missing-include-dirs -Wno-psabi -Wno-shift-negative-value 
-Wno-string-plus-int -Wno-tautological-type-limit-compare -Wno-typedef-redefinition 
-Wthread-safety -iquote . -iquote /home/rth/qemu/src -iquote /home/rth/qemu/src/include 
-iquote /home/rth/qemu/src/host/include/x86_64 -iquote 
/home/rth/qemu/src/host/include/generic -iquote /home/rth/qemu/src/tcg/i386 -pthread 
-msse2 -mcx16 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE 
-fno-strict-aliasing -fno-common -fwrapv -ftrivial-auto-var-init=zero 
-fzero-call-used-regs=used-gpr -fPIE -DCOMPILING_PER_TARGET 
'-DCONFIG_TARGET="aarch64-bsd-user-config-target.h"' 
'-DCONFIG_DEVICES="aarch64-bsd-user-config-devices.h"' -MD -MQ 
libqemu-aarch64-bsd-user.a.p/bsd-user_aarch64_target_arch_cpu.c.o -MF 
libqemu-aarch64-bsd-user.a.p/bsd-user_aarch64_target_arch_cpu.c.o.d -o 
libqemu-aarch64-bsd-user.a.p/bsd-user_aarch64_target_arch_cpu.c.o -c 
../src/bsd-user/aarch64/target_arch_cpu.c
In file included from ../src/bsd-user/aarch64/target_arch_cpu.c:20:
In file included from ../src/bsd-user/aarch64/target_arch.h:23:
../src/bsd-user/qemu.h:38:10: fatal error: 'target.h' file not found
   38 | #include "target.h"
      |          ^~~~~~~~~~
1 error generated.


r~