On some platforms, the /cpus entry contains cpu-to-cluster
map which deffinitely is not a CPU device. Its presence was
causing incrementing of "id" variable and reporting more
CPUs available than it should.
To make "id" valid, increment it only when an entry really
is a CPU.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
I'm not sure we need a goto in the loop, finding the next id is simple to do when we know we have a cpu node.
sys/dev/ofw/ofw_cpu.c | ||
---|---|---|
300 | u_int next_id; ... next_id = 0; ... for (child = OF_child(node); child != 0; child = OF_peer(child), id = next_id) { | |
310 | next_id++; | |
319 | Then revert this | |
329 | And this | |
333 | etc. |
Comment Actions
Just need to fix the minor style issue before committing.
sys/dev/ofw/ofw_cpu.c | ||
---|---|---|
300–310 | This line will need to be split as it was already at 80 characters. |