[Devel] [PATCH 0/6] backporting async_pf injection functionality

[Denis V. Lunev]

On 09/25/2017 04:00 PM, Kirill Tkhai wrote:
>
> On 25.09.2017 15:57, Denis V. Lunev wrote:
>> On 09/25/2017 03:47 PM, Kirill Tkhai wrote:
>>> Hi, Denis,
>>>
>>> your patches in base64 format. Could you resend them as plain-text like we
>>> usually do?
>>>
>>> Here is body of the patch 1/6:
>>>
>>> "X-MS-Exchange-Processed-By-BccFoldering: 15.20.0056.000
>>> X-Microsoft-Exchange-Diagnostics:
>>> 	1;HE1PR0801MB1338;27:2tKlmOLYPsUB4+tCQ+/VLLnyTS38TwWCV2ZWj5N2k5yUKz/VyQQs2QuWlAhcnCHJG3wiwYtk59+WvZbGgAc5/5I8Nb/S5hdsFssBCa4bX6OWopDBrSyQq5/K2D/CsTmJ
>>> MIME-Version: 1.0
>>>
>>> RnJvbTogV2FucGVuZyBMaSA8d2FucGVuZy5saUBob3RtYWlsLmNvbT4KCiBXQVJOSU5HOiBDUFU6
>>> IDMgUElEOiAyODQwIGF0IGFyY2gveDg2L2t2bS92bXguYzoxMDk2NiBuZXN0ZWRfdm14X3ZtZXhp
>>> dCsweGRjZC8weGRlMCBba3ZtX2ludGVsXQogQ1BVOiAzIFBJRDogMjg0MCBDb21tOiBxZW11LXN5
>>> c3RlbS14ODYgVGFpbnRlZDogRyAgICAgICAgICAgT0UgICA0LjEyLjAtcmMzKyAjMjMKIFJJUDog
>>> MDAxMDpuZXN0ZWRfdm14X3ZtZXhpdCsweGRjZC8weGRlMCBba3ZtX2ludGVsXQogQ2FsbCBUcmFj
>>> ZToKICA/IGt2bV9jaGVja19hc3luY19wZl9jb21wbGV0aW9uKzB4ZWYvMHgxMjAgW2t2bV0KICA/
>>> IHJjdV9yZWFkX2xvY2tfc2NoZWRfaGVsZCsweDc5LzB4ODAKICB2bXhfcXVldWVfZXhjZXB0aW9u
>>> KzB4MTA0LzB4MTYwIFtrdm1faW50ZWxdCiAgPyB2bXhfcXVldWVfZXhjZXB0aW9uKzB4MTA0LzB4
>>> MTYwIFtrdm1faW50ZWxdCiAga3ZtX2FyY2hfdmNwdV9pb2N0bF9ydW4rMHgxMTcxLzB4MWNlMCBb
>>> a3ZtXQogID8ga3ZtX2FyY2hfdmNwdV9sb2FkKzB4NDcvMHgyNDAgW2t2bV0KICA/IGt2bV9hcmNo
>>> X3ZjcHVfbG9hZCsweDYyLzB4MjQwIFtrdm1dCiAga3ZtX3ZjcHVfaW9jdGwrMHgzODQvMHg3YjAg
>>> W2t2bV0KICA/IGt2bV92Y3B1X2lvY3RsKzB4Mzg0LzB4N2IwIFtrdm1dCiAgPyBfX2ZnZXQrMHhm
>>> My8weDIxMAogIGRvX3Zmc19pb2N0bCsweGE0LzB4NzAwCiAgPyBfX2ZnZXQrMHgxMTQvMHgyMTAK
>>> ICBTeVNfaW9jdGwrMHg3OS8weDkwCiAgZG9fc3lzY2FsbF82NCsweDgxLzB4MjIwCiAgZW50cnlf
>>> U1lTQ0FMTDY0X3Nsb3dfcGF0aCsweDI1LzB4MjUKClRoaXMgaXMgdHJpZ2dlcmVkIG9jY2FzaW9u
>>> YWxseSBieSBydW5uaW5nIGJvdGggd2luNyBhbmQgd2luMjAxNiBpbiBMMiwgaW4KYWRkaXRpb24s
>>> IEVQVCBpcyBkaXNhYmxlZCBvbiBib3RoIEwxIGFuZCBMMi4gSXQgY2FuJ3QgYmUgcmVwcm9kdWNl
>>> ZCBlYXNpbHkuCgpDb21taXQgMGI2YWMzNDNmYyAoS1ZNOiBuVk1YOiBDb3JyZWN0IGhhbmRsaW5n
>>> IG9mIGV4Y2VwdGlvbiBpbmplY3Rpb24pIG1lbnRpb25lZAp0aGF0ICJLVk0gd2FudHMgdG8gaW5q
>>> ZWN0IHBhZ2UtZmF1bHRzIHdoaWNoIGl0IGdvdCB0byB0aGUgZ3Vlc3QuIFRoaXMgZnVuY3Rpb24K
>>> YXNzdW1lcyBpdCBpcyBjYWxsZWQgd2l0aCB0aGUgZXhpdCByZWFzb24gaW4gdm1jczAyIGJlaW5n
>>> IGEgI1BGIGV4Y2VwdGlvbiIuCkNvbW1pdCBlMDExYzY2MyAoS1ZNOiBuVk1YOiBDaGVjayBhbGwg
>>> ZXhjZXB0aW9ucyBmb3IgaW50ZXJjZXB0IGR1cmluZyBkZWxpdmVyeSB0bwpMMikgYWxsb3dzIHRv
>>> IGNoZWNrIGFsbCBleGNlcHRpb25zIGZvciBpbnRlcmNlcHQgZHVyaW5nIGRlbGl2ZXJ5IHRvIEwy
>>> LiBIb3dldmVyLAp0aGVyZSBpcyBubyBndWFyYW50ZWUgdGhlIGV4aXQgcmVhc29uIGlzIGV4Y2Vw
>>> dGlvbiBjdXJyZW50bHksIHdoZW4gdGhlcmUgaXMgYW4KZXh0ZXJuYWwgaW50ZXJydXB0IG9jY3Vy
>>> cmVkIG9uIGhvc3QsIG1heWJlIGEgdGltZSBpbnRlcnJ1cHQgZm9yIGhvc3Qgd2hpY2ggc2hvdWxk
>>> Cm5vdCBiZSBpbmplY3RlZCB0byBndWVzdCwgYW5kIHNvbWV3aGVyZSBxdWV1ZXMgYW4gZXhjZXB0
>>> aW9uLCB0aGVuIHRoZSBmdW5jdGlvbgpuZXN0ZWRfdm14X2NoZWNrX2V4Y2VwdGlvbigpIHdpbGwg
>>> YmUgY2FsbGVkIGFuZCB0aGUgdm1leGl0IGVtdWxhdGlvbiBjb2RlcyB3aWxsCnRyeSB0byBlbXVs
>>> YXRlIHRoZSAiQWNrbm93bGVkZ2UgaW50ZXJydXB0IG9uIGV4aXQiIGJlaGF2aW9yLCB0aGUgd2Fy
>>> bmluZyBpcwp0cmlnZ2VyZWQuCgpSZXVzaW5nIHRoZSBleGl0IHJlYXNvbiBmcm9tIHRoZSBMMi0+
>>> TDAgdm1leGl0IGlzIHdyb25nIGluIHRoaXMgY2FzZSwKdGhlIHJlYXNvbiBtdXN0IGFsd2F5cyBi
>>> ZSBFWENFUFRJT05fTk1JIHdoZW4gaW5qZWN0aW5nIGFuIGV4Y2VwdGlvbiBpbnRvCkwxIGFzIGEg
>>> bmVzdGVkIHZtZXhpdC4KCkNjOiBQYW9sbyBCb256aW5pIDxwYm9uemluaUByZWRoYXQuY29tPgpD
>>> YzogUmFkaW0gS3LEjW3DocWZIDxya3JjbWFyQHJlZGhhdC5jb20+ClNpZ25lZC1vZmYtYnk6IFdh
>>> bnBlbmcgTGkgPHdhbnBlbmcubGlAaG90bWFpbC5jb20+CkZpeGVzOiBlMDExYzY2M2I5YzcgKCJL
>>> Vk06IG5WTVg6IENoZWNrIGFsbCBleGNlcHRpb25zIGZvciBpbnRlcmNlcHQgZHVyaW5nIGRlbGl2
>>> ZXJ5IHRvIEwyIikKU2lnbmVkLW9mZi1ieTogUmFkaW0gS3LEjW3DocWZIDxya3JjbWFyQHJlZGhh
>>> dC5jb20+CihjaGVycnkgcGlja2VkIGZyb20gY29tbWl0IGQ0OTEyMjE1ZDEwMzFlNGZiM2QxMDM4
>>> ZDJlMTg1NzIxOGRiYTBkMGEpCmZpeCAjUFNCTS01NjQ5OApTaWduZWQtb2ZmLWJ5OiBEZW5pcyBQ
>>> bG90bmlrb3YgPGRwbG90bmlrb3ZAdmlydHVvenpvLmNvbT4KLS0tCiBhcmNoL3g4Ni9rdm0vdm14
>>> LmMgfCAyICstCiAxIGZpbGUgY2hhbmdlZCwgMSBpbnNlcnRpb24oKyksIDEgZGVsZXRpb24oLSkK
>>> CmRpZmYgLS1naXQgYS9hcmNoL3g4Ni9rdm0vdm14LmMgYi9hcmNoL3g4Ni9rdm0vdm14LmMKaW5k
>>> ZXggMDliMTg1MS4uYjUyYmExOCAxMDA2NDQKLS0tIGEvYXJjaC94ODYva3ZtL3ZteC5jCisrKyBi
>>> L2FyY2gveDg2L2t2bS92bXguYwpAQCAtMjI4OCw3ICsyMjg4LDcgQEAgc3RhdGljIGludCBuZXN0
>>> ZWRfdm14X2NoZWNrX2V4Y2VwdGlvbihzdHJ1Y3Qga3ZtX3ZjcHUgKnZjcHUsIHVuc2lnbmVkIG5y
>>> KQogCWlmICghKHZtY3MxMi0+ZXhjZXB0aW9uX2JpdG1hcCAmICgxdSA8PCBucikpKQogCQlyZXR1
>>> cm4gMDsKIAotCW5lc3RlZF92bXhfdm1leGl0KHZjcHUsIHRvX3ZteCh2Y3B1KS0+ZXhpdF9yZWFz
>>> b24sCisJbmVzdGVkX3ZteF92bWV4aXQodmNwdSwgRVhJVF9SRUFTT05fRVhDRVBUSU9OX05NSSwK
>>> IAkJCSAgdm1jc19yZWFkMzIoVk1fRVhJVF9JTlRSX0lORk8pLAogCQkJICB2bWNzX3JlYWRsKEVY
>>> SVRfUVVBTElGSUNBVElPTikpOwogCXJldHVybiAxOwotLSAKMi43LjQKCl9fX19fX19fX19fX19f
>>> X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkRldmVsIG1haWxpbmcgbGlzdApEZXZl
>>> bEBvcGVudnoub3JnCmh0dHBzOi8vbGlzdHMub3BlbnZ6Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL2Rl
>>> dmVsCg=="
>>>
>>>
>>> On 20.09.2017 17:30, Denis Plotnikov wrote:
>>>> the patch set is for
>>>> 1. Replace Roman's patch avoiding async_pf injection while in the guest mode with
>>>>    the similar patch from the mainstream for kernel code consistency
>>>> 2. Force a nested vmexit if the injected #PF is async_pf
>>>> 3. Let guest support delivery of async_pf from guest mode
>>>>
>>>> Denis Plotnikov (1):
>>>>   Revert "kvm/x86: skip async_pf when in guest mode"
>>>>
>>>> Wanpeng Li (5):
>>>>   KVM: nVMX: Fix exception injection
>>>>   KVM: async_pf: Add L1 guest async_pf #PF vmexit handler
>>>>   KVM: async_pf: Force a nested vmexit if the injected #PF is async_pf
>>>>   KVM: async_pf: avoid async pf injection when in guest mode
>>>>   KVM: async_pf: Let guest support delivery of async_pf from guest mode
>>>>
>>>>  Documentation/virtual/kvm/msr.txt    |  5 ++--
>>>>  arch/x86/include/asm/kvm_emulate.h   |  1 +
>>>>  arch/x86/include/asm/kvm_host.h      |  4 +++
>>>>  arch/x86/include/uapi/asm/kvm_para.h |  1 +
>>>>  arch/x86/kernel/kvm.c                |  7 ++++-
>>>>  arch/x86/kvm/mmu.c                   | 40 +++++++++++++++++++++++++++--
>>>>  arch/x86/kvm/mmu.h                   |  4 +++
>>>>  arch/x86/kvm/svm.c                   | 50 +++++++++++-------------------------
>>>>  arch/x86/kvm/vmx.c                   | 36 +++++++++++++++++---------
>>>>  arch/x86/kvm/x86.c                   | 20 ++++++++++-----
>>>>  10 files changed, 109 insertions(+), 59 deletions(-)
>> this is somehow broken on your side.
>> I have received these thread in plain text, as usual.
> Have you seen message sources?
yes! exactly!