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