Xen devel - Mar 2007 - [PATCH] Make save/restore of multiple VCPU contexts arch dependent

We need to look at how to support this on ia64, so how about making
this an architecture dependent feature for now?  Not sure what powerpc
needs here, so I''m just lumping it into the old way bucket.  Thanks,

	Alex

Signed-off-by: Alex Williamson <alex.williamson@hp.com>
---

diff -r 3ac19fda0bc2 tools/python/xen/xend/XendDomainInfo.py
--- a/tools/python/xen/xend/XendDomainInfo.py	Fri Mar 02 12:11:52 2007 +0000
+++ b/tools/python/xen/xend/XendDomainInfo.py	Mon Mar 05 10:30:22 2007 -0700
@@ -775,13 +775,18 @@ class XendDomainInfo:
         t.write(''vm'', self.vmpath)
 
     def _storeDomDetails(self):
+        if arch.type == "x86":
+            mp_suspend = 1
+        else:
+            mp_suspend = 0
+
         to_store = {
             ''domid'':              str(self.domid),
             ''vm'':                 self.vmpath,
             ''name'':              
self.info[''name_label''],
             ''console/limit'':     
str(xoptions.get_console_limit() * 1024),
             ''memory/target'':     
str(self.info[''memory_static_min''] * 1024),
-           
''control/platform-feature-multiprocessor-suspend'': str(1)
+           
''control/platform-feature-multiprocessor-suspend'':
str(mp_suspend)
             }
 
         def f(n, v):



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

On Mon, 2007-03-05 at 10:45 -0700, Alex Williamson
wrote:
>    We need to look at how to support this on ia64, so how about making
> this an architecture dependent feature for now?  Not sure what powerpc
> needs here, so I''m just lumping it into the old way bucket. 
Thanks,
Keir,

   Did you have any objections or comments about this patch?  Thanks,

	Alex

> diff -r 3ac19fda0bc2 tools/python/xen/xend/XendDomainInfo.py
> --- a/tools/python/xen/xend/XendDomainInfo.py	Fri Mar 02 12:11:52 2007
+0000
> +++ b/tools/python/xen/xend/XendDomainInfo.py	Mon Mar 05 10:30:22 2007
-0700
> @@ -775,13 +775,18 @@ class XendDomainInfo:
>          t.write(''vm'', self.vmpath)
>  
>      def _storeDomDetails(self):
> +        if arch.type == "x86":
> +            mp_suspend = 1
> +        else:
> +            mp_suspend = 0
> +
>          to_store = {
>              ''domid'':              str(self.domid),
>              ''vm'':                 self.vmpath,
>              ''name'':              
self.info[''name_label''],
>              ''console/limit'':     
str(xoptions.get_console_limit() * 1024),
>              ''memory/target'':     
str(self.info[''memory_static_min''] * 1024),
> -           
''control/platform-feature-multiprocessor-suspend'': str(1)
> +           
''control/platform-feature-multiprocessor-suspend'':
str(mp_suspend)
>              }
>  
>          def f(n, v):
> 
-- 
Alex Williamson                             HP Open Source & Linux Org.


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

On 7/3/07 18:14, "Alex Williamson" <alex.williamson@hp.com>
wrote:
> On Mon, 2007-03-05 at 10:45 -0700, Alex Williamson wrote:
>>    We need to look at how to support this on ia64, so how about making
>> this an architecture dependent feature for now?  Not sure what powerpc
>> needs here, so I''m just lumping it into the old way bucket. 
Thanks,
> 
> Keir,
> 
>    Did you have any objections or comments about this patch?  Thanks,
I think I missed the original email somehow. Yes, this patch looks fine.
I''ll check it in.

 -- Keir



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