[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID:
<VI1PR02MB39529C3384B3308D100CCD399CF5A@VI1PR02MB3952.eurprd02.prod.outlook.com>
Date: Mon, 20 Oct 2025 11:12:33 +0000
From: David Binderman <dcb314@...mail.com>
To: "maz@...nel.org" <maz@...nel.org>, "oliver.upton@...ux.dev"
<oliver.upton@...ux.dev>, "joey.gouly@....com" <joey.gouly@....com>,
"suzuki.poulose@....com" <suzuki.poulose@....com>, "yuzenghui@...wei.com"
<yuzenghui@...wei.com>, "catalin.marinas@....com" <catalin.marinas@....com>,
"will@...nel.org" <will@...nel.org>, "linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>, "kvmarm@...ts.linux.dev"
<kvmarm@...ts.linux.dev>, LKML <linux-kernel@...r.kernel.org>
Subject: linux-6.18-rc2/arch/arm64/kvm/vgic/vgic-v3.c:728: Possible || and |
mixup ?
Hello there,
Static analyser cppcheck said:
linux-6.18-rc2/arch/arm64/kvm/vgic/vgic-v3.c:728:48: style: inconclusive: Boolean expression 'common_trap' is used in bitwise operation. Did you mean '||'? [bitwiseOnBoolean]
Source code is
if (group0_trap || group1_trap || common_trap | dir_trap) {
Maybe better code:
if (group0_trap || group1_trap || common_trap || dir_trap) {
Regards
David Binderman
Powered by blists - more mailing lists