Message ID | 20180116150430.rlbw6qxb7v465gcv@armageddon.cambridge.arm.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 16/01/18 15:04, Catalin Marinas wrote: > On Mon, Jan 15, 2018 at 07:38:54PM +0000, James Morse wrote: >> diff --git a/arch/arm64/kernel/cpufeature.c b/arch/arm64/kernel/cpufeature.c >> index 9ef84d0def9a..d88cd0e88606 100644 >> --- a/arch/arm64/kernel/cpufeature.c >> +++ b/arch/arm64/kernel/cpufeature.c >> @@ -1303,8 +1303,8 @@ static bool __this_cpu_has_cap(const struct arm64_cpu_capabilities *cap_array, >> if (WARN_ON(preemptible())) >> return false; >> >> - for (caps = cap_array; caps->desc; caps++) >> - if (caps->capability == cap && caps->matches) >> + for (caps = cap_array; caps->matches; caps++) >> + if (caps->capability == cap) >> return caps->matches(caps, SCOPE_LOCAL_CPU); >> >> return false; > > Just to make sure I applied this correctly on top of for-next/core: > > diff --git a/arch/arm64/kernel/cpufeature.c b/arch/arm64/kernel/cpufeature.c > index a11311397430..630a40ec1332 100644 > --- a/arch/arm64/kernel/cpufeature.c > +++ b/arch/arm64/kernel/cpufeature.c > @@ -1149,9 +1149,8 @@ static bool __this_cpu_has_cap(const struct arm64_cpu_capabilities *cap_array, > if (WARN_ON(preemptible())) > return false; > > - for (caps = cap_array; caps->desc; caps++) > + for (caps = cap_array; caps->matches; caps++) > if (caps->capability == cap && > - caps->matches && > caps->matches(caps, SCOPE_LOCAL_CPU)) > return true; > return false; > Looks correct to me. Thanks Suzuki
diff --git a/arch/arm64/kernel/cpufeature.c b/arch/arm64/kernel/cpufeature.c index a11311397430..630a40ec1332 100644 --- a/arch/arm64/kernel/cpufeature.c +++ b/arch/arm64/kernel/cpufeature.c @@ -1149,9 +1149,8 @@ static bool __this_cpu_has_cap(const struct arm64_cpu_capabilities *cap_array, if (WARN_ON(preemptible())) return false; - for (caps = cap_array; caps->desc; caps++) + for (caps = cap_array; caps->matches; caps++) if (caps->capability == cap && - caps->matches && caps->matches(caps, SCOPE_LOCAL_CPU)) return true; return false;