All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andi Kleen <ak@linux.intel.com>
To: "Kuppuswamy,
	Sathyanarayanan"  <sathyanarayanan.kuppuswamy@linux.intel.com>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
	Rafael J Wysocki <rjw@rjwysocki.net>,
	Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>, "H . Peter Anvin" <hpa@zytor.com>,
	Peter Zijlstra <peterz@infradead.org>,
	Len Brown <lenb@kernel.org>,
	Robert Moore <robert.moore@intel.com>,
	Erik Kaneda <erik.kaneda@intel.com>,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
	"open list:ACPI COMPONENT ARCHITECTURE (ACPICA)"
	<devel@acpica.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	the arch/x86 maintainers <x86@kernel.org>,
	Sean Christopherson <sean.j.christopherson@intel.com>
Subject: Re: [PATCH v3 3/3] x86/acpi, x86/boot: Add multiprocessor wake-up support
Date: Mon, 10 May 2021 10:22:37 -0700	[thread overview]
Message-ID: <20210510172237.GU4032392@tassilo.jf.intel.com> (raw)
In-Reply-To: <4fa40e7a-bcb2-db0f-8dc5-28728b14377d@linux.intel.com>

On Mon, May 10, 2021 at 10:10:24AM -0700, Kuppuswamy, Sathyanarayanan wrote:
> 
> 
> On 5/10/21 9:55 AM, Rafael J. Wysocki wrote:
> > I'm not sure how my comment regarding the fact that for a given CPU
> > this function is only usable once has been addressed.
> > 
> > While it may not be a practical concern in the use case that you are
> > after (TDX), this is a generic mechanism and it needs to cover other
> > possible usage scenarios.
> 
> For the same CPU, if we try to use mailbox again, firmware will not
> respond to it. So the command will timeout and return error.

Right because the firmware code doesn't run anymore.

The only possibility would be for Linux to put back some code that spins
and waits again, but that would be quite pointless and wasteful.

-Andi

  reply	other threads:[~2021-05-10 17:22 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-26  2:39 [PATCH v3 0/3] Add multiprocessor wake-up support Kuppuswamy Sathyanarayanan
2021-04-26  2:39 ` [PATCH v3 1/3] ACPICA: ACPI 6.4: MADT: add Multiprocessor Wakeup Mailbox Structure Kuppuswamy Sathyanarayanan
2021-04-26  2:39 ` [PATCH v3 2/3] ACPI/table: Print MADT Wake table information Kuppuswamy Sathyanarayanan
2021-04-26  2:39 ` [PATCH v3 3/3] x86/acpi, x86/boot: Add multiprocessor wake-up support Kuppuswamy Sathyanarayanan
2021-05-10 16:32   ` Kuppuswamy, Sathyanarayanan
2021-05-10 16:55     ` Rafael J. Wysocki
2021-05-10 16:55       ` [Devel] " Rafael J. Wysocki
2021-05-10 17:10       ` Kuppuswamy, Sathyanarayanan
2021-05-10 17:22         ` Andi Kleen [this message]
2021-05-10 17:24           ` Rafael J. Wysocki
2021-05-10 17:24             ` [Devel] " Rafael J. Wysocki
2021-05-10 21:15             ` Kuppuswamy, Sathyanarayanan
2021-05-11 10:45               ` Rafael J. Wysocki
2021-05-11 10:45                 ` [Devel] " Rafael J. Wysocki
2021-05-13 21:37                 ` [PATCH v4 1/1] " Kuppuswamy Sathyanarayanan
2021-05-21 14:18                   ` Kuppuswamy, Sathyanarayanan
2021-05-21 14:45                   ` Peter Zijlstra
2021-05-21 15:14                     ` Kuppuswamy, Sathyanarayanan
2021-05-24  6:02                       ` [PATCH v5 " Kuppuswamy Sathyanarayanan
2021-05-24  6:40                         ` Mika Penttilä
2021-05-24 13:42                           ` Kuppuswamy, Sathyanarayanan
2021-05-24 14:51                         ` Rafael J. Wysocki
2021-05-24 15:35                           ` Kuppuswamy, Sathyanarayanan
2021-05-10 17:23         ` [PATCH v3 3/3] " Rafael J. Wysocki
2021-05-10 17:23           ` [Devel] " Rafael J. Wysocki

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=20210510172237.GU4032392@tassilo.jf.intel.com \
    --to=ak@linux.intel.com \
    --cc=devel@acpica.org \
    --cc=erik.kaneda@intel.com \
    --cc=hpa@zytor.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=peterz@infradead.org \
    --cc=rafael@kernel.org \
    --cc=rjw@rjwysocki.net \
    --cc=robert.moore@intel.com \
    --cc=sathyanarayanan.kuppuswamy@linux.intel.com \
    --cc=sean.j.christopherson@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.