On 03/03/20 09:07, Jay Zhou wrote: > #ifdef USE_CLEAR_DIRTY_LOG > - if (!kvm_check_cap(KVM_CAP_MANUAL_DIRTY_LOG_PROTECT2)) { > - fprintf(stderr, "KVM_CLEAR_DIRTY_LOG not available, skipping tests\n"); > + dirty_log_manual_caps = > + kvm_check_cap(KVM_CAP_MANUAL_DIRTY_LOG_PROTECT2); > + if (!dirty_log_manual_caps) { > + fprintf(stderr, "KVM_CLEAR_DIRTY_LOG not available, " > + "skipping tests\n"); > + exit(KSFT_SKIP); > + } > + if (dirty_log_manual_caps != KVM_DIRTY_LOG_MANUAL_CAPS && > + dirty_log_manual_caps != KVM_DIRTY_LOG_MANUAL_PROTECT_ENABLE) { > + fprintf(stderr, "KVM_CLEAR_DIRTY_LOG not valid caps " > + "%"PRIu64", skipping tests\n", > + dirty_log_manual_caps); > exit(KSFT_SKIP); > } > #endif > dirty_log_manual_caps &= (KVM_DIRTY_LOG_MANUAL_PROTECT_ENABLE | KVM_DIRTY_LOG_INITIALLY_SET); Paolo