From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932103AbdI0PxZ (ORCPT ); Wed, 27 Sep 2017 11:53:25 -0400 Received: from mail-by2nam03on0044.outbound.protection.outlook.com ([104.47.42.44]:38176 "EHLO NAM03-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752489AbdI0PxW (ORCPT ); Wed, 27 Sep 2017 11:53:22 -0400 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=brijesh.singh@amd.com; Cc: brijesh.singh@amd.com, Tom Lendacky , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Borislav Petkov , Andy Lutomirski , John Ogness , Matt Fleming , Laura Abbott , Dan Williams , "Kirill A. Shutemov" Subject: Re: [Part1 PATCH v5 06/17] x86/mm: Include SEV for encryption memory attribute changes To: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, x86@kernel.org References: <20170927151329.70011-1-brijesh.singh@amd.com> <20170927151329.70011-7-brijesh.singh@amd.com> From: Brijesh Singh Message-ID: <21170678-7a8e-a66e-37ef-7c32ba336a65@amd.com> Date: Wed, 27 Sep 2017 10:53:17 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20170927151329.70011-7-brijesh.singh@amd.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [165.204.77.1] X-ClientProxiedBy: DM5PR06CA0063.namprd06.prod.outlook.com (2603:10b6:3:37::25) To DM2PR12MB0156.namprd12.prod.outlook.com (2a01:111:e400:50ce::19) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 99db67b2-04a3-437d-c8d9-08d505bfe09b X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(2017030254152)(48565401081)(2017052603199)(201703131423075)(201703031133081)(201702281549075);SRVR:DM2PR12MB0156; X-Microsoft-Exchange-Diagnostics: 1;DM2PR12MB0156;3:rjAvsW/lJZux2V5lEor9N0bFQOJs9lU+DTb1bm4giEsGvWzAiyN+LDvZtOUXfyEuAoDKeaNdYJ3uQ8G1YCA4s8LSNlm2pfRNMUJ8EO4sepSWkygWu2Mjs9yhju+FmklJJaTEyd7LtfmwpetJqf6+GnOZGJWCtHkVKT/S7gQUm0iax9H8eIksTmpaaglxNU+KD/nRUcHLD2MQpWugWGJvEW8Fn+yUCHjREQddH14WAWIkbasEvPS/PLOmfXIuilb6;25:2TDNzdBJ0LckjIbxroWnpaZMRvhkUNosAI2yHiEE1KZ3ZVkRyb8DW2sgw/juMpoCs6PhcGDDoiv9eotrswJi38yptn/ZAzc/sUJa9AQWaMzSuRI7S3ZnkVMm+nTqqwOZIigPBRTZlM6/PG6jW8i0hxDS9oK6+rvglPThlRt34esgsycogXLb733jbJI0u0riTAnOYWHZQGOvhLXn7TGnN8mxESl7T7PRnYXMNYRrNv09pozUFBs+gZ7GZSAlnx8LxNlh+ZTTddTJbhYcIRYlN0QUXyqxb3QvlWZabxiE38ZoF7BStJODpvBm28SalKEDVwbH/VkW3VGG7MDk9HXsMg==;31:oUM9Vulk62uhHQ4vvum/1AbOJMliXP6iElBrwnNqQi8uKYcMxSIfoM4tldYXNnKn36okyzI9o7en/qBxSpSo9MreRJcoG3SoF+NZpM/UNWfyb7fzLcPqDA1s7c3ecP3+j3edUF1m669Hhu+lKGbwd92zwUrR5vPdslN+0O4zlWtGNq2jcwjcPENgq1+xMDAqlTgoWN3Gnp2re/71gs8nygFzE4RtvQFNLXmwtxjVll0= X-MS-TrafficTypeDiagnostic: DM2PR12MB0156: X-Microsoft-Exchange-Diagnostics: 1;DM2PR12MB0156;20:1lJ6zsSPvG5+OEJdyO9UWu2H0iUoC1mxwS4zLo5wJTm0XwdOG+fG6uX+XbBCFyBr0YX+Mc8KD5rJtCb0ONp0lldG8Lcc/pB/ZQNpprXwyOq421+YzZmW/kxu3bgwmNhLhbk1p5qfqC+zgK+BeiKVY4le9P1QS1MjSz8koxBfaQuVgtu9v0oSUYb7mfAiefIOFmauYzV2cyKw1pXbLMLOyI40ZVzMKMH9uCW9u7hQmOPEVhx0QwGQWIybKrjjgsdjqlLxAfBs3WWHYo/SzOBnp1MLgv3CAlQ37Tb7Pa4UOmMsDNJrBuqsSiN4z3V5siyrnDIN9JKJXjtr8EXkzuJbKhRYe5Zt0UqeQQxddMPnUMn76AWoSsAQ3S7QNV/Dpr82kB8Cam1qKAY3CSuJIqTY1XsLpHBX70XQ8zRZ0z4rnYz7ljg5KqkvQ4qFCkHfoKaaG24HZ3xggZDnOq1c7eykT7sUetKaEfMeGiCLxy1t9ZnCB+EN6N/uxLFu2L4fD0Fp;4:ZqFbpJqM85ayN7DCDP45zzWhBBxmj0TzhepG1QbE9vwMY0vc2Dq04cf3F2n2N0hGr2Fu3fUvqvcQXeSxakTrIMjIOrjX140gogeZx8za0Wpyel5+RCJ38m7LOHtVujnTTTFRTZ90vazhgVlR7dE84bKGd7CumahJcWADfG6J2F4Gft6w5CnT9A6FgAnXhaKqHRqQZwDxgXL4BiOJUMK++GggpwMR82sTpk/qrUx2yCHocPP9zXbtJ3QPBNuAOyHM6MNRhYAJJcNoFQu2YwReiU89LWftKKX3K0R6l8dBdu0l/2MG6+0GvTBLDBEyfx1UvFLtzDiazQu3YZEPNsEtr2Hk1J1Ur3WmMcWdN3F8WHg= X-Exchange-Antispam-Report-Test: UriScan:(9452136761055)(767451399110)(228905959029699); X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(3002001)(10201501046)(100000703101)(100105400095)(93006095)(93001095)(6055026)(6041248)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123555025)(20161123560025)(20161123562025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:DM2PR12MB0156;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:DM2PR12MB0156; X-Forefront-PRVS: 04433051BF X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(6049001)(6009001)(39860400002)(376002)(346002)(189002)(24454002)(377454003)(199003)(7736002)(4326008)(77096006)(3846002)(478600001)(6116002)(97736004)(31686004)(2950100002)(6486002)(86362001)(64126003)(54906003)(305945005)(31696002)(7416002)(229853002)(36756003)(966005)(66066001)(65806001)(83506001)(53546010)(65956001)(189998001)(58126008)(5660300001)(50986999)(50466002)(16526017)(16576012)(76176999)(65826007)(54356999)(53936002)(230700001)(101416001)(2906002)(33646002)(47776003)(316002)(6246003)(23676002)(6306002)(8676002)(8936002)(25786009)(6666003)(81156014)(68736007)(105586002)(106356001)(81166006)(6606295002);DIR:OUT;SFP:1101;SCL:1;SRVR:DM2PR12MB0156;H:[10.236.136.62];FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtETTJQUjEyTUIwMTU2OzIzOlNKdFc3YlRnVmxWc1R3Z1RBSVRxNXlhMEQ0?= =?utf-8?B?TWlqUElKSlhtcCtGWXAyeTJNZldSUVgvL2xOMk92Rm0rY3lCelQ0OEMraGJn?= =?utf-8?B?bDdrMHdvZUExOHFLTGdGcllQQlRDTU9vRHFmam9IWnpWL2Zxc1d5RDk4WHd3?= =?utf-8?B?WjN6eElKUnY3ZXpDSTVMTFhVTXhIZUswQXZuODVmTHNGcS85OVFuODFEa0Qy?= =?utf-8?B?NW14amx0WEhzRlkvN2t0dTYrZGNxL3NUdlhMNjRYcWR2SW9oTVRKT0hPZnVL?= =?utf-8?B?eGFvc3JTSTFrQ0xQeFBSWERrajBPcERWdXcwNStRa1diT0xEbkE2Z2lHWFhk?= =?utf-8?B?MUkxNUU1Z3MrTk5jeUUyaHRpd3Y4alJ5ckV4cDJma3pxelFFSU1uNnRTNUVV?= =?utf-8?B?SnJrcmhkSnhCS2xqbFZUbmhTVVhDZVpaajYvK2JRY3VvajlwcUlFa3BFMmY5?= =?utf-8?B?eTljcnF3WjFxam5oVzNGTHQrSW55aVJsa2hYNTJlWmlyYlJPcWV5eWVXaElP?= =?utf-8?B?Tm0vT2VMYUVhQVZjRk5ZMm9uQ2lIZHI3cnZCcGdCcnhZcWhGVm9MZHUvaG9r?= =?utf-8?B?YWp2cXVlK1hQVHhVTkRDYkxZYjdJckFDVTFOR0EybU53WjJKYW5qNzQ5OVhC?= =?utf-8?B?VndLNXB1TjYzZmwybEhEUEdwUit2bW1JWTNWZFJ2RG9DcndJN1pLQndZS2Np?= =?utf-8?B?eGU3NlhEb3JCS0JCVVBSZ2diU2txZHkzcCt4b1c5MDdnN3VXcE5ZWjhtOVJ6?= =?utf-8?B?NlNvc21UN0VpQ3ovVmxNTjAzTTZvdHhhZDdPdjB3STlPUG40dHptaXArb3c2?= =?utf-8?B?eTFiVFpDdU5ZSEFQaFZzY0dGMHg1YS9xUmluK2ttU2RQR2I3WjJWN3ZCcEo5?= =?utf-8?B?YWl1d2I4UUZBbTJnaVNYbGpKVEV4bEZZcWRpV0xGMXlOdkw2cGoyY092Mi83?= =?utf-8?B?bE14WHhCMzlqT25iVkhIcUlsOWt4cWx2MnhVUWF6a0swbkpkQ0dBbDVmQWJq?= =?utf-8?B?clBoL3g1bStrMEplWm9lSktzYzVYdDB1VXgwUG5Gcm83UFdmcGxxOHRyazBQ?= =?utf-8?B?dEMwMlUxWVpSVHVjb0VGNlUvU2hUbHg5KzlnMGpPSXlRbmpOVTNVejBJV2hD?= =?utf-8?B?WVluMkZWSU9UN3dSdncydUovTXp4KzR1SFFZNE93a2FubnMzbWxRaTYrbFFn?= =?utf-8?B?amh2R0E1a1dlekVaYnh5TFY5TGpHaXJ4K2JZNzJoTlNBeEsxL0hKekFRZkVq?= =?utf-8?B?MHZ3dm5TOGxFT3dHbE1BTURsU3p3dUx4YmhJTTIwZU1GQmliOHAxRHhWL2xy?= =?utf-8?B?cGVqaCs0WEh0b05RZXJEZ0h6NTEzUFVuSjV6QmNydlZmSWhUWFFxaEFNTXRT?= =?utf-8?B?T280NExaRmRLSWt6Z1NxSkdaOVkyejJtY3lid0FQNGhkZENDQ21Ea3ExQzRm?= =?utf-8?B?c2ZBTys3djhqbjJlemVlRFAxaU9yTU4rVDE1MGYxczBNbXR0OFM2RWtxMSsw?= =?utf-8?B?QlYrdXpPYW5wRGFDVnFJSnpONjkyQVpaUm81MVdodW94dUI3QUFBMVlqb1Vn?= =?utf-8?B?UGNLN2NHanhCSUQ0RmdQSWhGTnVFQWtIS0xDUVlqYUpOVTd6SUZzR1NtZHNJ?= =?utf-8?B?dno4NGQxMTVvOWswQmtkUEVlRWxKK28zZElSR1FycWlYYW9kK09lK0krV0Jj?= =?utf-8?B?c1haYi95U1lkSWJGN0ZnaHY1YzBHNUowVXA5djlrQ0tkQUxTUVNOOGplUC9N?= =?utf-8?B?SmlGa2VWYVA3OGRVS2tSVFhJWEEwdS9VTXI2T2VUOUYxK0ZaaTdOREJDWlNw?= =?utf-8?B?YjRjNzRKYktQNVM0aTFzRTE3T3NhblpiUHR1OGtLaXlCVG5kRStpOSsxVDBt?= =?utf-8?B?N0dLWHBndndyYmN1MGlWd3V6ZTFiRGJESVBTTnBZbHgvZ3QvQ3dtbXZWc1I1?= =?utf-8?Q?1TJ1HxYMw+r/sHlf7NdS18qxYJZ7ek=3D?= X-Microsoft-Exchange-Diagnostics: 1;DM2PR12MB0156;6:BQSlYcGqf5kAJCwpbvkIPzWrZq7ijpmFJKy0Nuc17MttSznm6YOAd+L1lJMjE/qYJqcYU8i50+zbQjfdVo+EePfzuo9Y4w2TLSzxjtLC22N5wvYksmjI3vWppHV5V5LZzEt0CFU5Kqb4TaoUbe3y0/pcMN1uFtz0Bk4gw6nmpG0n9szmhQbB89VlJRBSyGenKiRUemo7IK7ZEejJhutZpJVZgaIr2JxmYPmre57uell05Gtm21uZxMCQhGc8uUEwHgQESfXpRUW9kbOGr/5Em3/07332ziTjlOsoEkXy/ogAekiI5EDhVgDhDZvOruCiNO9L74zxt2p5pW72EeI3lw==;5:pDUoZgtefW8yVuQIzGvd6qod02omMDhBuU2WfojIr0lEh51McVt2eTOuXUIsceOaBKgErO4jSFruDVNPh7lqIXUWYXH9aoeKZrqGxBx9usR3//IMKHKkBrMnIpHgo3wUbBL0UUFAIP1kIJijSd3gVw==;24:81LWarVI0w1LSSctTQwtAQ0DMmr0SC37RRC7IlugIXbI0wtn16vwD4kqGqfUyouPxXlS1QO09QKCvEv9g4M7bwqbCRNRr30RV6uf/Qbm1DQ=;7:gMpB5QYEOf4A+dVinbE6MwRxJm7g1GU4X9rYYeiGgIVwIgJSuuNa9Khl/Wr9vDZDH19dK/Hf3lFBLFjQ2eGiuQUsNWVr9EnmpamsnZP4KBIg/h698LH0+bR5XJ3ejCQPgmvmcp/UPDd/ef3ioJlWnPthmJDRAs9CuiQZ7y9X/DL1QFkjSlaUaW/XB+w9izBaA8BCod3rDHkNgiVZ9kgYaJlNqoHwOyD90XsVk+aX/fQ= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;DM2PR12MB0156;20:QuRFZIx+HWkquefmps4IJ9UsATE8scJd+VUFjqM539KS1hiLGNudH3S73s9YoupRDQGhTXL3aW819vFDzPxrz7QiW62UMbSntWSag1d66Z0UAIHK7gOUgEDKRRvHgPsAZjwsZPPmXvyC3LpEJL56RNiOqP/otRS7N/P9WkBsFXA3Gi+gAGhvpjsZK1oLiuhmAczBL5rDpwGvgmlaR2I7H8ucYdbQ2tfORcyYzgdbnaOJy7ngriCOQchX4W4/KETg X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Sep 2017 15:53:20.2679 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR12MB0156 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/27/2017 10:13 AM, Brijesh Singh wrote: > From: Tom Lendacky > > The current code checks only for sme_active() when determining whether > to perform the encryption attribute change. Include sev_active() in this > check so that memory attribute changes can occur under SME and SEV. > > Cc: Thomas Gleixner > Cc: Ingo Molnar > Cc: "H. Peter Anvin" > Cc: Borislav Petkov > Cc: Andy Lutomirski > Cc: John Ogness > Cc: Matt Fleming > Cc: Laura Abbott > Cc: Dan Williams > Cc: "Kirill A. Shutemov" > Cc: linux-kernel@vger.kernel.org > Cc: x86@kernel.org > Signed-off-by: Tom Lendacky > Signed-off-by: Brijesh Singh > --- I just realized that I missed picking Boris R-b tag in the commit [1]. If another version is needed then I will include it. thanks [1] https://marc.info/?l=kvm&m=150565715508940&w=2