From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753351AbdC2RI6 (ORCPT ); Wed, 29 Mar 2017 13:08:58 -0400 Received: from mail-sn1nam01on0041.outbound.protection.outlook.com ([104.47.32.41]:35456 "EHLO NAM01-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752473AbdC2RIx (ORCPT ); Wed, 29 Mar 2017 13:08:53 -0400 Authentication-Results: davemloft.net; dkim=none (message not signed) header.d=none;davemloft.net; dmarc=none action=none header.from=amd.com; Subject: Re: [RFC PATCH v2 18/32] kvm: svm: Use the hardware provided GPA instead of page walk To: Borislav Petkov References: <148846752022.2349.13667498174822419498.stgit@brijesh-build-machine> <148846776540.2349.3123530065053870721.stgit@brijesh-build-machine> <20170329151413.l2on26mdyyskwqlu@pd.tnic> CC: , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , From: Brijesh Singh Message-ID: Date: Wed, 29 Mar 2017 12:08:38 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <20170329151413.l2on26mdyyskwqlu@pd.tnic> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [165.204.77.1] X-ClientProxiedBy: BN6PR03CA0059.namprd03.prod.outlook.com (10.173.137.21) To CY4PR1201MB0133.namprd12.prod.outlook.com (10.172.78.15) X-MS-Office365-Filtering-Correlation-Id: a2b5293d-7c12-4923-66b3-08d476c643fa X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(48565401081)(201703131423075)(201703031133081);SRVR:CY4PR1201MB0133; X-Microsoft-Exchange-Diagnostics: 1;CY4PR1201MB0133;3:T4Em2l64OiOzrbJhx1WMcnsjnm/ZM2FkN1AuJ9RKaDuDMC7rm+0+8xcl/mj0It5OCo9d5s87BdyMkZ1rB0e41jfGH29JNZvVO0AMcTK/8/gfZCBBmqY9PbGNk9mduaGRGUdDIsyka7SVWs1WwPg1/rY1TK1LmWBBzKBISoX2E1TP8STGLCD6CmtIG+4J6YQFRMGKvlJa9L4AX6WpaX7bI2ilcZ0sfveNkkxSBks4FXs2ozJeNaI3j+D0hre1AfOERXbKPqTAm2Bq0t+7zquBBrMJsiuS4Lsiu2pkm/7k9Gomde8e5tnNkn1/7c2AOh0cIS3KKIg7j/OtgozyB9LkDCvc8E2kVM2b8w3l1Jb/GUE=;25:KTAKhDLCrjx3Ef+FwQFn3wKPTuwEqogMux3u4KHa/MrIvb3aq92oLT8WHwnyarzNxeLYQ9UDZ7ggyAeWDXbxcF/w/CTHQgwewtIkliWaN8ySwk5SLZ48jws92XRYHjlM2ZsntNSAATfcVDWW7LPI4UQBZ2hU5pT5/1642DALZJaF0yX/HndR5bQCxqWpKMyj5qmH/MLV7OWfGoQih2Z0VCjD/KIkBW7q8YtO6Y/tTPQGLzQD3he+awlmaFdBWxz6Wg9Gx3OWbPC62rg8B/c2Pw3J3IfqUsgrFzgZGoSBsDzWy6yNVjErZFKm8hlOFA9DmqJOAEG13+yyKyMEf1DyF7BdjDrzVpbO+hwP5uxFgmd9M7R6UqVvCuyooCoyoxxdZMl9v3LqucG+J7x4i/LYJ2owBfTQjgk3K3x0bNTF8+91sjVlwexozPDruQrF2dH7Uh+sFOJkSiKPQbkE27/T2g== X-Microsoft-Exchange-Diagnostics: 1;CY4PR1201MB0133;31:7ZlOJvwBIuj53wkE/cjVXpoEa3ewMUzio7XEp5qXx3fr27kGCnq195KdTj5N6ktF9YwooHBkSfVbLMh2MB2gNS0fMGRSNgB3h1hAy6QKv5izLeitPtcwBvhV1Q4gOkij7DyMCLLyMuwARZPwYloY+4gkehsP6aySkhWAHqetkRT2QO5qaeCgHh+VPYvz733uA4930DN32S2DkdIPwIr4s2QwP68/IQJVidtImBVQYWF8xrxhSTJex7Th0gKprOBN;20:ZZ9o3XlYm0eyKgPngEqhzxO0ff2CXANAhySkraZ8Na9abmW5MPb+nCM2ndqo0o9QyUaMvUWWB0Ofq1nMuVpgdyKmBefnNesPngppPkj1wx5OltnpZL1kS8ru+LwIqilRjScaiOHUebLrsCoN9w8SdVqICnrEx41xjTStIWr5OraP2AA+GDETE+k3d0ILLqCwCi0nRHKde6JZpPZxn1LEPEe6Qbt+5a1t2kgd9bt8ipvqtlErLRZGXznBLNMW/e9kbW+NSvuJEiaB+bcyYlSFVLehWvRMC9lmZaqwOLUAKuR56tNhQgTLLIenKbse9Fd8iqaWSDLYLlxYWwOAH4FZhM0OPyAmMcjyFGlub1yECZYnBjN7nOTgY3dLbRb561p/tsgC3FDQLWQhAQsH725DnCqfVFYYvv8dKGvjzOcchkk//moZ8GjD556Onsr0LNC7G1ayX4jVj1aH6GA4ot4UKc+aB+SPvXofAUiFvDV80bpyagFRdrf0GhFPArxVCzB+ X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(767451399110); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040450)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6055026)(6041248)(20161123560025)(20161123558025)(201703131423075)(201702281528075)(201703061421075)(201703061406075)(20161123555025)(20161123564025)(20161123562025)(6072148);SRVR:CY4PR1201MB0133;BCL:0;PCL:0;RULEID:;SRVR:CY4PR1201MB0133; X-Microsoft-Exchange-Diagnostics: 1;CY4PR1201MB0133;4:e1Ryf6SRlNeFVCpVN9ITHYrxWfoE39JzYyjNudG7jcO272fo5G8wXF/S94OpHwmEHrJL/dkSwxWOEDAvKPgAgsI6iQ4yy4heNwRYiY3a8OuTBP4n4Sd8Q2UGdCs4dnWW+2EfhckP4RVTvUe+LIzH4A+mCUy/w6PE8knboNbeIj4b0HWGhBX/woUWw08vMmL8Cm3XJz3067URP7o70OCWABFiXXjvb1S8XJCDmyJ1wRxxoIQtgaurlCtJUOC7KDJWPe9WS08C96wcCn/Q3Mb0ev5Sze+rvccBIIC5KxSZFII/dSQTE9YOov4vCwkAxeb3ys4C44p694zx+A6wtJLfyFHpkY6/x8GAgT7LIcD4NsAHCYV6JGnfLAZTHyuBLDgRNga2Li3aKIJz57NgGNtN8It9EtTK3eOWBjoFXHS3OGZdCgkhG0lf5rqZ+8Iyg8D/7ezesAzOWwXoVTtLImk6t32jQS1LNVHWy8W8O+dxLaaDVfB2kuf5mlmLepzCfCj8OqMVoqgPfqlztMuuW/XlGZO+AOSkX8ll8iH693faj3+lKWZ6IK4bdPfUd2w3+FBo1zlSXFAWa7LPpOXCG7JXo+3y/AXKzgW/Z8wfBOifYlb4uN0HmKUBZG3AsIAM8YXtb2CTyfmhFm/HIhXtrZtNbepwGOL8HeF57BdBtf/GyACbFJsOLgPaajBYl/xrdOP7Ki9zZLvWCifQrQ3w1o4Y4dtDbcjqaZEwtAK/qg4cPPQiUs7aUm9g8oP+XY21KVuf4lbyMnB7TNRr5DIY2qhIrJ4S/wmY698bCF3qlrwUxbI= X-Forefront-PRVS: 0261CCEEDF X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(6049001)(39450400003)(39410400002)(39400400002)(39840400002)(39850400002)(39860400002)(24454002)(377454003)(54906002)(6666003)(2906002)(6916009)(2950100002)(110136004)(6246003)(66066001)(7416002)(83506001)(53936002)(38730400002)(90366009)(25786009)(5660300001)(4326008)(229853002)(23676002)(50466002)(189998001)(4001350100001)(6486002)(53546009)(77096006)(6116002)(42186005)(3846002)(305945005)(86362001)(31696002)(7736002)(7366002)(230700001)(31686004)(36756003)(54356999)(81166006)(8676002)(47776003)(76176999)(50986999)(33646002)(217873001);DIR:OUT;SFP:1101;SCL:1;SRVR:CY4PR1201MB0133;H:[10.236.136.62];FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtDWTRQUjEyMDFNQjAxMzM7MjM6MVFhM2xVMWdqZHNqdm5icW13d0JyMFFC?= =?utf-8?B?M1ZpK2tnL1lCeWxBbjRaTTcydmZaREwzN2puZUpSOVY1OEZsRGxrUDZiRG40?= =?utf-8?B?aVd2ZC9sNkNDQ0FSSTBUUmZZVWpXN0VBdW5LMDVRV1RCclpyalcvMjlNeE52?= =?utf-8?B?VVd1VTdZeTBpVHZCRlRWWWFZRFJCaW5COTRSY1Bpek9wQTlTMUVNOWMycDRV?= =?utf-8?B?eC9ONGl0NnMrSzEvd0pVcXdtZ3BPODhjbmVXMUxkalEyM0F4d1FlY0ZMMS8z?= =?utf-8?B?YXpIWDFPQTVqRmZVWjF6SFBuMS9Qa3VNbVpHSTd4dVJ3NnpnUTUzQW84Mkl6?= =?utf-8?B?SGtFYVRVVmpmcmNLUXBRVjFubnNXajRXRHJ5R1ExVUg2L0orL29sK0VVRWlH?= =?utf-8?B?VlkxcU1xem1LdnZYUHljMFlTUkQzbVdaeXNDWDhlZWdqZHRNL3lSWW11bllu?= =?utf-8?B?ZnRYL0tsM2t0YWtEZFAya0JwWFdLdTlkQktvd2h0czZUcWFHeWcwbFo0ZXVz?= =?utf-8?B?UTlmTFBHWnFqNUUwektWMVRTQ0IvOWY2U3hWWXdqTzE2dGhXQzZvZk1kYU1q?= =?utf-8?B?MEhYd3RHa0s2OWhjeWluY3pRVldvaUlKNUZYQ2w4YktvWVFDUW5WYzFONENL?= =?utf-8?B?MUVUMW5iYVFIODJuLzUvQ1ZSeUFzUzFHNkhaRUYvZHFtQUtuanJGVVN4SW9S?= =?utf-8?B?K3Y2MFhVWFE5NXJUM1VjS05jazVZcXpJQ3lYMjZ4L2tTR0NQUEo4TnJDVVND?= =?utf-8?B?Z1kzck01ZExidU1ZYlQ0MWxrTjM1WXFTcUNTYnpMMllMRmR1dkU5YktDT0hL?= =?utf-8?B?MUtsQ2l0dHVadlJXWnV4dUlobW45eEYyMWVDSGJxZnJxZlZMbXlMblFYUEFv?= =?utf-8?B?MDBibFZxdnFLTUcvSm9XT0tyeE9ERU5ERE9GdE1RZ0FkcnRDamY0T1R5Tmhy?= =?utf-8?B?Rm9rdE5LVi9SaisvNmRFUStsdnh5N0pmT0tZblRiUDUvVllyQkVpYkVkNnAx?= =?utf-8?B?clBHQmk5dktHbVZLaWR1bVN3YitBMzl4aHRyN0Z0cmFsR3RQZlMyajR0eGZ5?= =?utf-8?B?ZnUxK1dVcllOTGZMekNndjQvRzhId0RWRXpvT25YRm5RdEptRFRRMDRzeTdN?= =?utf-8?B?aHN6eUVrVitFQm9iNHFNdDdqODAyTEtzQy9HVlU3KzJFbHZVT2FhTVBUMHJV?= =?utf-8?B?Z2V3dG9uL0sxaFM5YjlCT2JTUkN2SEovRi9SYW02a2E4dXRmNVRGS1JGUzFm?= =?utf-8?B?aS9yTVdyQ2xrb3l6UUdLL20xT2FJZk9uem1wclB1SGdydTJVMkdXbE5KdWVN?= =?utf-8?B?N253K2hqdDhkQjJwWTRFTFZjOVZLL3JJdVZVRHNkVHZXS1pEVTVRSWdqb0xa?= =?utf-8?B?cUQwNFFCcENaTHh0VU53RWYrRWsvRkZnaHF2c1dvSmwwZEIyRWdpZDJiUFNh?= =?utf-8?B?QUhqSkp6STd6QzA0b2lsbFlraXZNWmpzWlNVMFQzbWdURWhzejFSVmg1VG5q?= =?utf-8?B?dUswZUoyMm1JNm9WcXZNa1lDVmRtSFMzMmo0Vy9hb29pS2U5a2dSeElUQ3Jw?= =?utf-8?B?V1NXMDduYUt3ZGttb0FIcEVYTFhlVmU5ZDdpTlplYzl0bHZrRElKY1NFRGJ6?= =?utf-8?B?Rk9lVTBBekNmcHVwQjFiTGZxWXlxazVkOEwzN2ttQWpDbHB1UnhNcWN5L2pj?= =?utf-8?B?b0xFVFIyL3lBU0N0ZStaN0hHS1VYc01aVCtqWGxVN09lL2V5Z2tkTmdGckJr?= =?utf-8?B?R3F4TVpvZy9PVk5teUdKLzJLaWxPQWgvK1U3U2JZQU9XUlRmUzhQUWtqT29C?= =?utf-8?Q?K8E57Gob1ZxgeY2?= X-Microsoft-Exchange-Diagnostics: 1;CY4PR1201MB0133;6:SHHN9HAaQkAimzdlBPSe5QDcH8aT/NIA6yQ5d5CeO7TIhlwaqnmmUcar0JMGPkgHi2+M+c0PwrGOKVZQl/dsy5iC0ehn0I+yWtLIac06jvaPm4iD13wLC1XmRh+qaKlc24HBXI6xqOsgrpn6o4ct8PknkRoX9dB8JpanVmOHcAfdZwVkna+28zur8wmkL/5vkn/IqGfv0CcaYX0cwhDtWoWSFfr1H6DfT/Qapve75eUXqvD+m/1gZma+RFJqS3M4W/7UDgRUVg0zw2GjutJQWjRf5HQn7OHzNwdLxllhfE2PK9O+i/GFiZ22JWod+2GNqWOZs+R/z/nHNkBlTz5lRMKjePoPYQ6wj44DHY47149jAjAS+MBm1F+WS3z46rOGtXKE12I2sCb3J3it9h0agtAt8OPNJ4c9QNNE9w6nBqs=;5:al8ISCMuVKMYSkRFlbtrg9885Oi/5DXDV8XaPJc74/rpyVn5QK1pvO0nBVwRqwXVhEoWkfgISdlpYC+Qb1IHMZw1iXhtBdUSrjBB3lyyI0GsYvPiPsa5TcaU4U3+FUsGH9y8RIwT9ItIgKgEP8+FCA==;24:UupCCUXqxnKpc2bcGeFojbahxibfeM7TUUthWN+GbYyqr2r30ietpy8KxUmsaVeywVQ2e1vXBe0oCOmsd7pE3uFsBF/TnFU64qf4T3dyvcU= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;CY4PR1201MB0133;7:7MyXU05+nbKYoVKDPrm5Ak0TdkwlK5qfqlliirKecRVCEJxmk4YG/g82NvuAQrImekqlS82Mtnf+pmCAlDAc9mkVaeNDF+K1/ECqIpOHQQRGJADBl9kM3RNwv3vZOTaeBomuj5pO9fdlgaNA4LHO81fcVAsv4FGfMARXITMElGAtDk1NbrpILI/E4JsV96ANbGCq8Ii+mWiQ1TNoGTAWuV3iWxyLbRRuyCBK52ep7vWIak85yKyNpGrYd8/UkOPzyCNC4l4kvMSuRtWAClrPV5aEfjX4ehzTCD5ZHlJ76ENdccfwWzYDb7aUO93vvYAcXHaxS0sVIiQP+SwSz9LY5w==;20:GxiA23UxiLw8+1AmeM8yMOlsfp/3dMPgK+5xqwY9Tf9Bc/u4Au1EtvZHCc/fxrhseeQzXsof+dGBpLePXV+GzjUo3a2eVwbgTre99QlUh4MzWRxHAIxUFI/7u0hVfdNrOA1uVHCdmCMxSKxRFjJtSI2j0oFkfLhw65Kf5waBp0i32LYQcbRR46amhzLIG3BNf5xdsa5qc2ZNc1Hm3Wxz+e+XwroHWjlsFMjuCdNM189gGni/PsGlN+cEXLg+CAww X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Mar 2017 17:08:42.1637 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR1201MB0133 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Boris, On 03/29/2017 10:14 AM, Borislav Petkov wrote: > On Thu, Mar 02, 2017 at 10:16:05AM -0500, Brijesh Singh wrote: >> From: Tom Lendacky >> >> When a guest causes a NPF which requires emulation, KVM sometimes walks >> the guest page tables to translate the GVA to a GPA. This is unnecessary >> most of the time on AMD hardware since the hardware provides the GPA in >> EXITINFO2. >> >> The only exception cases involve string operations involving rep or >> operations that use two memory locations. With rep, the GPA will only be >> the value of the initial NPF and with dual memory locations we won't know >> which memory address was translated into EXITINFO2. >> >> Signed-off-by: Tom Lendacky >> Reviewed-by: Borislav Petkov > > I think I already asked you to remove Revewed-by tags when you have to > change an already reviewed patch in non-trivial manner. Why does this > one still have my Reviewed-by tag? > Actually this patch is included in RFCv2 series for the completeness. The patch is already been reviewed and accepted in kvm upstream tree but it was not present in the tip branch hence I cherry-picked into RFC so that we do not break the build. SEV runtime behavior needs this patch. I have tried to highlight it in cover letter. It was my bad that I missed fixing the Reviewed-by tag during cherry picking. Sorry about that and will be extra careful next time around. Thanks ~ Brijesh