Re: [PATCH][QEMU] vmxcap: Open MSR file in unbuffered mode

[Gleb Natapov <gleb@redhat.com>]

On Wed, Feb 13, 2013 at 12:43:10PM +0100, Jan Kiszka wrote:
> Python may otherwise decide to to read larger chunks, applying the seek
> only on the software buffer. This will return results from the wrong
> MSRs.
> 
> Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Applied, thanks.

> ---
>  scripts/kvm/vmxcap |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/scripts/kvm/vmxcap b/scripts/kvm/vmxcap
> index 0b23f77..6363e73 100755
> --- a/scripts/kvm/vmxcap
> +++ b/scripts/kvm/vmxcap
> @@ -27,9 +27,9 @@ MSR_IA32_VMX_VMFUNC = 0x491
>  class msr(object):
>      def __init__(self):
>          try:
> -            self.f = file('/dev/cpu/0/msr')
> +            self.f = open('/dev/cpu/0/msr', 'r', 0)
>          except:
> -            self.f = file('/dev/msr0')
> +            self.f = open('/dev/msr0', 'r', 0)
>      def read(self, index, default = None):
>          import struct
>          self.f.seek(index)
> -- 
> 1.7.3.4

--
			Gleb.