Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <d0c5c217-646c-fb02-51e6-41c8fcdc00b4@intel.com>
Date:   Sun, 23 Apr 2023 14:12:14 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.0
Subject: Re: [PATCH v2 00/21] Enable CET Virtualization
Content-Language: en-US
To:     Peter Zijlstra <peterz@infradead.org>
CC:     <seanjc@google.com>, <pbonzini@redhat.com>, <john.allen@amd.com>,
        <kvm@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <rick.p.edgecombe@intel.com>
References: <20230421134615.62539-1-weijiang.yang@intel.com>
 <20230422130240.GJ1214746@hirez.programming.kicks-ass.net>
From:   "Yang, Weijiang" <weijiang.yang@intel.com>
In-Reply-To: <20230422130240.GJ1214746@hirez.programming.kicks-ass.net>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?S0UyQ3ExTklieWZ3djdwdHZzZEJ6SG9tSUdKWFkwNXc5UUFGNTc4UlNHQ3JQ?=
 =?utf-8?B?N3pKTTJoVkNWSG1waGFpN3NxYnljL2VlVVBmbUVSeXk4OERLVmFDaTMwVVJU?=
 =?utf-8?B?am5LS2FDK1pPVy9aL3I2amZKZVFSOWx5ZE9yY1NKTGNVVXdtaEpMTUZSakJJ?=
 =?utf-8?B?QjBqclIzRkdQckllWHZjYjhYNXBjR1U3Nm8ySG1DSStTNUZRaTUxYnZpSk00?=
 =?utf-8?B?dW9SejhhT1NrSm11cU1pa2FjM3ZEWk9qUkI2SFpJVkR2WG4yRjVKVVVQMWtj?=
 =?utf-8?B?R2pRRFBuSWxCTS9RVFJRQVpuSUk0N212ZE82aFRSdDJqZFZZakRkYlpRTVlR?=
 =?utf-8?B?TWpNUTQzUXN3TjVtdzJjWlBmcjVGckx3aHRqamMvelJnaHpmR2Q2b0RUN0lq?=
 =?utf-8?B?NEpjK1k5NHY0WitBZ3hkaEFaWkRheVJ0a2JycHNTZGdOUElSL0lxQS83SjR6?=
 =?utf-8?B?NjFOWHBmS1ByWUhWWThFNFhBR3RFZ0QxalA3VzZWeURjdXc1L2hYZVBDZ3dZ?=
 =?utf-8?B?VXR1OU9WSExRZ0wwSFBZU3QzdlF2NVdWdjcreklXMmxITjhITkJrS29HSm1O?=
 =?utf-8?B?NW00UTljWkdOK3p3UThQaGZLeXJ6b1puNDBqaW5mSzhnbUxsbDJ4YzZvWkdl?=
 =?utf-8?B?YU9OcHZFTXZwN1hvRmFqMldsYjFJNlVwMHlBRXk5UW9CQ2tIcmR2bUx4NDNh?=
 =?utf-8?B?NDVwYy9od0l4WVlUUnUzK1k4ZFQ1eWdaeCtwOEVBOTVFUzY0RHkxdSszMjc0?=
 =?utf-8?B?VSswaEFReWg3YzQ0cWZuU3RZRHUyL1kwRGZFYWNJY2t5K3A2ZE1vcVk0VkdG?=
 =?utf-8?B?SGlnUnBRSlcwVk5DWXpoeUFCRFAzcFFURk41VHFmaHJvd0J0SWFIZkYxdFl4?=
 =?utf-8?B?SXJxUFprT0l5QWVabWswWVNwbzJOY1RjY1MrKzdTMVZzY29Obzd2cU13TTZp?=
 =?utf-8?B?d3VXQnI1WEc2Q01NSmxrZWFFRW1KYkFKeUY1VVgydE9mRUNYbWJ2S0J1cGtz?=
 =?utf-8?B?ZU5sZTJUd3lCR1BNait5NGwvWnRzU1k5QU9lOFEwdGdhNmc4Zk56ck5CU0E4?=
 =?utf-8?B?a3Z4d0thSnduY1dtUlRBYTB3cHphRXp0U1BsQkw5VnRpbzQyMlJZWUxCbndW?=
 =?utf-8?B?cHdYcXQyZVd2citJZXZuUVlDWkIvcUdtTW9jR1MvK3EyKzhUMitUSDZ6WWdP?=
 =?utf-8?B?T0kwQ0YzZ2tPaEhJTFVFU09KRDdCbHQ3Q2FXUFA3TE1PWHBrS0R6RGpqMWpm?=
 =?utf-8?B?QkJLV0c2NmJNQWNIL2M5N0JudnM1WFk1djFpQ2NCVXdMMHZzTHlGdkwyTyt2?=
 =?utf-8?B?aS80SHMzSndXRi9HSmRvcHpsOFVQUG0rd0dPWU9EaTUxYlBRbzhGTXlNYlVF?=
 =?utf-8?B?aW5lVzdLSGVIRXNsTWN3VjlMREVVNXhVR2VyZ2xaQ1JpR2JlT0EwL2o4anlM?=
 =?utf-8?B?VGJGZ1FPdkYxbWVqN1M1SDhLK0pTVWhjVGExUTBTL0VqSDVMUmxFdUt0cUtt?=
 =?utf-8?B?VzNUUGZZZEw3UzI3UHcydEhveXlQc3loOExKZHd0bDdPQktBaXhESEhndzE5?=
 =?utf-8?B?ZVBtcS9jSExsNHFVMGJXdTNXVlROYVlSOXhDcWJBSjIzQkhRSyt2OWgvcFBr?=
 =?utf-8?B?SWVwa1R1eVlpMlJVN29icWphMlZoZmppT21hR2IxVGhIdU9BWDczelluT1FV?=
 =?utf-8?B?N25vYm9CVEROTk9wZG41R2wrcWRTdDMwb2JwajZudWI4MkhsakI3T0UxWnJ1?=
 =?utf-8?B?dTZVU2pvZ1pzeGNUTjhXaGVDVU1XR05iTEFaK1lBbmJ1NWEybWk5ZGpxcUlU?=
 =?utf-8?B?YVo3T2hMZTBWY2Z6dno3QVB0QWNWNlpFZ0FIOEw0amFLQ0xQQzBMYTE1bHRv?=
 =?utf-8?B?cE1tQVBJd0hyY2xrWXR2NVRSdzhnTnZ2dE5FY2oxcGN0QWdOOVlWOHdXWXlm?=
 =?utf-8?B?b3k0NngydnY4RmNGU2tZY3ZETHYxVTRscC95dFBaQW42MFVnaGFHSTR5QVI2?=
 =?utf-8?B?Qi9EZTlzK0FLNFMwV25HZG1pZjlxRC9UcDFOeWZmOU41YmphWlNSeU9iZG5H?=
 =?utf-8?B?d1RHUmI5MW9Jd1Mvb2JvWStqZ3c5eEFramlTVjcxdWE1SGl5T2hwMmlTQ0JU?=
 =?utf-8?B?Y0JvUllmSm1LbDJlYjdaaGI3dms1TDRQQ016cXNDUCtjMDZxcVBjZnlZakcx?=
 =?utf-8?B?UWc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 14c0328d-05dc-4643-c4fd-08db43c1b376
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB4965.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Apr 2023 06:12:22.4744
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: DsPZKR9/XtGkt/Ulh4li04jkEvAePdOCmd0q9MUlpojlzHL+GsNYlrIDT7fGiIpQ0QEIerAWKAw/CsHVoGslZg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB6351
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-6.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 4/22/2023 9:02 PM, Peter Zijlstra wrote:
> On Fri, Apr 21, 2023 at 09:45:54AM -0400, Yang Weijiang wrote:
>
>> Implementation:
>> --------------------------------------------------------------------------
>> Historically, the early KVM patches can support both user SHSTK and IBT,
>> and most of the early patches are carried forward with changes by this new
>> series. Then with kernel IBT feature merged in 5.18, a new patch was added
>> to support the feature for guest. The last patch is introduced to support
> Yeah, at the time I had to hack up kernel IBT guest support, because the
> platform I had to use (tgl-nuc) didn't have serial and so I had to use
> KVM :/

