Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some max turbo boosting frequency may cause XCPM: P-state table mismatch (error:0x4) #104

Closed
ixsander opened this issue Dec 19, 2014 · 2 comments

Comments

@ixsander
Copy link

Hi,
With your help for the frequency vector (Thanks again!), I got my power management working under stock Turbo boosting mode. Now I am trying to raise turbo boosting frequency above stock. If I raise the Turbo boosting frequency from 4400(stock) to 4500 or 4700, I got

2014-12-19 1:01:35.000 PM kernel[0]: XCPM: P-state table mismatch (error:0x4)
2014-12-19 1:01:35.000 PM kernel[0]: X86PlatformShim::sendPStates - pmCPUControl (SETPSTATETABLE) returned 0x4
2014-12-19 1:01:35.000 PM kernel[0]: X86PlatformShim::start - Failed to send PStates

However, if I raise frequency to 4600, the pstates error is gone.
I noticed if I ran
ssdtPRGen.sh -turbo 4500
...
Number of Turbo States: 6 (4100-4500 MHz)
...

ssdtPRGen.sh
...
Number of Turbo States: 4 (4100-4400 MHz)
...

As you noticed, there is extra states created for 4500, 4700
in the master branch line 3939,

let turboStates=$(printf "%.f" $(echo "scale=1;((($maxTurboFrequency - $frequency) / $gBusFrequency)+0.5)" | bc))
Piker-Alpha added a commit that referenced this issue Dec 19, 2014
Piker-Alpha added a commit that referenced this issue Dec 19, 2014
@Piker-Alpha
Copy link
Owner

Can you please verify the latest update because this is what I get when I run:

./ssdtPRGen.sh -p i7-4790K -turbo 4500

Number of Turbo States: 5 (4100-4500 MHz)
Number of P-States: 38 (800-4500 MHz)

@ixsander
Copy link
Author

Yes. The last update works. Thank you!!!

Piker-Alpha added a commit that referenced this issue Jan 23, 2015
- Improved Pentium® support.
- Various whitespace only changes.
- Check value of argument -bclk.
- Improved error reporting for -bclk, -lfm and -t.
- Undo change for #104
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants