Re: [Xen-devel] [PATCH 1/2] xen/sched: fix locking in a653sched_free_vdata()

2019-09-27 Thread Jürgen Groß
On 27.09.19 10:20, Jan Beulich wrote: On 27.09.2019 09:23, Jürgen Groß wrote: On 25.09.19 12:59, Dario Faggioli wrote: On Wed, 2019-09-25 at 09:05 +0200, Juergen Gross wrote: The arinc653 scheduler's free_vdata() function is missing proper locking: as it is modifying the scheduler's private v

Re: [Xen-devel] [PATCH 1/2] xen/sched: fix locking in a653sched_free_vdata()

2019-09-27 Thread Jan Beulich
On 27.09.2019 09:23, Jürgen Groß wrote: > On 25.09.19 12:59, Dario Faggioli wrote: >> On Wed, 2019-09-25 at 09:05 +0200, Juergen Gross wrote: >>> The arinc653 scheduler's free_vdata() function is missing proper >>> locking: as it is modifying the scheduler's private vcpu list it >>> needs >>> to t

Re: [Xen-devel] [PATCH 1/2] xen/sched: fix locking in a653sched_free_vdata()

2019-09-27 Thread Jürgen Groß
On 25.09.19 12:59, Dario Faggioli wrote: On Wed, 2019-09-25 at 09:05 +0200, Juergen Gross wrote: The arinc653 scheduler's free_vdata() function is missing proper locking: as it is modifying the scheduler's private vcpu list it needs to take the scheduler lock during that operation. Signed-off-b

Re: [Xen-devel] [PATCH 1/2] xen/sched: fix locking in a653sched_free_vdata()

2019-09-25 Thread Dario Faggioli
On Wed, 2019-09-25 at 09:05 +0200, Juergen Gross wrote: > The arinc653 scheduler's free_vdata() function is missing proper > locking: as it is modifying the scheduler's private vcpu list it > needs > to take the scheduler lock during that operation. > > Signed-off-by: Juergen Gross > Reviewed-by:

Re: [Xen-devel] [PATCH 1/2] xen/sched: fix locking in a653sched_free_vdata()

2019-09-25 Thread Jan Beulich
On 25.09.2019 09:05, Juergen Gross wrote: > The arinc653 scheduler's free_vdata() function is missing proper > locking: as it is modifying the scheduler's private vcpu list it needs > to take the scheduler lock during that operation. > > Signed-off-by: Juergen Gross Reviewed-by: Jan Beulich (an

[Xen-devel] [PATCH 1/2] xen/sched: fix locking in a653sched_free_vdata()

2019-09-25 Thread Juergen Gross
The arinc653 scheduler's free_vdata() function is missing proper locking: as it is modifying the scheduler's private vcpu list it needs to take the scheduler lock during that operation. Signed-off-by: Juergen Gross --- xen/common/sched_arinc653.c | 6 ++ 1 file changed, 6 insertions(+) diff