Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-15 Thread Julien Grall
On 11/15/18 6:57 PM, Stefano Stabellini wrote: On Wed, 14 Nov 2018, Julien Grall wrote: On 14/11/2018 23:04, Stefano Stabellini wrote: On Wed, 14 Nov 2018, Julien Grall wrote:   -    return -ENOSYS; Why do you return -ENOSYS until this patch? Should not have it be 0? To distinguish

Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-15 Thread Stefano Stabellini
On Wed, 14 Nov 2018, Julien Grall wrote: > On 14/11/2018 23:04, Stefano Stabellini wrote: > > On Wed, 14 Nov 2018, Julien Grall wrote: > >> Hi Mirela, > >> > >> On 14/11/2018 13:00, Mirela Simonovic wrote: > >>> > >>> > >>> On 11/14/2018 11:52 AM, Julien Grall wrote: > Hi Mirela, > >

Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-14 Thread Julien Grall
On 14/11/2018 23:04, Stefano Stabellini wrote: > On Wed, 14 Nov 2018, Julien Grall wrote: >> Hi Mirela, >> >> On 14/11/2018 13:00, Mirela Simonovic wrote: >>> >>> >>> On 11/14/2018 11:52 AM, Julien Grall wrote: Hi Mirela, On 12/11/2018 11:30, Mirela Simonovic wrote: > Non-boot

Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-14 Thread Stefano Stabellini
On Wed, 14 Nov 2018, Julien Grall wrote: > Hi Mirela, > > On 14/11/2018 13:00, Mirela Simonovic wrote: > > > > > > On 11/14/2018 11:52 AM, Julien Grall wrote: > > > Hi Mirela, > > > > > > On 12/11/2018 11:30, Mirela Simonovic wrote: > > > > Non-boot CPUs have to be disabled on suspend and

Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-14 Thread Julien Grall
Hi Mirela, On 14/11/2018 13:00, Mirela Simonovic wrote: On 11/14/2018 11:52 AM, Julien Grall wrote: Hi Mirela, On 12/11/2018 11:30, Mirela Simonovic wrote: Non-boot CPUs have to be disabled on suspend and enabled on resume (hotplug-based mechanism). Disabling non-boot CPUs will lead to

Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-14 Thread Mirela Simonovic
On 11/14/2018 11:52 AM, Julien Grall wrote: Hi Mirela, On 12/11/2018 11:30, Mirela Simonovic wrote: Non-boot CPUs have to be disabled on suspend and enabled on resume (hotplug-based mechanism). Disabling non-boot CPUs will lead to PSCI CPU_OFF to be called by each non-boot CPU. Depending on

Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-14 Thread Julien Grall
Hi, On 13/11/2018 22:35, Stefano Stabellini wrote: On Mon, 12 Nov 2018, Mirela Simonovic wrote: Non-boot CPUs have to be disabled on suspend and enabled on resume (hotplug-based mechanism). Disabling non-boot CPUs will lead to PSCI CPU_OFF to be called by each non-boot CPU. Depending on the

Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-14 Thread Julien Grall
Hi Mirela, On 12/11/2018 11:30, Mirela Simonovic wrote: Non-boot CPUs have to be disabled on suspend and enabled on resume (hotplug-based mechanism). Disabling non-boot CPUs will lead to PSCI CPU_OFF to be called by each non-boot CPU. Depending on the underlying platform capabilities, this may

Re: [Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-13 Thread Stefano Stabellini
On Mon, 12 Nov 2018, Mirela Simonovic wrote: > Non-boot CPUs have to be disabled on suspend and enabled on resume > (hotplug-based mechanism). Disabling non-boot CPUs will lead to PSCI > CPU_OFF to be called by each non-boot CPU. Depending on the underlying > platform capabilities, this may lead

[Xen-devel] [PATCH 08/18] xen/arm: Disable/enable non-boot physical CPUs on suspend/resume

2018-11-12 Thread Mirela Simonovic
Non-boot CPUs have to be disabled on suspend and enabled on resume (hotplug-based mechanism). Disabling non-boot CPUs will lead to PSCI CPU_OFF to be called by each non-boot CPU. Depending on the underlying platform capabilities, this may lead to the physical powering down of CPUs. Tested on