fel: Try to detect VM environments and print a warning for those#98
fel: Try to detect VM environments and print a warning for those#98n1tehawk wants to merge 1 commit into
Conversation
Signed-off-by: Bernhard Nortmann <bernhard.nortmann@web.de>
There was a problem hiding this comment.
Acked-by: Siarhei Siamashka siarhei.siamashka@gmail.com
It would be nice if @kotc could test it in his VM.
| libusb_device **usb; | ||
| struct libusb_device_descriptor desc; | ||
|
|
||
| libusb_init(&ctx); |
There was a problem hiding this comment.
Do we need to check for error code from libusb_init() too? http://libusb.sourceforge.net/api-1.0/group__lib.html#ga9517c37281bba0b51cc62eba728be48b
apritzel
left a comment
apritzel
left a comment
There was a problem hiding this comment.
Can't you check the hypervisor CPUID bit instead? ECX[bit 31] with EAX=1? AFAIK at least Xen and KVM set it, not sure about VMware, though.
|
Tested-By: @kotc tested in virtualbox@win7_64 (debian64 gues), vm was detected and warning printed ok @apritzel: yeah, in cpuinfo: flags : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx lm constant_tsc rep_good nopl pni ssse3 sse4_1 hypervisor lahf_lm |
|
Should we merge this as-is or try to implement @apritzel's suggestion? |
|
Okay, I will push this to the master branch in a few days. |
4 participants
We try to detect virtual machines by parsing the vendor IDs of all USB devices, and will print a warning message if such an environment is encountered.