[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/2] TestSupport.pm: introduce host_get_free_memory



Wei Liu writes ("[PATCH 1/2] TestSupport.pm: introduce host_get_free_memory"):
> ... which returns the size of free memory in MB.
...
> +sub host_get_free_memory($$) {
> +    my ($ho,$toolstack) = @_;
> +    # The line is as followed:
> +    # free_memory       :   XXXX
> +    my $cmd = "$toolstack info | grep free_memory | awk {'print \$3'}";
> +    my $free_memory_mb = target_cmd_root($ho, $cmd, 10);
> +    return $free_memory_mb;
> +}

Surely it would be better do to the parsing on the control host in
Perl rather than sending grep/awk runes to the target ?  One regexp
ought to do it.

Also, did you test this ?  target_cmd_root doesn't return you the
output.

Thanks,
Ian.

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.