Re: Cpu pools discussion

[Juergen Gross <juergen.gross@ts.fujitsu.com>]

Keir Fraser wrote:
> On 30/07/2009 06:46, "Juergen Gross" <juergen.gross@ts.fujitsu.com> wrote:
> 
>>> Another alternative might be to create a 'hypervisor thread', either
>>> dynamically, or a per-cpu worker thread, and do the work in that. Of course
>>> that has its own complexities and these threads would also have their own
>>> interactions with cpu pools to keep them pinned on the appropriate physical
>>> cpu. I don't know whether this would really work out simpler.
>> There should be an easy solution for this: What you are suggesting here sounds
>> like a "hypervisor domain" similar to the the idle domain, but with high
>> priority and normally all vcpus blocked.
>>
>> The interactions of this domain with cpupools would be the same as for the
>> idle domain.
>>
>> I think this approach could be attractive, but the question is if the pros
>> outweigh the cons. OTOH such a domain could open interesting opportunities.
> 
> I think especially if cpupools are added into the mix then this becomes more
> attractive than the current approach. The other alternative is to modify the
> two existing problematic callers to work okay from softirq context (or not
> need continue_hypercall_on_cpu() at all, which might be possible at least in
> the case of CPU hotplug). I would be undecided between these two just now --
> it depends on how easily those two callers can be fixed up.

I'll try to set up a patch to add a hypervisor domain. Regarding all the
problems I got with switching cpus between pools (avoid running on the cpu to
be switched etc.) this solution could make life much easier.

And George would be happy to see all the borrow cpu stuff vanish :-)


Juergen

-- 
Juergen Gross                 Principal Developer Operating Systems
TSP ES&S SWE OS6                       Telephone: +49 (0) 89 636 47950
Fujitsu Technolgy Solutions               e-mail: juergen.gross@ts.fujitsu.com
Otto-Hahn-Ring 6                        Internet: ts.fujitsu.com
D-81739 Muenchen                 Company details: ts.fujitsu.com/imprint.html