[Autotest PATCH] KVM test: Fixup memory size shown in 'GB' in get_memory_size

[Autotest PATCH] KVM test: Fixup memory size shown in 'GB' in get_memory_size

[Yolkfull Chow]

In guest RHEL-3.9 $mem_chk_cmd will catch memory size in GB which
will be computed wrongly in get_memory_size. This patch fix the problem.

Thanks akong for pointing this out.

Signed-off-by: Yolkfull Chow <yzhou@redhat.com>
---
 client/tests/kvm/kvm_vm.py |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py
index cc314d4..7229b79 100755
--- a/client/tests/kvm/kvm_vm.py
+++ b/client/tests/kvm/kvm_vm.py
@@ -854,7 +854,11 @@ class VM:
                 mem_size = 0
                 for m in mem:
                     mem_size += int(m)
-                if not "MB" in mem_str:
+                if "GB" in mem_str:
+                    mem_size *= 1024
+                elif "MB" in mem_str:
+                    pass
+                else:
                     mem_size /= 1024
                 return int(mem_size)
             return None
-- 
1.6.5.5

Re: [Autotest PATCH] KVM test: Fixup memory size shown in 'GB' in get_memory_size

[Jason Wang]

Applied, thanks
> In guest RHEL-3.9 $mem_chk_cmd will catch memory size in GB which
> will be computed wrongly in get_memory_size. This patch fix the problem.
>
> Thanks akong for pointing this out.
>
> Signed-off-by: Yolkfull Chow <yzhou@redhat.com>
> ---
>  client/tests/kvm/kvm_vm.py |    6 +++++-
>  1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py
> index cc314d4..7229b79 100755
> --- a/client/tests/kvm/kvm_vm.py
> +++ b/client/tests/kvm/kvm_vm.py
> @@ -854,7 +854,11 @@ class VM:
>                  mem_size = 0
>                  for m in mem:
>                      mem_size += int(m)
> -                if not "MB" in mem_str:
> +                if "GB" in mem_str:
> +                    mem_size *= 1024
> +                elif "MB" in mem_str:
> +                    pass
> +                else:
>                      mem_size /= 1024
>                  return int(mem_size)
>              return None
>   

Re: [Autotest] [Autotest PATCH] KVM test: Fixup memory size shown in 'GB' in get_memory_size

[Jason Wang]

Sorry for the noise, please ignore this mail.
> Applied, thanks
>   
>> In guest RHEL-3.9 $mem_chk_cmd will catch memory size in GB which
>> will be computed wrongly in get_memory_size. This patch fix the problem.
>>
>> Thanks akong for pointing this out.
>>
>> Signed-off-by: Yolkfull Chow <yzhou@redhat.com>
>> ---
>>  client/tests/kvm/kvm_vm.py |    6 +++++-
>>  1 files changed, 5 insertions(+), 1 deletions(-)
>>
>> diff --git a/client/tests/kvm/kvm_vm.py b/client/tests/kvm/kvm_vm.py
>> index cc314d4..7229b79 100755
>> --- a/client/tests/kvm/kvm_vm.py
>> +++ b/client/tests/kvm/kvm_vm.py
>> @@ -854,7 +854,11 @@ class VM:
>>                  mem_size = 0
>>                  for m in mem:
>>                      mem_size += int(m)
>> -                if not "MB" in mem_str:
>> +                if "GB" in mem_str:
>> +                    mem_size *= 1024
>> +                elif "MB" in mem_str:
>> +                    pass
>> +                else:
>>                      mem_size /= 1024
>>                  return int(mem_size)
>>              return None
>>   
>>     
>
> _______________________________________________
> Autotest mailing list
> Autotest@test.kernel.org
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
>