From: Dario Faggioli <dfaggioli@suse.com>
To: Mirela Simonovic <mirela.simonovic@aggios.com>, xen-devel@lists.xen.org
Cc: edgar.iglesias@xilinx.com,
George Dunlap <george.dunlap@eu.citrix.com>,
julien.grall@arm.com, sstabellini@kernel.org, dm@aggios.com
Subject: Re: [PATCH v3 06/10] xen/common: Restore IRQ affinity when hotplugging a pCPU
Date: Thu, 10 May 2018 09:33:18 +0200 [thread overview]
Message-ID: <ec82f39caaae39ba9530cf41e373bb6f1546adb5.camel@suse.com> (raw)
In-Reply-To: <20180427171258.28852-7-mirela.simonovic@aggios.com>
[-- Attachment #1.1: Type: text/plain, Size: 1436 bytes --]
On Fri, 2018-04-27 at 19:12 +0200, Mirela Simonovic wrote:
> Non-boot pCPUs are being hot-unplugged during the system suspend to
> RAM and hotplugged during the resume. When non-boot pCPUs are
> hot-unplugged the interrupts that were targeted to them are migrated
> to the boot pCPU.
> On suspend, each guest could have its own wake-up devices/interrupts
> (passthrough) that could trigger the system resume. These interrupts
> could be targeted to a non-boot pCPU, e.g. if the guest's vCPU is
> pinned to a non-boot pCPU. Due to the hot-unplug of non-boot pCPUs
> during the suspend such interrupts will be migrated from non-boot
> pCPUs
> to the boot pCPU (this is fine). However, when non-boot pCPUs are
> hotplugged on resume, these interrupts are not migrated back to non-
> boot
> pCPUs, i.e. IRQ affinity is not restored on resume (this is wrong).
> This patch adds the restoration of IRQ affinity when a pCPU is
> hotplugged.
>
> Signed-off-by: Mirela Simonovic <mirela.simonovic@aggios.com>
>
> ---
> CC: George Dunlap <george.dunlap@eu.citrix.com>
> CC: Dario Faggioli <dfaggioli@suse.com>
>
Reviewed-by: Dario Faggioli <dfaggioli@suse.com>
Regards,
Dario
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Software Engineer @ SUSE https://www.suse.com/
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 157 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
next prev parent reply other threads:[~2018-05-10 7:33 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-27 17:12 [PATCH v3 00/10] xen/arm64: Suspend preconditions and CPU hotplug fixes Mirela Simonovic
2018-04-27 17:12 ` [PATCH v3 01/10] xen/arm64: Added handling of the trapped access to OSLSR register Mirela Simonovic
2018-04-27 17:12 ` [PATCH v3 02/10] xen/arm: Ignore write to GICD_ISACTIVERn registers (vgic-v2) Mirela Simonovic
2018-04-30 14:32 ` Julien Grall
2018-04-27 17:12 ` [PATCH v3 03/10] xen/arm: Implement CPU_OFF PSCI call (physical interface) Mirela Simonovic
2018-04-30 14:36 ` Julien Grall
2018-04-27 17:12 ` [PATCH v3 04/10] xen/arm: Remove __initdata and __init to enable CPU hotplug Mirela Simonovic
2018-04-27 17:12 ` [PATCH v3 05/10] xen/arm: Setup virtual paging for non-boot CPUs on hotplug/resume Mirela Simonovic
2018-04-30 14:47 ` Julien Grall
2018-05-07 14:55 ` Mirela Simonovic
2018-05-08 14:14 ` Julien Grall
2018-05-08 14:28 ` Mirela Simonovic
2018-05-09 10:32 ` Julien Grall
2018-04-27 17:12 ` [PATCH v3 06/10] xen/common: Restore IRQ affinity when hotplugging a pCPU Mirela Simonovic
2018-05-10 7:33 ` Dario Faggioli [this message]
2018-04-27 17:12 ` [PATCH v3 07/10] xen/arm: Release maintenance interrupt when CPU is hot-unplugged Mirela Simonovic
2018-04-30 14:51 ` Julien Grall
2018-04-27 17:12 ` [PATCH v3 08/10] xen/arm: Release timer interrupts " Mirela Simonovic
2018-04-30 15:58 ` Julien Grall
2018-05-07 15:33 ` Mirela Simonovic
2018-04-27 17:12 ` [PATCH v3 09/10] xen/arm: Free memory allocated for sibling/core maps on CPU hot-unplug Mirela Simonovic
2018-04-30 15:55 ` Julien Grall
2018-04-27 17:12 ` [PATCH v3 10/10] xen/arm: Enable errata for secondary CPU on hotplug after the boot Mirela Simonovic
2018-04-30 16:09 ` Julien Grall
2018-05-09 15:48 ` Mirela Simonovic
2018-05-09 16:32 ` Julien Grall
2018-05-10 11:57 ` Mirela Simonovic
2018-05-10 13:24 ` Mirela Simonovic
2018-05-10 13:29 ` Julien Grall
2018-05-10 14:12 ` Mirela Simonovic
2018-05-10 14:35 ` Julien Grall
2018-05-10 14:25 ` Dario Faggioli
2018-05-10 15:00 ` Mirela Simonovic
2018-05-10 15:13 ` Julien Grall
2018-05-10 15:49 ` Mirela Simonovic
2018-05-10 16:02 ` Julien Grall
2018-05-10 16:21 ` Dario Faggioli
2018-05-10 16:24 ` Dario Faggioli
2018-05-11 10:41 ` Mirela Simonovic
2018-05-11 10:54 ` Julien Grall
2018-05-11 12:07 ` Mirela Simonovic
2018-05-11 12:20 ` Mirela Simonovic
2018-05-11 13:08 ` Julien Grall
2018-05-11 13:31 ` Dario Faggioli
[not found] ` <CAKPH-Nj2znmdcjZEfFf83WmrcBS_u4R33yPOxAPWw37RHVZ38g@mail.gmail.com>
2018-05-11 14:14 ` Dario Faggioli
2018-05-11 21:47 ` Stefano Stabellini
2018-05-14 9:44 ` Julien Grall
2018-05-14 16:59 ` Stefano Stabellini
2018-05-15 11:45 ` Mirela Simonovic
2018-05-11 13:12 ` Dario Faggioli
2018-05-11 13:01 ` Dario Faggioli
2018-05-10 16:12 ` Dario Faggioli
2018-05-10 13:33 ` Dario Faggioli
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=ec82f39caaae39ba9530cf41e373bb6f1546adb5.camel@suse.com \
--to=dfaggioli@suse.com \
--cc=dm@aggios.com \
--cc=edgar.iglesias@xilinx.com \
--cc=george.dunlap@eu.citrix.com \
--cc=julien.grall@arm.com \
--cc=mirela.simonovic@aggios.com \
--cc=sstabellini@kernel.org \
--cc=xen-devel@lists.xen.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).