[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] x86/msr: fix X2APIC_LAST
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Edwin Torok <edvin.torok@xxxxxxxxxx>
- Date: Tue, 26 Jul 2022 15:29:55 +0000
- Accept-language: en-GB, en-US
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=5hL7EQoyvMmCBQsG+KGVhCad1N/ciyZZhE8Vz9ykLWs=; b=a2Wv2yOh9FUnQFC/ivjqSFhzqQdVxv5CxKPIQN+EDTn752Mz17es4bwQJZfCxExG0yvbaV5GbQDDsUUqOeNP+Vvc0n1xLCay4s+cbTomNjdbbUH7Soz1dsRRQclGxqu+Oy+AaJEiOol0jrcSGqUSDislkkqJcgOkbKoMq/DtezpLoAXaD2c1tfgf/B+GbDbYLfheCn2WbilLLSvgDFKc7ztD11aujhe863tPYyj7r2xogXrFoffKGJvvpVV85nnNSJdG4+lOkM4rJd0S5q6ZKL/65U6eQYP3bxXpdhBa5JHkA+O3C31k4TKyXHqh0etTjcOHXmQakdWhFOzIQ2rx/Q==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kL0EnB/O8qImq+wiT6CjD2oOn8Q9VEjs9Dgp4po6XMoB2gNm4YeN4mQDZebg0GgpZvO6riAaAl7EmnTb3H4QW1UQGhMj0mUnuOean4kO1AWoy1/x1O9e3VlXCfTFSevaUKyykofhfLLpSLlhM4K2q/pYZd3AxVUNkNUq37YIrUPd14NPQwLvcZ2loKGcBDGMENpwBimG+FqJJTQX3tczk6wGSyq87EMSz4qBq3v5O4TTG8GnS+wxkGMf6702/ZlD12cssWwdIl8wdt7lnU1os44Y+Zt4xgSGnsQf6dod4pq5tH84gBnrlRj32fFHKx8RSJqqztNvvIOzJxm96ArNcg==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
- Cc: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>, Roger Pau Monne <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Tue, 26 Jul 2022 15:30:07 +0000
- Ironport-data: A9a23:BZd4F65efsqJ5mKc1P/KwgxRtCTGchMFZxGqfqrLsTDasY5as4F+v jEdWGnSO/jbNGLxLdonaoyy9RwA7J/XnNAwSlE9q3hjHi5G8cbLO4+Ufxz6V8+wwmwvb67FA +E2MISowBUcFyeEzvuVGuG96yM6jclkf5KkYMbcICd9WAR4fykojBNnioYRj5VhxNO0GGthg /uryyHkEALjimUc3l48sfrZ8ksw5qmq4lv0g3RlDRx1lA6G/5UqJMp3yZGZdxPQXoRSF+imc OfPpJnRErTxpkpF5nuNy94XQ2VSKlLgFVHmZkl+AsBOtiNqtC0qupvXAdJHAathZ5dlqPgqo DlFncTYpQ7EpcQgksxFO/VTO3kW0aGrZNYriJVw2CCe5xSuTpfi/xlhJFsmZokb3uxyO0pP3 8Y1AWEhVE2Ilu3jldpXSsE07igiBO/CGdpF/11Fk3TeB/tgRo3fSaLX49MexC03ms1FAffZY YwedCZraxPDJRZIPz/7CrpnxLvu2ia5Lmwe8Q3KzUY0yzG7IAhZ3bTzMdyTZtuQQsZ9lUeEv GPWuW/+B3n2MfTAmGDeqir12IcjmwvfBp0eDbKmrcRlnQa9ljAZChlLFl+09KzRZkmWHog3x 1Yv0igkoLU29UerZsLgRBD+q3mB1jYDX/JAHut87xuCooLE7gDcCmUaQzppbN09qNRwVTEsz kWOnd7iGXpoqrL9dJ6G3rKdrDf3NS1LK2YHPHUAVVFcvIalp5wvhBXSSNolCLSyktD+BTD3x XaNsTQ6gLIQy8UM0s1X4Gz6vt5lnbCRJiZd2+kddjjNAt9RDGJ9W7GV1A==
- Ironport-hdrordr: A9a23:4jq0CqODbvXjv8BcT2L155DYdb4zR+YMi2TDiHoddfUFSKalfp 6V98jzjSWE8wr4WBkb6LO90DHpewKQyXcH2/hqAV7EZnirhILIFvAp0WKG+VHd8kLFh4lgPM tbEpSWTeeAdWSS7vyKrzVQcexQpuVvmZrA7Yix854ud3ASV0gK1XYaNu/vKDwTeOAwP+tdKH Pz3Kp6jgvlXU5SQtWwB3EDUeSGjcbMjojabRkPAANiwBWSjBuzgYSKUiSw71M7aXdi0L0i+W /Kn0jS/aO4qcy2zRfayiv684lWot380dFObfb8yvT9aw+cyTpAVr4RHoFqjwpF5N1HL2xa1+ Ukli1QffibLUmhOF1d7yGdgjUImwxelkMKgWXo/UcL5/aJCg7SQvAx+75xY1/X7VEts8p717 8O12WFt4BPBReFhyjl4cPUPisa33ZdMRIZ4JEuZlFkIPwjgYVq3Poi1VIQFI1FEDPx6YghHu UrBMbA5OxOeVffa3zCpGFgzNGlQ3x2R369MwM/k93Q1yITkGFyzkMeysBalnAc9IglQ50B4+ jfKKxnmLxHU8dTZ6NgA+UKR9exFwX2MFrxGXPXJU6iGLAMOnrLpZKy6LIp5PuycJhN15c2kI SpaiItiYfzQTOaNSSj5uw6zvmWehTNYd3E8LAs27Fp/rvhWbHsLSqPDFgzjsrImYRsPvHm
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Thread-index: AQHYoP4qyGlTjJTj1UaX3N7imIGaP62QwNEAgAAGTYA=
- Thread-topic: [PATCH] x86/msr: fix X2APIC_LAST
> On 26 Jul 2022, at 16:07, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>
> On 26.07.2022 16:43, Edwin Török wrote:
>> --- a/xen/arch/x86/include/asm/msr-index.h
>> +++ b/xen/arch/x86/include/asm/msr-index.h
>> @@ -148,7 +148,7 @@
>> #define MSR_INTERRUPT_SSP_TABLE 0x000006a8
>>
>> #define MSR_X2APIC_FIRST 0x00000800
>> -#define MSR_X2APIC_LAST 0x00000bff
>> +#define MSR_X2APIC_LAST 0x000008ff
>
> May I ask that then the now open-coded values of MSR_X2APIC_LAST
> (two instances in vmx.c using MSR_X2APIC_FIRST + 0xff) be replaced
> at the same time?
> On 26 Jul 2022, at 16:03, Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx> wrote:
>
> On 26/07/2022 15:43, Edwin Török wrote:
>> The latest Intel manual now says the X2APIC reserved range is only
>> 0x800 to 0x8ff (NOT 0xbff). The AMD manual documents 0x800-0x8ff too.
>>
>> There are non-X2APIC MSRs in the 0x900-0xbff range now:
>> e.g. 0x981 is IA32_TME_CAPABILITY, an architectural MSR.
>>
>> The new MSR in this range appears to have been introduced in Icelake,
>> so this commit should be backported to Xen versions supporting Icelake.
>>
>> Backport: 4.13+
>>
>> Signed-off-by: Edwin Török <edvin.torok@xxxxxxxxxx>
>
> Having done some archaeology, this changed between SDM 68 (Nov 2018) and
> SDM 69 (Jan 2019)
Thanks, I've done both of these updates in the v2 I've sent out.
Best regards,
--Edwin
|