cpumask: prepare for iterators to only go to nr_cpu_ids/nr_cpumask_bits: net
In future all cpumask ops will only be valid (in general) for bit numbers < nr_cpu_ids. So use that instead of NR_CPUS in iterators and other comparisons. This is always safe: no cpu number can be >= nr_cpu_ids, and nr_cpu_ids is initialized to NR_CPUS at boot. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> Signed-off-by: Mike Travis <travis@sgi.com> Acked-by: Ingo Molnar <mingo@elte.hu> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									f201a8a451
								
							
						
					
					
						commit
						0f23174aa8
					
				| @ -2414,7 +2414,7 @@ static void *neigh_stat_seq_start(struct seq_file *seq, loff_t *pos) | ||||
| 	if (*pos == 0) | ||||
| 		return SEQ_START_TOKEN; | ||||
| 
 | ||||
| 	for (cpu = *pos-1; cpu < NR_CPUS; ++cpu) { | ||||
| 	for (cpu = *pos-1; cpu < nr_cpu_ids; ++cpu) { | ||||
| 		if (!cpu_possible(cpu)) | ||||
| 			continue; | ||||
| 		*pos = cpu+1; | ||||
| @ -2429,7 +2429,7 @@ static void *neigh_stat_seq_next(struct seq_file *seq, void *v, loff_t *pos) | ||||
| 	struct neigh_table *tbl = pde->data; | ||||
| 	int cpu; | ||||
| 
 | ||||
| 	for (cpu = *pos; cpu < NR_CPUS; ++cpu) { | ||||
| 	for (cpu = *pos; cpu < nr_cpu_ids; ++cpu) { | ||||
| 		if (!cpu_possible(cpu)) | ||||
| 			continue; | ||||
| 		*pos = cpu+1; | ||||
|  | ||||
| @ -291,7 +291,7 @@ static void *ct_cpu_seq_start(struct seq_file *seq, loff_t *pos) | ||||
| 	if (*pos == 0) | ||||
| 		return SEQ_START_TOKEN; | ||||
| 
 | ||||
| 	for (cpu = *pos-1; cpu < NR_CPUS; ++cpu) { | ||||
| 	for (cpu = *pos-1; cpu < nr_cpu_ids; ++cpu) { | ||||
| 		if (!cpu_possible(cpu)) | ||||
| 			continue; | ||||
| 		*pos = cpu+1; | ||||
| @ -306,7 +306,7 @@ static void *ct_cpu_seq_next(struct seq_file *seq, void *v, loff_t *pos) | ||||
| 	struct net *net = seq_file_net(seq); | ||||
| 	int cpu; | ||||
| 
 | ||||
| 	for (cpu = *pos; cpu < NR_CPUS; ++cpu) { | ||||
| 	for (cpu = *pos; cpu < nr_cpu_ids; ++cpu) { | ||||
| 		if (!cpu_possible(cpu)) | ||||
| 			continue; | ||||
| 		*pos = cpu+1; | ||||
|  | ||||
| @ -429,7 +429,7 @@ static void *rt_cpu_seq_start(struct seq_file *seq, loff_t *pos) | ||||
| 	if (*pos == 0) | ||||
| 		return SEQ_START_TOKEN; | ||||
| 
 | ||||
| 	for (cpu = *pos-1; cpu < NR_CPUS; ++cpu) { | ||||
| 	for (cpu = *pos-1; cpu < nr_cpu_ids; ++cpu) { | ||||
| 		if (!cpu_possible(cpu)) | ||||
| 			continue; | ||||
| 		*pos = cpu+1; | ||||
| @ -442,7 +442,7 @@ static void *rt_cpu_seq_next(struct seq_file *seq, void *v, loff_t *pos) | ||||
| { | ||||
| 	int cpu; | ||||
| 
 | ||||
| 	for (cpu = *pos; cpu < NR_CPUS; ++cpu) { | ||||
| 	for (cpu = *pos; cpu < nr_cpu_ids; ++cpu) { | ||||
| 		if (!cpu_possible(cpu)) | ||||
| 			continue; | ||||
| 		*pos = cpu+1; | ||||
|  | ||||
| @ -200,7 +200,7 @@ static void *ct_cpu_seq_start(struct seq_file *seq, loff_t *pos) | ||||
| 	if (*pos == 0) | ||||
| 		return SEQ_START_TOKEN; | ||||
| 
 | ||||
| 	for (cpu = *pos-1; cpu < NR_CPUS; ++cpu) { | ||||
| 	for (cpu = *pos-1; cpu < nr_cpu_ids; ++cpu) { | ||||
| 		if (!cpu_possible(cpu)) | ||||
| 			continue; | ||||
| 		*pos = cpu + 1; | ||||
| @ -215,7 +215,7 @@ static void *ct_cpu_seq_next(struct seq_file *seq, void *v, loff_t *pos) | ||||
| 	struct net *net = seq_file_net(seq); | ||||
| 	int cpu; | ||||
| 
 | ||||
| 	for (cpu = *pos; cpu < NR_CPUS; ++cpu) { | ||||
| 	for (cpu = *pos; cpu < nr_cpu_ids; ++cpu) { | ||||
| 		if (!cpu_possible(cpu)) | ||||
| 			continue; | ||||
| 		*pos = cpu + 1; | ||||
|  | ||||
		Loading…
	
		Reference in New Issue
	
	Block a user