[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv2] Avoid page_set_flags() assert in qemu-user ho
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCHv2] Avoid page_set_flags() assert in qemu-user host page protection code |
Date: |
Fri, 9 Apr 2010 22:02:41 +0200 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Wed, Mar 31, 2010 at 11:00:36PM +0200, Juergen Lock wrote:
> V2 that uses endaddr = end-of-guest-address-space if !h2g_valid(endaddr)
> after I found out that indeed works; and also disables the FreeBSD 6.x
> /compat/linux/proc/self/maps fallback because it can return partial lines
> if (at least I think that's the reason) the mappings change between
> subsequent read() calls.
>
> Signed-off-by: Juergen Lock <address@hidden>
Thanks, applied.
> --- a/exec.c
> +++ b/exec.c
> @@ -306,13 +306,14 @@ static void page_init(void)
>
> if (h2g_valid(endaddr)) {
> endaddr = h2g(endaddr);
> - page_set_flags(startaddr, endaddr, PAGE_RESERVED);
> } else {
> #if TARGET_ABI_BITS <= L1_MAP_ADDR_SPACE_BITS
> endaddr = ~0ul;
> - page_set_flags(startaddr, endaddr, PAGE_RESERVED);
> +#else
> + endaddr = ((abi_ulong)1 << L1_MAP_ADDR_SPACE_BITS) -
> 1;
> #endif
> }
> + page_set_flags(startaddr, endaddr, PAGE_RESERVED);
> }
> }
> free(freep);
> @@ -323,11 +324,7 @@ static void page_init(void)
>
> last_brk = (unsigned long)sbrk(0);
>
> -#if defined(__FreeBSD__) || defined(__FreeBSD_kernel__) ||
> defined(__DragonFly__)
> - f = fopen("/compat/linux/proc/self/maps", "r");
> -#else
> f = fopen("/proc/self/maps", "r");
> -#endif
> if (f) {
> mmap_lock();
>
> @@ -343,7 +340,11 @@ static void page_init(void)
> if (h2g_valid(endaddr)) {
> endaddr = h2g(endaddr);
> } else {
> +#if TARGET_ABI_BITS <= L1_MAP_ADDR_SPACE_BITS
> endaddr = ~0ul;
> +#else
> + endaddr = ((abi_ulong)1 << L1_MAP_ADDR_SPACE_BITS) -
> 1;
> +#endif
> }
> page_set_flags(startaddr, endaddr, PAGE_RESERVED);
> }
>
>
>
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net
- Re: [Qemu-devel] [PATCHv2] Avoid page_set_flags() assert in qemu-user host page protection code,
Aurelien Jarno <=