Re: [PATCH -next] kvm/ppc/mpic: fix missing unlock in set_base_addr()

2013-05-02 Thread Alexander Graf

On 02.05.2013, at 07:17, Wei Yongjun wrote:

 From: Wei Yongjun yongjun_...@trendmicro.com.cn
 
 Add the missing unlock before return from function set_base_addr()
 when disables the mapping.
 
 Introduced by commit 5df554ad5b7522ea62b0ff9d5be35183494efc21
 (kvm/ppc/mpic: in-kernel MPIC emulation)
 
 Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn

Thanks, applied to kvm-ppc-queue.


Alex

 ---
 arch/powerpc/kvm/mpic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/arch/powerpc/kvm/mpic.c b/arch/powerpc/kvm/mpic.c
 index f3148f8..0047a70 100644
 --- a/arch/powerpc/kvm/mpic.c
 +++ b/arch/powerpc/kvm/mpic.c
 @@ -1475,8 +1475,8 @@ static int set_base_addr(struct openpic *opp, struct 
 kvm_device_attr *attr)
 
   map_mmio(opp);
 
 - mutex_unlock(opp-kvm-slots_lock);
 out:
 + mutex_unlock(opp-kvm-slots_lock);
   return 0;
 }
 
 

--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH -next] kvm/ppc/mpic: fix missing unlock in set_base_addr()

2013-05-02 Thread Alexander Graf

On 02.05.2013, at 07:17, Wei Yongjun wrote:

 From: Wei Yongjun yongjun_...@trendmicro.com.cn
 
 Add the missing unlock before return from function set_base_addr()
 when disables the mapping.
 
 Introduced by commit 5df554ad5b7522ea62b0ff9d5be35183494efc21
 (kvm/ppc/mpic: in-kernel MPIC emulation)
 
 Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn

Thanks, applied to kvm-ppc-queue.


Alex

 ---
 arch/powerpc/kvm/mpic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/arch/powerpc/kvm/mpic.c b/arch/powerpc/kvm/mpic.c
 index f3148f8..0047a70 100644
 --- a/arch/powerpc/kvm/mpic.c
 +++ b/arch/powerpc/kvm/mpic.c
 @@ -1475,8 +1475,8 @@ static int set_base_addr(struct openpic *opp, struct 
 kvm_device_attr *attr)
 
   map_mmio(opp);
 
 - mutex_unlock(opp-kvm-slots_lock);
 out:
 + mutex_unlock(opp-kvm-slots_lock);
   return 0;
 }
 
 

--
To unsubscribe from this list: send the line unsubscribe kvm-ppc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH -next] kvm/ppc/mpic: fix missing unlock in set_base_addr()

2013-05-01 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn

Add the missing unlock before return from function set_base_addr()
when disables the mapping.

Introduced by commit 5df554ad5b7522ea62b0ff9d5be35183494efc21
(kvm/ppc/mpic: in-kernel MPIC emulation)

Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
 arch/powerpc/kvm/mpic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/kvm/mpic.c b/arch/powerpc/kvm/mpic.c
index f3148f8..0047a70 100644
--- a/arch/powerpc/kvm/mpic.c
+++ b/arch/powerpc/kvm/mpic.c
@@ -1475,8 +1475,8 @@ static int set_base_addr(struct openpic *opp, struct 
kvm_device_attr *attr)
 
map_mmio(opp);
 
-   mutex_unlock(opp-kvm-slots_lock);
 out:
+   mutex_unlock(opp-kvm-slots_lock);
return 0;
 }
 

--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH -next] kvm/ppc/mpic: fix missing unlock in set_base_addr()

2013-05-01 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn

Add the missing unlock before return from function set_base_addr()
when disables the mapping.

Introduced by commit 5df554ad5b7522ea62b0ff9d5be35183494efc21
(kvm/ppc/mpic: in-kernel MPIC emulation)

Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
 arch/powerpc/kvm/mpic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/powerpc/kvm/mpic.c b/arch/powerpc/kvm/mpic.c
index f3148f8..0047a70 100644
--- a/arch/powerpc/kvm/mpic.c
+++ b/arch/powerpc/kvm/mpic.c
@@ -1475,8 +1475,8 @@ static int set_base_addr(struct openpic *opp, struct 
kvm_device_attr *attr)
 
map_mmio(opp);
 
-   mutex_unlock(opp-kvm-slots_lock);
 out:
+   mutex_unlock(opp-kvm-slots_lock);
return 0;
 }
 

--
To unsubscribe from this list: send the line unsubscribe kvm-ppc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html