Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 12 Aug 2011 13:11:04 -0400
From:      Michael Butler <imb@protected-networks.net>
To:        Bernhard Froehlich <decke@FreeBSD.org>
Cc:        freebsd-emulation@FreeBSD.org, Jung-uk Kim <jkim@FreeBSD.org>
Subject:   Re: PG_UNMANAGED disappearance in -current
Message-ID:  <4E455EA8.70705@protected-networks.net>
In-Reply-To: <aa5e184058b9c50d26fdaf3b3aaf06da@bluelife.at>
References:  <4E441A9D.1060102@protected-networks.net> <201108111544.20198.jkim@FreeBSD.org> <aa5e184058b9c50d26fdaf3b3aaf06da@bluelife.at>

next in thread | previous in thread | raw e-mail | index | archive | help
On 08/12/11 02:51, Bernhard Froehlich wrote:
> 
> I have started the same way and then talked to kib. He told me that
> vm_phys_alloc_contig() already initializes the page as unmanaged so this
> stuff should be a noop in the kmod. I have not runtime tested it myself
> though so i would welcome some feedback on that.

This appears to work just fine too - thanks! :-)

	imb

> http://home.bluelife.at/patches/patch-src-VBox-Runtime-r0drv-freebsd-memobj-r0drv-freebsd.c
> 
> ---
> src/VBox/Runtime/r0drv/freebsd/memobj-r0drv-freebsd.c.orig	2011-04-05
> 14:34:09.000000000 +0200
> +++ src/VBox/Runtime/r0drv/freebsd/memobj-r0drv-freebsd.c	2011-08-11
> 15:06:46.000000000 +0200
> @@ -394,9 +394,13 @@
>      pPage->wire_count = 1;
>      pPage->pindex     = iPage;
>      pPage->act_count  = 0;
> -    pPage->oflags     = 0;
> -    pPage->flags      = PG_UNMANAGED;
>      atomic_add_int(&cnt.v_wire_count, 1);
> +
> +#if __FreeBSD_version >= 900040
> +    Assert(pPage->oflags & VPO_UNMANAGED != 0);
> +#else
> +    Assert(pPage->flags & PG_UNMANAGED != 0);
> +#endif
>  }
> 




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4E455EA8.70705>