From: Dave Hansen <dave.hansen@intel.com>
To: Sathyanarayanan Kuppuswamy
<sathyanarayanan.kuppuswamy@linux.intel.com>,
"Kirill A. Shutemov" <kirill@shutemov.name>
Cc: "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>,
Borislav Petkov <bp@alien8.de>, Andy Lutomirski <luto@kernel.org>,
Thomas Gleixner <tglx@linutronix.de>,
Elena Reshetova <elena.reshetova@intel.com>,
x86@kernel.org, linux-coco@lists.linux.dev,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/4] x86/tdx: Use ReportFatalError to report missing SEPT_VE_DISABLE
Date: Mon, 12 Dec 2022 08:10:14 -0800 [thread overview]
Message-ID: <e13331b2-a68c-c823-d02f-ba50949214cb@intel.com> (raw)
In-Reply-To: <2e305bb5-9595-3531-6134-24344ff5c797@linux.intel.com>
On 12/9/22 12:51, Sathyanarayanan Kuppuswamy wrote:
>>>> + while (1) {
>>>> + __tdx_hypercall(&args, 0);
>>>> + }
>>> Instead of an infinite loop, I'm wondering if the guest should panic after
>>> retrying for few times.
>> Hm. What difference would it make?
> IIUC, the goal of this patch is to report the fatal error to VMM and panic.
> But, if VMM does not terminate the guest as we expect, rather than trying
> continuously, isn't it better to panic ourselves? That way the behavior
> will be similar to what we have currently.
What does "panic ourselves" mean exactly? What is the current behavior
which that would match?
next prev parent reply other threads:[~2022-12-12 16:10 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-12-09 13:25 [PATCH 0/4] x86/tdx: Changes for TDX guest initialization Kirill A. Shutemov
2022-12-09 13:25 ` [PATCH 1/4] x86/tdx: Expand __tdx_hypercall() to handle more arguments Kirill A. Shutemov
2022-12-13 22:44 ` Dave Hansen
2022-12-09 13:25 ` [PATCH 2/4] x86/tdx: Use ReportFatalError to report missing SEPT_VE_DISABLE Kirill A. Shutemov
2022-12-09 15:42 ` Sathyanarayanan Kuppuswamy
2022-12-09 17:06 ` Kirill A. Shutemov
2022-12-09 20:51 ` Sathyanarayanan Kuppuswamy
2022-12-12 16:10 ` Dave Hansen [this message]
2022-12-12 16:37 ` Sathyanarayanan Kuppuswamy
2022-12-12 16:39 ` Dave Hansen
2022-12-13 23:06 ` Dave Hansen
2022-12-15 17:12 ` Kirill A. Shutemov
2022-12-15 18:18 ` Dave Hansen
2022-12-15 18:51 ` Kirill A. Shutemov
2022-12-15 21:09 ` Dave Hansen
2022-12-16 2:38 ` Kirill A. Shutemov
2022-12-16 15:22 ` Reshetova, Elena
2022-12-09 13:25 ` [PATCH 3/4] x86/tdx: Relax SEPT_VE_DISABLE check for debug TD Kirill A. Shutemov
2022-12-09 15:45 ` Sathyanarayanan Kuppuswamy
2022-12-09 17:08 ` Kirill A. Shutemov
2022-12-13 23:13 ` Dave Hansen
2022-12-15 15:40 ` Kirill A. Shutemov
2022-12-09 13:25 ` [PATCH 4/4] x86/tdx: Disable NOTIFY_ENABLES Kirill A. Shutemov
2022-12-09 15:50 ` Sathyanarayanan Kuppuswamy
2022-12-09 17:10 ` Kirill A. Shutemov
2022-12-13 23:17 ` Dave Hansen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e13331b2-a68c-c823-d02f-ba50949214cb@intel.com \
--to=dave.hansen@intel.com \
--cc=bp@alien8.de \
--cc=elena.reshetova@intel.com \
--cc=kirill.shutemov@linux.intel.com \
--cc=kirill@shutemov.name \
--cc=linux-coco@lists.linux.dev \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@kernel.org \
--cc=sathyanarayanan.kuppuswamy@linux.intel.com \
--cc=tglx@linutronix.de \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.