diff options
author | Andrzej Hajda <a.hajda@samsung.com> | 2015-09-24 16:00:23 +0200 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2015-10-15 15:58:16 +1100 |
commit | d4cd4f9586f87a5fc828b4c4698aa4faf56c96fc (patch) | |
tree | 15bf264ceb150cc65c10059640bb38e74cf1793f /arch/powerpc/kvm | |
parent | 6a14c2222419daa7ecebc7f566841b9b4d63b397 (diff) | |
download | op-kernel-dev-d4cd4f9586f87a5fc828b4c4698aa4faf56c96fc.zip op-kernel-dev-d4cd4f9586f87a5fc828b4c4698aa4faf56c96fc.tar.gz |
KVM: PPC: e500: fix handling local_sid_lookup result
The function can return negative value.
The problem has been detected using proposed semantic patch
scripts/coccinelle/tests/assign_signed_to_unsigned.cocci [1].
[1]: http://permalink.gmane.org/gmane.linux.kernel/2046107
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Acked-by: Scott Wood <scottwood@freescale.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'arch/powerpc/kvm')
-rw-r--r-- | arch/powerpc/kvm/e500.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/powerpc/kvm/e500.c b/arch/powerpc/kvm/e500.c index b29ce75..32fdab5 100644 --- a/arch/powerpc/kvm/e500.c +++ b/arch/powerpc/kvm/e500.c @@ -237,7 +237,8 @@ void kvmppc_e500_tlbil_one(struct kvmppc_vcpu_e500 *vcpu_e500, struct kvm_book3e_206_tlb_entry *gtlbe) { struct vcpu_id_table *idt = vcpu_e500->idt; - unsigned int pr, tid, ts, pid; + unsigned int pr, tid, ts; + int pid; u32 val, eaddr; unsigned long flags; |