From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexander Graf Date: Thu, 14 Jul 2016 11:10:13 +0000 Subject: Re: [patch] KVM: PPC: fix a sanity check Message-Id: <57877315.5060903@suse.de> List-Id: References: <20160714101545.GB18175@mwanda> In-Reply-To: <20160714101545.GB18175@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter , Bharat Bhushan Cc: Michael Ellerman , Benjamin Herrenschmidt , linuxppc-dev@lists.ozlabs.org, Paolo Bonzini , rkrcmar@redhat.com, Paul Mackerras , kernel-janitors@vger.kernel.org, kvm@vger.kernel.org, kvm-ppc@vger.kernel.org On 07/14/2016 12:15 PM, Dan Carpenter wrote: > We use logical negate where bitwise negate was intended. It means that > we never return -EINVAL here. > > Fixes: ce11e48b7fdd ('KVM: PPC: E500: Add userspace debug stub support') > Signed-off-by: Dan Carpenter Oops :). Reviewed-by: Alexander Graf Alex From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) (using TLSv1 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3rqtKD5hgFzDqxj for ; Thu, 14 Jul 2016 21:10:24 +1000 (AEST) Subject: Re: [patch] KVM: PPC: fix a sanity check To: Dan Carpenter , Bharat Bhushan References: <20160714101545.GB18175@mwanda> Cc: Michael Ellerman , Benjamin Herrenschmidt , linuxppc-dev@lists.ozlabs.org, Paolo Bonzini , rkrcmar@redhat.com, Paul Mackerras , kernel-janitors@vger.kernel.org, kvm@vger.kernel.org, kvm-ppc@vger.kernel.org From: Alexander Graf Message-ID: <57877315.5060903@suse.de> Date: Thu, 14 Jul 2016 13:10:13 +0200 MIME-Version: 1.0 In-Reply-To: <20160714101545.GB18175@mwanda> Content-Type: text/plain; charset=windows-1252; format=flowed List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 07/14/2016 12:15 PM, Dan Carpenter wrote: > We use logical negate where bitwise negate was intended. It means that > we never return -EINVAL here. > > Fixes: ce11e48b7fdd ('KVM: PPC: E500: Add userspace debug stub support') > Signed-off-by: Dan Carpenter Oops :). Reviewed-by: Alexander Graf Alex