In message <[email protected]>, "Josh
Paetz
el" writes:
>
>
> On Thu, Sep 26, 2019, at 12:34 PM, Warner Losh wrote:
> > Kinda my point exactly...
> >
> > Warner
> >
> > On Thu, Sep 26, 2019 at 11:25 AM Ryan Stone <[email protected]> wrote:
> >> We also shouldn't have ifdef's that change the size of a kernel
> >> structure that's part of the KBI. Isn't struct epoch_tracker part of
> >> the KBI?
> >>
> >> On Thu, Sep 26, 2019 at 12:46 PM Warner Losh <[email protected]> wrote:
> >> >
> >> > But we shouldn't be including opt_foo.h files in sys/*.h files. That's
> the root cause of trouble here.
> >> >
> >> > Warner
> >> >
> >> > On Thu, Sep 26, 2019 at 9:11 AM Niclas Zeising <[email protected]> wr
> ote:
> >> >>
> >> >> On 2019-09-26 17:03, Charlie Li via freebsd-x11 wrote:
> >> >> > Kyle Evans wrote:
> >> >> >> On Thu, Sep 26, 2019 at 9:49 AM Charlie Li wrote:
> >> >> >>> This breaks building the drm-kmod ports, as the build cannot find
> >> >> >>> opt_epoch.h (drm-devel-kmod example shown, drm-current-kmod dies t
> he
> >> >> >>> exact same way):
> >> >> >>>
> >> >> >>> --- linux_anon_inodes.o ---
> >> >> >>> cc -O2 -pipe -fno-strict-aliasing -include
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/dr
> ivers/gpu/drm/drm_os_config.h
> >> >> >>> '-DKBUILD_MODNAME="linuxkpi_gplv2"' -Werror -D_KERNEL -DKLD_MODULE
> >> >> >>> -nostdinc
> >> >> >>> -I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/
> include -I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/lin
> uxkpi/dummy/include
> >> >> >>> -I/wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/
> linuxkpi/gplv2/include
> >> >> >>> -I/usr/src/sys/compat/linuxkpi/common/include -I. -I/usr/src/sys
> >> >> >>> -I/usr/src/sys/contrib/ck/include -fno-common -fno-omit-frame-poin
> ter
> >> >> >>> -mno-omit-leaf-frame-pointer
> >> >> >>> -fdebug-prefix-map=./machine=/usr/src/sys/amd64/include
> >> >> >>> -fdebug-prefix-map=./x86=/usr/src/sys/x86/include -MD
> >> >> >>> -MF.depend.linux_anon_inodes.o -MTlinux_anon_inodes.o -mcmodel=ker
> nel
> >> >> >>> -mno-red-zone -mno-mmx -mno-sse -msoft-float
> >> >> >>> -fno-asynchronous-unwind-tables -ffreestanding -fwrapv -fstack-pro
> tector
> >> >> >>> -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes
> >> >> >>> -Wmissing-prototypes -Wpointer-arith -Wcast-qual -Wundef
> >> >> >>> -Wno-pointer-sign -D__printf__=__freebsd_kprintf__
> >> >> >>> -Wmissing-include-dirs -fdiagnostics-show-option -Wno-unknown-prag
> mas
> >> >> >>> -Wno-error-tautological-compare -Wno-error-empty-body
> >> >> >>> -Wno-error-parentheses-equality -Wno-error-unused-function
> >> >> >>> -Wno-error-pointer-sign -Wno-error-shift-negative-value
> >> >> >>> -Wno-address-of-packed-member -Wno-format-zero-length -Wno-pointer
> -arith
> >> >> >>> -mno-aes -mno-avx -std=iso9899:1999 -c
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/src/linux_anon_inodes.c
> >> >> >>> -o linux_anon_inodes.o
> >> >> >>> In file included from
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/src/linux_anon_inodes.c:12:
> >> >> >>> In file included from
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/include/linux/anon_inodes.h:4:
> >> >> >>> In file included from
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/include/linux/fs.h:6:
> >> >> >>> In file included from
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/include/linux/shrinker.h:5:
> >> >> >>> In file included from
> >> >> >>> /usr/src/sys/compat/linuxkpi/common/include/linux/list.h:56:
> >> >> >>> In file included from /usr/src/sys/net/if_var.h:83:
> >> >> >>> /usr/src/sys/sys/epoch.h:44:10: fatal error: 'opt_epoch.h' file no
> t found
> >> >> >>> #include "opt_epoch.h"
> >> >> >>> ^~~~~~~~~~~~~
> >> >> >>> --- linux_anon_inodefs.o ---
> >> >> >>> In file included from
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/src/linux_anon_inodefs.c:45:
> >> >> >>> In file included from
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/include/linux/debugfs.h:18:
> >> >> >>> In file included from
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/include/linux/fs.h:6:
> >> >> >>> In file included from
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi/gplv2/include/linux/shrinker.h:5:
> >> >> >>> In file included from
> >> >> >>> /usr/src/sys/compat/linuxkpi/common/include/linux/list.h:56:
> >> >> >>> In file included from /usr/src/sys/net/if_var.h:83:
> >> >> >>> /usr/src/sys/sys/epoch.h:44:10: fatal error: 'opt_epoch.h' file no
> t found
> >> >> >>> #include "opt_epoch.h"
> >> >> >>> ^~~~~~~~~~~~~
> >> >> >>> --- linux_anon_inodes.o ---
> >> >> >>> 1 error generated.
> >> >> >>> *** [linux_anon_inodes.o] Error code 1
> >> >> >>>
> >> >> >>> make[2]: stopped in
> >> >> >>> /wrkdirs/usr/ports/graphics/drm-devel-kmod/work/kms-drm-dc414a9/li
> nuxkpi
> >> >> >>> --- linux_anon_inodefs.o ---
> >> >> >>> 1 error generated.
> >> >> >>> *** [linux_anon_inodefs.o] Error code 1
> >> >> >>>
> >> >> >>> Interestingly enough, does not happen when drm-current-kmod is bui
> lt as
> >> >> >>> part of buildkernel (using an existing installed package with SOUR
> CE on).
> >> >> >>>
> >> >> >>
> >> >> >> FWIW, johalun noticed this yesterday and addressed it here:
> >> >> >> https://github.com/FreeBSDDesktop/kms-drm/commit/b486949e7e9f0cfe8d
> ac5f0ac7fe1a660300981d
> >> >> >>
> >> >> > Ah, of course I would miss these commits in the kms-drm repo,
> >> >> > considering that I watch them roll in. Will wait for the updated
> >> >> > snapshots in ports.
> >> >> >
> >> >>
> >> >> I'll get to updating the ports as soon as I can.
> >> >> Regards
> >> >> --
> >> >> Niclas Zeising
> >> >> _______________________________________________
> >> >> [email protected] mailing list
> >> >> https://lists.freebsd.org/mailman/listinfo/freebsd-x11
> >> >> To unsubscribe, send any mail to "[email protected]"
>
> This commit broke emulators/open-vm-tools (which builds an out of tree if_vmx
> )
>
> Should I chase a fix for it or wait for this to get resolved in src?
virtualbox-ose-kmod:
--- VBoxNetFlt-freebsd.o ---
In file included from VBoxNetFlt-freebsd.c:51:
In file included from /usr/src/sys/net/if_var.h:83:
/usr/src/sys/sys/epoch.h:44:10: fatal error: 'opt_epoch.h' file not found
#include "opt_epoch.h"
^~~~~~~~~~~~~
1 error generated.
*** [VBoxNetFlt-freebsd.o] Error code 1
--
Cheers,
Cy Schubert <[email protected]>
FreeBSD UNIX: <[email protected]> Web: http://www.FreeBSD.org
The need of the many outweighs the greed of the few.
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"
