Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-19 Thread Julien Grall
On 18/04/18 23:52, Stefano Stabellini wrote: On Tue, 17 Apr 2018, Julien Grall wrote: On 17/04/18 11:52, Mirela Simonovic wrote: Hi Julien, Hi Mirela, On Mon, Apr 16, 2018 at 5:21 PM, Julien Grall wrote: On 16/04/18 14:41, Mirela Simonovic wrote: On Mon, Apr 16, 2018 at 3:14 PM, Jul

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-18 Thread Stefano Stabellini
On Tue, 17 Apr 2018, Julien Grall wrote: > On 17/04/18 11:52, Mirela Simonovic wrote: > > Hi Julien, > > Hi Mirela, > > > On Mon, Apr 16, 2018 at 5:21 PM, Julien Grall wrote: > > > > > > > > > On 16/04/18 14:41, Mirela Simonovic wrote: > > > > > > > > On Mon, Apr 16, 2018 at 3:14 PM, Julien G

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-17 Thread Julien Grall
On 17/04/18 11:52, Mirela Simonovic wrote: Hi Julien, Hi Mirela, On Mon, Apr 16, 2018 at 5:21 PM, Julien Grall wrote: On 16/04/18 14:41, Mirela Simonovic wrote: On Mon, Apr 16, 2018 at 3:14 PM, Julien Grall wrote: On 12/04/18 22:31, Stefano Stabellini wrote: On Thu, 12 Apr 2018,

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-17 Thread Mirela Simonovic
Hi Julien, On Mon, Apr 16, 2018 at 5:21 PM, Julien Grall wrote: > > > On 16/04/18 14:41, Mirela Simonovic wrote: >> >> On Mon, Apr 16, 2018 at 3:14 PM, Julien Grall >> wrote: >>> >>> On 12/04/18 22:31, Stefano Stabellini wrote: On Thu, 12 Apr 2018, Julien Grall wrote: > > On 12

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-16 Thread Julien Grall
On 16/04/18 14:41, Mirela Simonovic wrote: On Mon, Apr 16, 2018 at 3:14 PM, Julien Grall wrote: On 12/04/18 22:31, Stefano Stabellini wrote: On Thu, 12 Apr 2018, Julien Grall wrote: On 12/04/18 00:46, Stefano Stabellini wrote: On Wed, 11 Apr 2018, Julien Grall wrote: On 11/04/18 14:19, Mi

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-16 Thread Mirela Simonovic
Hi Julien, Stefano, Thanks for the feedback and suggestions. On Mon, Apr 16, 2018 at 3:14 PM, Julien Grall wrote: > Hi, > > > On 12/04/18 22:31, Stefano Stabellini wrote: >> >> On Thu, 12 Apr 2018, Julien Grall wrote: >>> >>> On 12/04/18 00:46, Stefano Stabellini wrote: On Wed, 11 Apr

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-16 Thread Julien Grall
Hi, On 12/04/18 22:31, Stefano Stabellini wrote: On Thu, 12 Apr 2018, Julien Grall wrote: On 12/04/18 00:46, Stefano Stabellini wrote: On Wed, 11 Apr 2018, Julien Grall wrote: On 11/04/18 14:19, Mirela Simonovic wrote: Freeing percpu area is done when a non-boot CPU is disabled upon suspend.

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-12 Thread Stefano Stabellini
On Thu, 12 Apr 2018, Julien Grall wrote: > On 12/04/18 00:46, Stefano Stabellini wrote: > > On Wed, 11 Apr 2018, Julien Grall wrote: > > > On 11/04/18 14:19, Mirela Simonovic wrote: > > > > Freeing percpu area is done when a non-boot CPU is disabled upon > > > > suspend. > > > > This use to be sche

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-12 Thread Julien Grall
On 12/04/18 00:46, Stefano Stabellini wrote: On Wed, 11 Apr 2018, Julien Grall wrote: On 11/04/18 14:19, Mirela Simonovic wrote: Freeing percpu area is done when a non-boot CPU is disabled upon suspend. This use to be scheduled for execution after a period of time, what caused the following r

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-11 Thread Stefano Stabellini
On Wed, 11 Apr 2018, Julien Grall wrote: > On 11/04/18 14:19, Mirela Simonovic wrote: > > Freeing percpu area is done when a non-boot CPU is disabled upon suspend. > > This use to be scheduled for execution after a period of time, what caused > > the following racing issues. If CPU is enabled after

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-11 Thread Julien Grall
Hi, On 11/04/18 14:19, Mirela Simonovic wrote: Freeing percpu area is done when a non-boot CPU is disabled upon suspend. This use to be scheduled for execution after a period of time, what caused the following racing issues. If CPU is enabled after it is disabled and before the freeing of percpu

[Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-11 Thread Mirela Simonovic
Freeing percpu area is done when a non-boot CPU is disabled upon suspend. This use to be scheduled for execution after a period of time, what caused the following racing issues. If CPU is enabled after it is disabled and before the freeing of percpu area is performed, Xen would crash upon initializ