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 02 of 12] [PCR0047] Renames function "VM_memory_constra

To: xen-api@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-API] [PATCH 02 of 12] [PCR0047] Renames function "VM_memory_constraints.valid" to "VM_memory_constraints.are_valid" in anticipation of using a common prefix for additional functions to be added in a further patch
From: Jonathan Knowles <jonathan.knowles@xxxxxxxxxxxxx>
Date: Mon, 1 Feb 2010 16:32:04 +0000
Delivery-date: Mon, 01 Feb 2010 08:36:13 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <patchbomb.1265041922@radon>
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 Jonathan Knowles <jonathan.knowles@xxxxxxxxxxxxx>
# Date 1265039941 0
# Node ID 3aaa09cfd961b30c2d3af22e3b057d4c22203d9c
# Parent  03168d007c006a76260daf2b7bd31a6f22b8209d
[PCR0047] Renames function "VM_memory_constraints.valid" to 
"VM_memory_constraints.are_valid" in anticipation of using a common prefix for 
additional functions to be added in a further patch.

Signed-off-by: Jonathan Knowles <jonathan.knowles@xxxxxxxxxxxxx>

diff -r 03168d007c00 -r 3aaa09cfd961 ocaml/util/vm_memory_constraints.ml
--- a/ocaml/util/vm_memory_constraints.ml       Mon Feb 01 15:59:00 2010 +0000
+++ b/ocaml/util/vm_memory_constraints.ml       Mon Feb 01 15:59:01 2010 +0000
@@ -56,7 +56,7 @@
        (** Returns true if and only if the given memory constraints are in 
valid
          * order such that: static_min <= dynamic_min <= dynamic_max <= 
static_max
          *)
-       val valid : constraints:t -> bool
+       val are_valid : constraints:t -> bool
 
        (** Takes the given set of possibly-invalid memory constraints {i s}, 
and
          * returns a new set of valid and unballooned constraints {i t} s.t.:
@@ -114,7 +114,7 @@
                        dynamic_max = dynamic_max;
                }
 
-       let valid ~constraints =
+       let are_valid ~constraints =
                Listext.List.is_sorted compare [
                        constraints.static_min;
                        constraints.dynamic_min;
diff -r 03168d007c00 -r 3aaa09cfd961 ocaml/xapi/xapi_vm.ml
--- a/ocaml/xapi/xapi_vm.ml     Mon Feb 01 15:59:00 2010 +0000
+++ b/ocaml/xapi/xapi_vm.ml     Mon Feb 01 15:59:01 2010 +0000
@@ -139,7 +139,7 @@
                static_min = min;
                static_max = max;
        } in
-       if not (Vm_memory_constraints.valid ~constraints)
+       if not (Vm_memory_constraints.are_valid ~constraints)
        then raise (
                Api_errors.Server_error (
                        Api_errors.memory_constraint_violation, ["min or 
max"]));
@@ -172,7 +172,7 @@
                dynamic_max = dynamic_max;
                static_max  = static_max;
        } in
-       if not (Vm_memory_constraints.valid ~constraints)
+       if not (Vm_memory_constraints.are_valid ~constraints)
        then raise (Api_errors.Server_error (
                Api_errors.memory_constraint_violation,
                ["Memory limits must be in valid order: \
diff -r 03168d007c00 -r 3aaa09cfd961 ocaml/xapi/xapi_vm_helpers.ml
--- a/ocaml/xapi/xapi_vm_helpers.ml     Mon Feb 01 15:59:00 2010 +0000
+++ b/ocaml/xapi/xapi_vm_helpers.ml     Mon Feb 01 15:59:01 2010 +0000
@@ -578,7 +578,7 @@
                dynamic_min = min;
                target = min;
                dynamic_max = max } in
-       if not (Vm_memory_constraints.valid ~constraints)
+       if not (Vm_memory_constraints.are_valid ~constraints)
        then raise 
(Api_errors.Server_error(Api_errors.memory_constraint_violation,
                ["min or max"]));
 
@@ -611,7 +611,7 @@
        (* Make sure the new target is within the acceptable range. *)
        let constraints = Vm_memory_constraints.get_live ~__context 
~vm_ref:self in
        let constraints = {constraints with Vm_memory_constraints.target = 
target} in
-       if not (Vm_memory_constraints.valid ~constraints) then raise (
+       if not (Vm_memory_constraints.are_valid ~constraints) then raise (
                Api_errors.Server_error (
                Api_errors.memory_constraint_violation, ["invalid target"]));
        (* We've been forwarded, so the VM is local to this machine. *)
3 files changed, 6 insertions(+), 6 deletions(-)
ocaml/util/vm_memory_constraints.ml |    4 ++--
ocaml/xapi/xapi_vm.ml               |    4 ++--
ocaml/xapi/xapi_vm_helpers.ml       |    4 ++--


Attachment: xen-api.hg-12.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>