If it isn't NR_CPUS we're pretty broken if we set it or if we don't. My previous version did a BUILD_BUG_ON() but Ben didn't like that and said he preferred just a warning message. _______________________________________________+ if(*cores != NR_CPUS) + prom_printf("client-architecture structure corrupted\n"); + *cores = (NR_CPUS / prom_smt_way()); + prom_printf("setting client-architecture cores to %x\n", *cores);I don't know if I'm painting a bike shed of if this is a real concern, but if *cores isn't NR_CPUS shouldn't we do nothing rather then clobbering it? Yours Tony
Linuxppc-dev mailing list [email protected] https://lists.ozlabs.org/listinfo/linuxppc-dev
