ACK: [PATCH] cpu: msr: do not check SYSENTER_* MSRs

[ivanhu]

On 04/06/2018 09:11 AM, Alex Hung wrote:
> This is the same as ca55cff002afb73f but for Intel's CPU
>
> Signed-off-by: Alex Hung <alex.hung at canonical.com>
> ---
>  src/cpu/msr/msr.c | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/src/cpu/msr/msr.c b/src/cpu/msr/msr.c
> index 156f75f..362bb33 100644
> --- a/src/cpu/msr/msr.c
> +++ b/src/cpu/msr/msr.c
> @@ -338,9 +338,14 @@ static const msr_info IA32_MSRs[] = {
>  	{ "FEATURE_CONTROL",		0x0000003a,	0x000000000000ff07ULL, NULL },
>  	{ "BIOS_SIGN_ID",		0x0000008b,	0xffffffff00000000ULL, NULL },
>  	{ "MTRRCAP",			0x000000fe,	0x0000000000000fffULL, NULL },
> +	/*
> +	 * LP#1582005 - Do not check sysenter MSRs, they will be different on
> +	 * each CPU, so checking them across CPUs is incorrect
> +	 *
>  	{ "SYSENTER_CS",		0x00000174,	0x000000000000ffffULL, NULL },
>  	{ "SYSENTER_ESP",		0x00000175,	0xffffffffffffffffULL, NULL },
>  	{ "SYSENTER_EIP",		0x00000176,	0xffffffffffffffffULL, NULL },
> +	 */
>  	{ "MCG_CAP",			0x00000179,	0x0000000001ff0fffULL, NULL },
>  	{ "MCG_STATUS",			0x0000017a,	0xffffffffffffffffULL, NULL },
>  	{ "MCG_CTL",			0x0000017b,	0xffffffffffffffffULL, NULL },
Acked-by: Ivan Hu <ivan.hu at canonical.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20180411/8c7cc124/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20180411/8c7cc124/attachment.sig>