You did it and beat all the hurdles :-)

>> supervisor SHSTK but the feature is not enabled on Intel platform for now,
>> the main purpose of this patch is to facilitate AMD folks to enable the
>> feature.
>>
>> In summary, this new series enables CET user SHSTK/IBT and kernel IBT, but
>> doesn't fully support CET supervisor SHSTK, the enabling work is left for
>> the future.
>>
>> Supported CET sub-features:
>>
>>                    |
>>      User SHSTK    |    User IBT      (user mode)
>> --------------------------------------------------
>>      s-SHSTK (X)   |    Kernel IBT    (kernel mode)
>>                    |
>>
>> The user SHSTK/IBT relies on host side XSAVES support(XSS[bit 11]) for user
>> mode CET states. The kernel IBT doesn't have dependency on host XSAVES.
>> The supervisor SHSTK relies on host side XSAVES support(XSS[bit 12]) for
>> supervisor mode CET states.
>>
>> This version removed unnecessary checks for host CET enabling status before
>> enabling guest CET support, making guest CET support apart from that of host.
>> By doing so, it's expected to be more friendly to cloud computing scenarios.
> I've on ideas about cloud stuff, but there is fundamentally no relation
> bewteen the host making use of IBT/SHSTK and a guest doing so, so there
> should be no dependency there.

Definitely as long as there's no quirk required for the features! Also 
eliminated the

upgrade efforts for host end users in order to play with CET.

>> To run user shadow stack test and kernel IBT test in VM, you need an CET
>> capable platform, e.g., Sapphire Rapids server, and follow below steps to
>> build host/guest kernel properly:
>>
>> 1. Buld host kernel. Patch this series to kernel tree and build kernel
>> with CET capable gcc version(e.g., >=8.5.0).
> Why does the host kernel require a CET capable toolchain if the host
> kernel does not in fact need to make use of these features in order to
> provide them to the guest?

Oops, this should be a typo, guest instead of host build requires 
qualified gcc version.

Thanks for the comments!