Re: Add the hook kvm_arch_set_memory region to hold mmu-specific changes

[Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>]

Zhang, Xiantao wrote:
> Avi Kivity wrote:
>   
>> Carsten Otte wrote:
>>     
>>> Zhang, Xiantao wrote:
>>>       
>>>> User-allocation should be what we are heading. But considering
>>>> compatibility with old user-space support, I think kernel-allocation
>>>> approach should exist for a long time.
>>>>         
>>> That's right. This is why I would prefer to have the corresponding
>>> code out of kvm_main.c: it may exist for a long time for x86.
>>>
>>>       
>>>> I think we don't need to consider
>>>> this case now. Once the kernel-allocation approach is abandoned in
>>>> future, as you say, we can move them all into x86.
>>>>         
>>> I'd rather prefer to move it upfront. Otherwise, I'd have to consider
>>> that case for s390 as long as it remains in common. At least I'd have
>>> to make sure it does'nt get used on s390 using an if() or #ifdef.
>>>       
>> I agree, other archs shouldn't have to suffer.
>>     
>
> So,  now we move the whole thing(__kvm_set_memory_region) into arch ? :)
> Xiantao
>   

It's fairly generic.  Only the if (!user_alloc) parts need to move.


-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/