search

KVM-VMI / kvm

Fork of KVM with Virtual Machine Introspection patches
Other
33 stars 27 forks source link

support for handling cpuid event added #41

Closed 0xarash closed 3 years ago

Wenzel commented 3 years ago

sorry for the lack of feedback @0xarash , @mdontu, @adlazar, can you take a look ?

0xarash commented 3 years ago

sorry for the lack of feedback @0xarash , @mdontu, @adlazar, can you take a look ?

@wenzel It's OK :)

adlazar commented 3 years ago

It looks good.

0xarash commented 3 years ago

Apparently my branch had a conflict with new kvmi-v7 branch, I merged the kvmi-v7 into mine to fix it.

adlazar commented 3 years ago

@0xarash, you can have a look at https://github.com/KVM-VMI/kvm/blob/kvmi-v7/tools/testing/selftests/kvm/x86_64/kvmi_test.c and see if you could add a test for this new event. It should look liketest_event_xsetbv(). The scripts used with Travis CI run something like: make -C tools/testing/selftests/kvm TEST_GEN_PROGS_x86_64=x86_64/kvmi_test run_tests DEBUG=1 (when the KVMI kernel is active)

0xarash commented 3 years ago

@adlazar, OK first chance I would take a look.