-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Il 01/10/2014 19:14, Nadav Amit ha scritto: >>>>> + ASSERT(!is_long_mode(vcpu) || is_pae(vcpu)); > I am ok with removing the assertion. Due to the multiple changes, I > lost track what it was supposed to do. Anyhow, removing the second > part was required since there are no reserved bits in non-pae (they > are ignored - not reserved). It becomes a bit clearer if you apply De Morgan: ASSERT(!(is_long_mode(vcpu) && !is_pae(vcpu))); or almost equivalently WARN_ON(is_long_mode(vcpu) && !is_pae(vcpu)); We should change ASSERT to positive logic (MMU_WARN_ON for example). Paolo -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBAgAGBQJUNQEgAAoJEBRUblpOawnX9hwH/AghSII0PKLF1RDC9GqvKBb6 m+UYFmYxcJjidhsvcZuNg/pRuRfJYNOFoJWO13eTYUL/eSnxXEZqy1nQTneFUFjm WKFebVc5FWc8DAXpEehrHuKUn/QmOEDj8qo41Bf0kHiptzh6W1jIEjH1AaPIwta3 5MFmFN6T+BmU1pBqOgGY5OJkAQnM9WmnsjsFDRJEFW520GP1Xvws+XxRA31Q6Qol 1qLvK2kSeuCUlGDwNWTFT0w79wQwpwuXCJfII5vzRp02pVgDKtl6sNLyKKGthxtv ONPpn0Uq0mFRbxasPk8glwqaZtqRNJKG+jysSYEf3aBmMhP4hAcGoADP6C5Umj8= =7S50 -----END PGP SIGNATURE----- -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html