On Wed, Dec 18, 2013 at 05:39:42PM +0000, Lorenzo Pieralisi wrote:
On Tue, Dec 17, 2013 at 07:55:19PM +0000, Mark Brown wrote:
I do think it's useful to to have the function there partly just as a hook but more for the warning it generates if we manage to boot a core that we didn't hook up with the topology information. This seems like
I think you have a point, the question is whether we need that function to run the check you mention or not, we can run the check for all possible cpus after completing the topology parsing. Actually, we might be
Yeah, it should be possible to do it elsewhere but then since the SMP code is already enumerating all the cores it's going to boot it seemed sensible to keep piggybacking on that and not duplicate the logic.
building topology for cores that do not come online as well, I do not think this is a problem, but I have to cast a proper look into that before jumping to conclusions, thanks for pointing that out.
I'd expect that could reasonably happen if cores are administratively disabled for some reason - that's one of the reasons for the status marking in DT.