WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-api

[Xen-API] [PATCH 1 of 4] CA-33707: Stop using the internal RPC call VM.h

To: xen-api@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-API] [PATCH 1 of 4] CA-33707: Stop using the internal RPC call VM.hard_reboot_internal in the event thread VM reboot path. This used to be necessary when memory settings could change across reboot but ever since the integration of DMC this is nolonger possible. Removing this simplifies the start/reboot locking
From: David Scott <dave.scott@xxxxxxxxxxxxx>
Date: Thu, 10 Dec 2009 23:04:55 +0000
Delivery-date: Thu, 10 Dec 2009 15:08:02 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <patchbomb.1260486294@xxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-api-request@lists.xensource.com?subject=help>
List-id: Discussion of API issues surrounding Xen <xen-api.lists.xensource.com>
List-post: <mailto:xen-api@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-api>, <mailto:xen-api-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-api>, <mailto:xen-api-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-api-bounces@xxxxxxxxxxxxxxxxxxx
# HG changeset patch
# User David Scott <dave.scott@xxxxxxxxxxxxx>
# Date 1260486287 0
# Node ID 43f67984ff0f3737bc3c041325446ec5fcd27a6e
# Parent  1f49f00797faf725de371513be573e0ca0badb26
CA-33707: Stop using the internal RPC call VM.hard_reboot_internal in the event 
thread VM reboot path. This used to be necessary when memory settings could 
change across reboot but ever since the integration of DMC this is nolonger 
possible. Removing this simplifies the start/reboot locking.

Note we leave the forwarding code in place to cope with rolling upgrade to the 
next release.

Signed-off-by: David Scott <dave.scott@xxxxxxxxxxxxx>

diff -r 1f49f00797fa -r 43f67984ff0f ocaml/idl/datamodel.ml
--- a/ocaml/idl/datamodel.ml    Fri Dec 04 16:04:36 2009 +0000
+++ b/ocaml/idl/datamodel.ml    Thu Dec 10 23:04:47 2009 +0000
@@ -1590,6 +1590,7 @@
   ~params:[Ref _vm, "vm", "The VM to reboot"]
   ~pool_internal:true
   ~hide_from_docs:true
+  ~internal_deprecated_since:rel_midnight_ride
   ~allowed_roles:_R_LOCAL_ROOT_ONLY
   ()
   
diff -r 1f49f00797fa -r 43f67984ff0f ocaml/xapi/events.ml
--- a/ocaml/xapi/events.ml      Fri Dec 04 16:04:36 2009 +0000
+++ b/ocaml/xapi/events.ml      Thu Dec 10 23:04:47 2009 +0000
@@ -143,7 +143,8 @@
     else clear_reboot_delay ~__context ~vm;
 
     try
-      Helpers.call_api_functions ~__context (fun rpc session_id -> 
Client.Client.VM.hard_reboot_internal rpc session_id vm)
+         Xapi_vm.Reboot.in_dom0_already_locked { Xapi_vm.TwoPhase.__context = 
__context; vm=vm; api_call_name="reboot"; clean=false };
+         update_allowed_ops_using_api ~__context vm
     with e ->
       (* NB this can happen if the user has change the VM configuration to onw 
which
         cannot boot (eg not enough memory) and then rebooted inside the guest 
*)
2 files changed, 3 insertions(+), 1 deletion(-)
ocaml/idl/datamodel.ml |    1 +
ocaml/xapi/events.ml   |    3 ++-


Attachment: xen-api.hg-4.patch
Description: Text Data

_______________________________________________
xen-api mailing list
xen-api@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/mailman/listinfo/xen-api
<Prev in Thread] Current Thread [Next in Thread>