Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-16 Thread Julien Grall
Hi Mirela, On 16/04/18 15:06, Mirela Simonovic wrote: On Mon, Apr 16, 2018 at 1:33 PM, Julien Grall wrote: On 13/04/18 11:19, Mirela Simonovic wrote: On Thu, Apr 12, 2018 at 10:43 AM, Julien Grall On 11/04/18 17:37, Mirela Simonovic wrote: On Wed, Apr 11, 2018 at 6:02 PM, Julien Grall mail

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-16 Thread Mirela Simonovic
Hi Julien, On Mon, Apr 16, 2018 at 1:33 PM, Julien Grall wrote: > Hi, > > > On 13/04/18 11:19, Mirela Simonovic wrote: >> >> On Thu, Apr 12, 2018 at 10:43 AM, Julien Grall >> wrote: >>> >>> >>> >>> On 11/04/18 17:37, Mirela Simonovic wrote: Hi Julien, >>> >>> >>> >>> Hi, >>> >>> M

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-16 Thread Julien Grall
Hi, On 13/04/18 11:19, Mirela Simonovic wrote: On Thu, Apr 12, 2018 at 10:43 AM, Julien Grall wrote: On 11/04/18 17:37, Mirela Simonovic wrote: Hi Julien, Hi, May I ask you to configure your mail client to use > for quoting and use plain text? Otherwise, this is going to be really diff

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-13 Thread Mirela Simonovic
Hi Julien, On Thu, Apr 12, 2018 at 10:43 AM, Julien Grall wrote: > > > On 11/04/18 17:37, Mirela Simonovic wrote: >> >> Hi Julien, > > > Hi, > > May I ask you to configure your mail client to use > for quoting and use > plain text? Otherwise, this is going to be really difficult to follow the > d

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-12 Thread Julien Grall
On 11/04/18 17:37, Mirela Simonovic wrote: Hi Julien, Hi, May I ask you to configure your mail client to use > for quoting and use plain text? Otherwise, this is going to be really difficult to follow the discussion after few round (see already below). On Wed, Apr 11, 2018 at 6:02 PM, Ju

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Mirela Simonovic
Hi Julien, On Wed, Apr 11, 2018 at 6:02 PM, Julien Grall wrote: > Hi, > > On 11/04/18 16:58, Mirela Simonovic wrote: > >> On 04/11/2018 05:07 PM, Julien Grall wrote: >> >>> On 11/04/18 14:19, Mirela Simonovic wrote: >>> >> Migrating interrupts when turning off a CPU already works. However, when

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Julien Grall
Hi, On 11/04/18 16:58, Mirela Simonovic wrote: On 04/11/2018 05:07 PM, Julien Grall wrote: On 11/04/18 14:19, Mirela Simonovic wrote: Migrating interrupts when turning off a CPU already works. However, when a CPU is turned back on there is no interrupt migration back to the hotplugged CPU - a

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Mirela Simonovic
Hi Julien, Thank you for the feedback. On 04/11/2018 05:07 PM, Julien Grall wrote: Hi Mirela, Thank you for sending the series. On 11/04/18 14:19, Mirela Simonovic wrote: This patch set contains fixes required to enable CPU hotplug for secondary CPUs. CPU hotplug of secondary CPUs will be

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Julien Grall
Hi, I forgot to mention that the title of most of the patch gives the impression you fixes hotplug for both Arm32 and Arm64. After a deeper look, it is only arm64. Please make clear over commit message and cover letter. Cheers, On 11/04/18 14:19, Mirela Simonovic wrote: This patch set cont

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Julien Grall
Hi Mirela, Thank you for sending the series. On 11/04/18 14:19, Mirela Simonovic wrote: This patch set contains fixes required to enable CPU hotplug for secondary CPUs. CPU hotplug of secondary CPUs will be used for suspend to RAM support for ARM. With these patches calling disable_nonboot_cpu

[Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Mirela Simonovic
This patch set contains fixes required to enable CPU hotplug for secondary CPUs. CPU hotplug of secondary CPUs will be used for suspend to RAM support for ARM. With these patches calling disable_nonboot_cpus() from the boot CPU will cause all secondary CPUs to be stopped. When a CPU is stopped it