Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 26 Jan 2011 16:25:28 +0200
From:      Mikolaj Golub <to.my.trociny@gmail.com>
To:        Bernhard Froehlich <decke@FreeBSD.org>
Cc:        freebsd-emulation@freebsd.org, Julian Elischer <julian@freebsd.org>, freebsd-virtualization@freebsd.org
Subject:   Re: VirtualBox + VIMAGE
Message-ID:  <8662tbkahz.fsf@zhuzha.ua1>
In-Reply-To: <62fcd16074e526547c3417a8f259174e@bluelife.at> (Bernhard Froehlich's message of "Wed, 26 Jan 2011 10:22:40 %2B0100")
References:  <86oc77heqk.fsf@kopusha.home.net> <4D3D2DE9.2060004@freebsd.org> <86tygwy9p0.fsf@kopusha.home.net> <4D3F8357.2070005@freebsd.org> <62fcd16074e526547c3417a8f259174e@bluelife.at>

next in thread | previous in thread | raw e-mail | index | archive | help

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.

I am not familiar with VBox build process so there might be a better solution.

I suppose we should have an option in ports, something:

.if ${OSVERSION} < 800500
OPTIONS+=        VIMAGE "Build for VIMAGE kernel" off
.endif

and if it is on set VIMAGE make environment variable. Or may be you have
a better solution.

 BF> For Mikolaj Golub to get this patch upstream we need that you either
 BF> agree that his patch is under the MIT License or sign a Sun Contributors
 BF> Agreement. Then we can submit that patch upstream or you can do it
 BF> yourself if you prefer. Read here:
 BF> http://www.virtualbox.org/wiki/Contributor_information

MIT License is ok.

It would be very nice of you if you submitted the patch upstream :-). If
nobody do this then I will try to submit it myself.

-- 
Mikolaj Golub



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?8662tbkahz.fsf>