Re: [PATCH] PPC: Fix race in mtmsr paravirt implementation

2011-10-13 Thread Alexander Graf
Am 13.10.2011 um 07:40 schrieb Bharat Bhushan r65...@freescale.com: The current implementation of mtmsr and mtmsrd are racy in that it does: * check (int_pending == 0) --- host sets int_pending = 1 --- * write shared page * done while instead we should check for int_pending after

RE: [PATCH] PPC: Fix race in mtmsr paravirt implementation

2011-10-13 Thread Bhushan Bharat-R65777
-Original Message- From: Alexander Graf [mailto:ag...@suse.de] Sent: Thursday, October 13, 2011 2:36 PM To: Bhushan Bharat-R65777 Cc: kvm-ppc@vger.kernel.org; bharatb.ya...@gmail.com; Bhushan Bharat- R65777 Subject: Re: [PATCH] PPC: Fix race in mtmsr paravirt implementation

Re: [PATCH] PPC: Fix race in mtmsr paravirt implementation

2011-10-13 Thread Alexander Graf
On 10/13/2011 11:22 AM, Bhushan Bharat-R65777 wrote: -Original Message- From: Alexander Graf [mailto:ag...@suse.de] Sent: Thursday, October 13, 2011 2:36 PM To: Bhushan Bharat-R65777 Cc:kvm-ppc@vger.kernel.org;bharatb.ya...@gmail.com; Bhushan Bharat- R65777 Subject: Re: [PATCH] PPC:

[PATCH V2] PPC: Fix race in mtmsr paravirt implementation

2011-10-13 Thread Bharat Bhushan
The current implementation of mtmsr and mtmsrd are racy in that it does: * check (int_pending == 0) --- host sets int_pending = 1 --- * write shared page * done while instead we should check for int_pending after the shared page is written. Signed-off-by: Bharat Bhushan

Re: [PATCH V2] PPC: Fix race in mtmsr paravirt implementation

2011-10-13 Thread Alexander Graf
On 10/13/2011 11:47 AM, Bharat Bhushan wrote: The current implementation of mtmsr and mtmsrd are racy in that it does: * check (int_pending == 0) --- host sets int_pending = 1--- * write shared page * done while instead we should check for int_pending after the shared page is