8.1-PRERELEASE: CPU packages not detected correctly
Oliver Fromme
olli at lurza.secnetix.de
Mon Jul 19 15:47:37 UTC 2010
Oliver Fromme wrote:
> Jung-uk Kim wrote:
> > On Thursday 15 July 2010 01:56 pm, Andriy Gapon wrote:
> > > on 15/07/2010 19:57 Oliver Fromme said the following:
> > > > I patched topo_probe() so it calls topo_probe_0x4() after
> > > > topo_probe_0xb() if cpu_cores is still 0. I think this
> > > > is a better fallback procedure. With this patch, cpu_cores
> > > > gets the value 4 which is the correct one, finally:
> > > [...]
> > > I think that your addition achieves this effect, perhaps just not
> > > as explicitly as I would preferred.
> > >
> > > Jung-uk, what do you think?
> >
> > Yes, you're right. Please try new patch:
> >
> > http://people.freebsd.org/~jkim/mp_machdep2.diff
>
> Thank you!
>
> I will have access to that particular machine on Monday again,
> so testing the new patch will have to wait until Monday.
> But from looking at your patch it should have the same result
> as my simpler patch, so it should work fine.
As expected, the patch works perfectly fine.
The cores are detected correctly:
FreeBSD/SMP: Multiprocessor System Detected: 8 CPUs
FreeBSD/SMP: 2 package(s) x 4 core(s)
kern.sched.topology_spec: <groups>
<group level="1" cache-level="0">
<cpu count="8" mask="0xff">0, 1, 2, 3, 4, 5, 6, 7</cpu>
<flags></flags>
<children>
<group level="3" cache-level="2">
<cpu count="4" mask="0xf">0, 1, 2, 3</cpu>
<flags></flags>
</group>
<group level="3" cache-level="2">
<cpu count="4" mask="0xf0">4, 5, 6, 7</cpu>
<flags></flags>
</group>
</children>
</group>
</groups>
Thanks for fixing this!
Best regards
Oliver
--
Oliver Fromme, secnetix GmbH & Co. KG, Marktplatz 29, 85567 Grafing b. M.
Handelsregister: Registergericht Muenchen, HRA 74606, Geschäftsfuehrung:
secnetix Verwaltungsgesellsch. mbH, Handelsregister: Registergericht Mün-
chen, HRB 125758, Geschäftsführer: Maik Bachmann, Olaf Erb, Ralf Gebhart
FreeBSD-Dienstleistungen, -Produkte und mehr: http://www.secnetix.de/bsd
Perl is worse than Python because people wanted it worse.
-- Larry Wall
More information about the freebsd-stable
mailing list