Summary: | emulators/virtualbox-ose-kmod: build failure/compiler error: 'vm_map_protect' declared here | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | O. Hartmann <ohartmann> | ||||
Component: | Individual Port(s) | Assignee: | Virtualbox Team (Nobody) <vbox> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Only Me | CC: | 0mp, Trond.Endrestol, freebsd, grahamperrin, groenveld, kib, madpilot, olevole, rozhuk.im, swills | ||||
Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(vbox) madpilot: merge-quarterly+ |
||||
Version: | Latest | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Attachments: |
|
Description
O. Hartmann
2021-01-14 16:36:01 UTC
This may be related to: https://cgit.freebsd.org/src/commit/?id=0659df6faddfb27ba54a2cae2a12552cf4f823a0 Created attachment 221687 [details]
patch
Not sure that VM_MAP_PROTECT_SET_PROT must be here, probably VM_MAP_PROTECT_SET_MAXPROT.
At least this allow build.
Here's another instance not corrected by the above patch: --- memobj-r0drv-freebsd.o --- /construction/xports/emulators/virtualbox-ose-kmod/work/VirtualBox-5.2.44/out/freebsd.amd64/release/bin/src/vboxdrv/r0drv/freebsd/memobj-r0drv-freebsd.c:890:80: error: too few arguments to function call, expected 6, have 5 int krc = vm_map_protect(pVmMap, AddrStart, AddrEnd, ProtectionFlags, FALSE); ~~~~~~~~~~~~~~ ^ /usr/src/sys/vm/vm_map.h:517:5: note: 'vm_map_protect' declared here int vm_map_protect(vm_map_t map, vm_offset_t start, vm_offset_t end, ^ 1 error generated. *** [memobj-r0drv-freebsd.o] Error code 1 (In reply to Trond.Endrestol from comment #3) The patch is off by 1. The change to vm_map_protect() affects version 1300135. (In reply to rozhuk.im from comment #2) works fine for me. tested on FreeBSD 13.0-ALPHA2 gf76393a6305 *** Bug 252953 has been marked as a duplicate of this bug. *** I submitted a similar patch on bug #252953. My patch references version 1300135 as affected, but otherwise is the same. I tested my patch in poudriere with all supported versions and it runs fine on my machine (14.0-CURRENT main-c256228-g0d528e63541). BTW this should be committed quickly and also merged to to quarterly 2021Q1, to ensure a working package for 13.0-RELEASE. A commit references this bug: Author: madpilot Date: Tue Jan 26 14:49:23 UTC 2021 New revision: 562683 URL: https://svnweb.freebsd.org/changeset/ports/562683 Log: Fix VirtualBox build on recent head and 13 after src commit 0659df6faddf PR: 252675 Submitted by: O. Hartmann <ohartmann@walstatt.org> Mateusz Piotrowski <0mp@FreeBSD.org> (duplicate) Reported by: se (mailing lists), others Approved by: ports-secteam (fluffy) MFH: 2020Q1 Changes: head/emulators/virtualbox-ose/files/patch-src_VBox_Runtime_r0drv_freebsd_memobj-r0drv-freebsd.c A commit references this bug: Author: madpilot Date: Tue Jan 26 14:51:10 UTC 2021 New revision: 562684 URL: https://svnweb.freebsd.org/changeset/ports/562684 Log: MFH: r562683 Fix VirtualBox build on recent head and 13 after src commit 0659df6faddf PR: 252675 Submitted by: O. Hartmann <ohartmann@walstatt.org> Mateusz Piotrowski <0mp@FreeBSD.org> (duplicate) Reported by: se (mailing lists), others Approved by: ports-secteam (fluffy) Changes: _U branches/2021Q1/ branches/2021Q1/emulators/virtualbox-ose/files/patch-src_VBox_Runtime_r0drv_freebsd_memobj-r0drv-freebsd.c Fix committed and merged to quarterly. *** Bug 253029 has been marked as a duplicate of this bug. *** |