xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCH for 4.7] doc: document that Domain-0 can't be migrated across cpupools
@ 2016-05-09  8:29 Juergen Gross
  2016-05-09  8:36 ` Andrew Cooper
  0 siblings, 1 reply; 4+ messages in thread
From: Juergen Gross @ 2016-05-09  8:29 UTC (permalink / raw)
  To: xen-devel
  Cc: Juergen Gross, sstabellini, wei.liu2, George.Dunlap,
	andrew.cooper3, ian.jackson, tim, jbeulich

Domain-0 is always member of Pool-0 (or, to be precise: of the cpuppol
with cpupool-id 0). Document this in the xl man page.

Signed-off-by: Juergen Gross <jgross@suse.com>
---
 docs/man/xl.pod.1 | 1 +
 1 file changed, 1 insertion(+)

diff --git a/docs/man/xl.pod.1 b/docs/man/xl.pod.1
index e2bd32d..9887f1b 100644
--- a/docs/man/xl.pod.1
+++ b/docs/man/xl.pod.1
@@ -1263,6 +1263,7 @@ exact same syntax as in B<cpupool-cpu-add> above.
 =item B<cpupool-migrate> I<domain> I<cpu-pool>
 
 Moves a domain specified by domain-id or domain-name into a cpu-pool.
+Domain-0 can't be moved to another cpu-pool.
 
 =item B<cpupool-numa-split>
 
-- 
2.6.6


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH for 4.7] doc: document that Domain-0 can't be migrated across cpupools
  2016-05-09  8:29 [PATCH for 4.7] doc: document that Domain-0 can't be migrated across cpupools Juergen Gross
@ 2016-05-09  8:36 ` Andrew Cooper
  2016-05-09  8:53   ` Juergen Gross
  0 siblings, 1 reply; 4+ messages in thread
From: Andrew Cooper @ 2016-05-09  8:36 UTC (permalink / raw)
  To: Juergen Gross, xen-devel
  Cc: sstabellini, wei.liu2, George.Dunlap, tim, ian.jackson, jbeulich

On 09/05/2016 09:29, Juergen Gross wrote:
> Domain-0 is always member of Pool-0 (or, to be precise: of the cpuppol
> with cpupool-id 0). Document this in the xl man page.
>
> Signed-off-by: Juergen Gross <jgross@suse.com>

Out of interest, why?

Is this a limitation of the current implementation, or something which
couldn't be made to work?

Given that it is all just vcpus and linked lists in the end, I can't see
why this should necessarily be the case.

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH for 4.7] doc: document that Domain-0 can't be migrated across cpupools
  2016-05-09  8:36 ` Andrew Cooper
@ 2016-05-09  8:53   ` Juergen Gross
  2016-05-09 11:06     ` Wei Liu
  0 siblings, 1 reply; 4+ messages in thread
From: Juergen Gross @ 2016-05-09  8:53 UTC (permalink / raw)
  To: Andrew Cooper, xen-devel
  Cc: sstabellini, wei.liu2, George.Dunlap, ian.jackson, tim, jbeulich

On 09/05/16 10:36, Andrew Cooper wrote:
> On 09/05/2016 09:29, Juergen Gross wrote:
>> Domain-0 is always member of Pool-0 (or, to be precise: of the cpuppol
>> with cpupool-id 0). Document this in the xl man page.
>>
>> Signed-off-by: Juergen Gross <jgross@suse.com>
> 
> Out of interest, why?
> 
> Is this a limitation of the current implementation, or something which
> couldn't be made to work?
> 
> Given that it is all just vcpus and linked lists in the end, I can't see
> why this should necessarily be the case.

Main reason was to keep cpu hotplugging simple. New cpus are always
added to Pool-0 and should be available to dom0 immediately without
having to move them to another cpupool.

It could be changed, of course. But I don't see any advantage in
allowing to move dom0 to another cpupool, while the code would be
more complicated.


Juergen


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH for 4.7] doc: document that Domain-0 can't be migrated across cpupools
  2016-05-09  8:53   ` Juergen Gross
@ 2016-05-09 11:06     ` Wei Liu
  0 siblings, 0 replies; 4+ messages in thread
From: Wei Liu @ 2016-05-09 11:06 UTC (permalink / raw)
  To: Juergen Gross
  Cc: sstabellini, wei.liu2, George.Dunlap, Andrew Cooper, tim,
	xen-devel, jbeulich, ian.jackson

On Mon, May 09, 2016 at 10:53:30AM +0200, Juergen Gross wrote:
> On 09/05/16 10:36, Andrew Cooper wrote:
> > On 09/05/2016 09:29, Juergen Gross wrote:
> >> Domain-0 is always member of Pool-0 (or, to be precise: of the cpuppol
> >> with cpupool-id 0). Document this in the xl man page.
> >>
> >> Signed-off-by: Juergen Gross <jgross@suse.com>
> > 
> > Out of interest, why?
> > 
> > Is this a limitation of the current implementation, or something which
> > couldn't be made to work?
> > 
> > Given that it is all just vcpus and linked lists in the end, I can't see
> > why this should necessarily be the case.
> 
> Main reason was to keep cpu hotplugging simple. New cpus are always
> added to Pool-0 and should be available to dom0 immediately without
> having to move them to another cpupool.
> 
> It could be changed, of course. But I don't see any advantage in
> allowing to move dom0 to another cpupool, while the code would be
> more complicated.
> 
> 

I think this patch is valid in that it documents the current situation.
Whether migration of Dom0 should be supported or not is a separate
topic.

Acked-by: Wei Liu <wei.liu2@citrix.com>
Release-acked-by: Wei Liu <wei.liu2@citrix.com>

I've queued this up for committing.

Wei.

> Juergen
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-05-09 11:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-09  8:29 [PATCH for 4.7] doc: document that Domain-0 can't be migrated across cpupools Juergen Gross
2016-05-09  8:36 ` Andrew Cooper
2016-05-09  8:53   ` Juergen Gross
2016-05-09 11:06     ` Wei Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).