Date: Thu, 27 Jan 2011 00:19:57 +0100 From: Bernhard Froehlich <decke@FreeBSD.org> To: Mikolaj Golub <to.my.trociny@gmail.com> Cc: freebsd-emulation@freebsd.org, Julian Elischer <julian@freebsd.org>, freebsd-virtualization@freebsd.org Subject: Re: VirtualBox + VIMAGE Message-ID: <c51ff723c5ffb7109b7be6e3cf8f7014@bluelife.at> In-Reply-To: <8662tbkahz.fsf@zhuzha.ua1> References: <86oc77heqk.fsf@kopusha.home.net> <4D3D2DE9.2060004@freebsd.org> <86tygwy9p0.fsf@kopusha.home.net> <4D3F8357.2070005@freebsd.org> <62fcd16074e526547c3417a8f259174e@bluelife.at> <8662tbkahz.fsf@zhuzha.ua1>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 26 Jan 2011 16:25:28 +0200, Mikolaj Golub wrote: > On Wed, 26 Jan 2011 10:22:40 +0100 Bernhard Froehlich wrote: > > BF> Sounds like it's my turn now. Which FreeBSD version is required to be > BF> able to use it? > > As Bjoern noted it is for __FreeBSD_version >= 800500. > > BF> Is VIMAGE enabled per default and what happens if VIMAGE is disabled - > BF> does it at least build fine with that patch? > > We have VIMAGE disabled by default. > > I have added to src/VBox/HostDrivers/VBoxNetFlt/freebsd/Makefile: > > .if defined(VIMAGE) > CFLAGS += -DVIMAGE > .endif > > So to build the driver for VIMAGE enabled kernel one should run > > VIMAGE=1 make > > If VIMAGE variable is not defined the module for VIMAGE disabled kernel will > be built. http://home.bluelife.at/patches/virtualbox-ose-kmod-devel-VIMAGE.diff I've integrated it a bit better into the VirtualBox build system, added the ports stuff and updated the patch for VirtualBox 4.0.2. It is currently unclear to me why you add VIMAGE to CFLAGS but nowhere check for VIMAGE in VBoxNetFlt-freebsd.c. Shouldn't we add a check for VIMAGE in the #if defined line or is this already done somewhere deep in the included headers? -- Bernhard Froehlich http://www.bluelife.at/
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?c51ff723c5ffb7109b7be6e3cf8f7014>