[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Unposted reserved_va patch
From: |
Richard Henderson |
Subject: |
[Qemu-devel] Unposted reserved_va patch |
Date: |
Wed, 02 Jun 2010 10:20:50 -0700 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100430 Fedora/3.0.4-3.fc13 Thunderbird/3.0.4 |
Re: 68a1c816868b3e35a1da698af412b29e61b1948a
In general, I like the idea (especially since I've proposed it before. ;-)
However:
+ if (have_guest_base) {
+ flags |= MAP_FIXED;
+ }
I think this is broken. If the user specifies -G n -R m they're hoping
or guessing that the range [n,n+m) is free. What they're not expecting
is for the qemu application or any of the required shared libraries to
get forcibly unmapped.
I think instead you should simply adjust the error reporting after the
mmap attempt without MAP_FIXED.
r~
- [Qemu-devel] Unposted reserved_va patch,
Richard Henderson <=