Re: [PATCH v4] kvm: better MWAIT emulation for guests

[Gabriel L. Somlo]

On Wed, Mar 15, 2017 at 04:21:41PM -0400, Gabriel L. Somlo wrote:
> > 
> >  - do you see VM exits on the "hung" VCPU?
> 
> how would I go about looking ?
> 
> >  - what is your CPU model?
> 
> $ cat /proc/cpuinfo
> ...
> processor       : 3
> vendor_id       : GenuineIntel
> cpu family      : 6
> model           : 15
> model name      : Intel(R) Xeon(R) CPU            5150  @ 2.66GHz
> stepping        : 6
> microcode       : 0xd2
> cpu MHz         : 2659.966
> cache size      : 4096 KB
> physical id     : 3
> siblings        : 2
> core id         : 0
> cpu cores       : 2
> apicid          : 6
> initial apicid  : 6
> fpu             : yes
> fpu_exception   : yes
> cpuid level     : 10
> wp              : yes
> flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx lm constant_tsc arch_perfmon pebs bts rep_good nopl cpuid aperfmperf pni dtes64 monitor ds_cpl vmx est tm2 ssse3 cx16 xtpr pdcm dca lahf_lm tpr_shadow dtherm
> bugs            :
> bogomips        : 5320.04
> clflush size    : 64
> cache_alignment : 64
> address sizes   : 36 bits physical, 48 bits virtual
> power management:
> 
> (this is 2x dual-core Xeon on a Mac Pro 1,1 -- all I had to spare for
> testing, to avoid having to reboot my primary desktop :)

Oh, in case you meant what I use on the qemu command line:

qemu-system-x86_64 -enable-kvm -m 2048 -cpu core2duo \
  -machine q35 -smp 4,cores=2 \
  -device isa-applesmc,osk="...osk.value.from.smc.goes.here..." \
  -usb -device usb-kbd -device usb-mouse \
  -smbios type=2 -kernel ./chameleon_boot \
  -device ide-drive,bus=ide.2,drive=MacHDD \
  -drive id=MacHDD,if=none,snapshot=on,file=./mac_10.7.img \
  -monitor stdio

so, core2duo -- it's the only thing OS X 10.7 will find acceptable.

> 
> >  - what do you get after running this C program on host and guest?
> > 
> >    #include <stdint.h>
> >    #include <stdio.h>
> >    
> >    int main(void) {
> >    	uint32_t eax = 5, ebx, ecx = 0, edx;
> >    	asm ("cpuid" : "+a"(eax), "=b"(ebx), "+c"(ecx), "=d"(edx));
> >    
> >    	printf("eax=%#08x ebx=%#08x ecx=%#08x edx=%#08x\n", eax, ebx, ecx, edx);
> >    
> >    	return 0;
> >    }
> 
> eax=0x000040 ebx=0x000040 ecx=0x000003 edx=0x000020
> 
> HTH,
> --G