Page MenuHomeFreeBSD

amd64: Call clock_init() after finishidentcpu()
AbandonedPublic

Authored by khng on Mar 4 2022, 5:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 11, 4:11 AM
Unknown Object (File)
Wed, Sep 9, 8:05 PM
Unknown Object (File)
Tue, Sep 8, 8:02 AM
Unknown Object (File)
Sun, Sep 6, 2:34 AM
Unknown Object (File)
Sat, Sep 5, 3:10 PM
Unknown Object (File)
Fri, Sep 4, 10:07 PM
Unknown Object (File)
Fri, Sep 4, 4:43 PM
Unknown Object (File)
Fri, Sep 4, 1:51 PM
Subscribers

Details

Reviewers
markj
Summary

In a subsequent commit clock_init() will attempt to determine the TSC
frequency, and this requires that CPU identification is finalized.

MFC after: 2 weeks
Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44656
Build 41544: arc lint + arc unit

Event Timeline

khng requested review of this revision.Mar 4 2022, 5:08 PM

In a subsequent commit clock_init() will attempt to determine the TSC frequency, and this requires that CPU identification is finalized.

Need to remove "In a subsequent commit" and change "will attempt" to "attempts".

sys/amd64/amd64/machdep.c
1456

A more conservative approach is to move the clock_init() call instead, but ok.

This revision is now accepted and ready to land.Mar 4 2022, 6:34 PM