On Fri, Jul 22, 2016 at 04:31:13PM +0100, Julien Grall wrote: > Hi Konrad, > > On 22/07/16 15:18, Konrad Rzeszutek Wilk wrote: > >On Wed, Jul 20, 2016 at 04:25:55PM +0100, Julien Grall wrote: > >>diff --git a/xen/arch/arm/cpufeature.c b/xen/arch/arm/cpufeature.c > >>index 7a1b56b..088625b 100644 > >>--- a/xen/arch/arm/cpufeature.c > >>+++ b/xen/arch/arm/cpufeature.c > >>@@ -24,6 +24,22 @@ > >> > >> DECLARE_BITMAP(cpu_hwcaps, ARM_NCAPS); > >> > >>+void update_cpu_capabilities(const struct arm_cpu_capabilities *caps, > >>+ const char *info) > >>+{ > >>+ int i; > >>+ > >>+ for ( i = 0; caps[i].matches; i++ ) > >>+ { > >>+ if ( !caps[i].matches(&caps[i]) ) > > > >So what if the 'struct arm_cpu_capabilitues' has '->matches' set to > >NULL? > > It is the exit condition of the loop: > > for ( i = 0; caps[i].matches; i++ )
Duh! Ignore my query then please. > > Regards, > > -- > Julien Grall _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel