From: "Rafael J. Wysocki" <rafael@kernel.org> 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>, Andi Kleen <ak@linux.intel.com> Subject: Re: [PATCH v3 3/3] x86/acpi, x86/boot: Add multiprocessor wake-up support Date: Mon, 10 May 2021 19:23:02 +0200 [thread overview] Message-ID: <CAJZ5v0jMo9+ohUJ_=KHL_QwEo5GmAn_fMkng==Zmy-AJ=MPRog@mail.gmail.com> (raw) In-Reply-To: <4fa40e7a-bcb2-db0f-8dc5-28728b14377d@linux.intel.com> On Mon, May 10, 2021 at 7:10 PM Kuppuswamy, Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com> 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. Well, theoretically, but what if it checks the mailbox every time for all CPUs? Or similar? > So the command will timeout and return error. If the firmware behaves as expected, but what if it doesn't?
WARNING: multiple messages have this Message-ID (diff)
From: Rafael J. Wysocki <rafael at kernel.org> To: devel@acpica.org Subject: [Devel] Re: [PATCH v3 3/3] x86/acpi, x86/boot: Add multiprocessor wake-up support Date: Mon, 10 May 2021 19:23:02 +0200 [thread overview] Message-ID: <CAJZ5v0jMo9+ohUJ_=KHL_QwEo5GmAn_fMkng==Zmy-AJ=MPRog@mail.gmail.com> (raw) In-Reply-To: 4fa40e7a-bcb2-db0f-8dc5-28728b14377d@linux.intel.com [-- Attachment #1: Type: text/plain, Size: 791 bytes --] On Mon, May 10, 2021 at 7:10 PM Kuppuswamy, Sathyanarayanan <sathyanarayanan.kuppuswamy(a)linux.intel.com> 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. Well, theoretically, but what if it checks the mailbox every time for all CPUs? Or similar? > So the command will timeout and return error. If the firmware behaves as expected, but what if it doesn't?
next prev parent reply other threads:[~2021-05-10 17:23 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 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 ` Rafael J. Wysocki [this message] 2021-05-10 17:23 ` [Devel] Re: [PATCH v3 3/3] " 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='CAJZ5v0jMo9+ohUJ_=KHL_QwEo5GmAn_fMkng==Zmy-AJ=MPRog@mail.gmail.com' \ --to=rafael@kernel.org \ --cc=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=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: linkBe 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.