Opened 13 years ago
Closed 7 years ago
#8444 closed enhancement (fixed)
x86: add cpuid feature 6 flags
Reported by: | yongcong | Owned by: | nobody |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | System/Kernel | Version: | R1/Development |
Keywords: | gsoc2012 | Cc: | |
Blocked By: | Blocking: | ||
Platform: | All |
Description (last modified by )
Add "Digital Thermal Sensor and Power Management Parameters" function 06H flags detection. ARAT and ENERGY_PERF_BIAS features are important for Operating Systems
Attachments (3)
Change History (18)
by , 13 years ago
Attachment: | 0001-x86-add-cpuid-feature-6-flags.patch added |
---|
comment:1 by , 13 years ago
patch: | 0 → 1 |
---|
comment:2 by , 13 years ago
comment:3 by , 13 years ago
Add "Digital Thermal Sensor and Power Management Parameters" function 06H flags detection. ARAT and ENERGY_PERF_BIAS features are important for Operating Systems
comment:7 by , 13 years ago
please fix the patch, it fials on 4 or 6 hunks. I will retest but is this Ia32 only or will this adress all x86 cpus ?
by , 13 years ago
Attachment: | 0001-x86-add-cpuid-feature-6-flags.2.patch added |
---|
comment:8 by , 13 years ago
rebase and resend the patch. The new patch can be applied to the latest master branch. Thank SeanCollins very much ;)
As the your question, FWICT, it address all x86 cpus at least those cpus from intel and amd
by , 13 years ago
comment:9 by , 13 years ago
2nd boot is with this patch applied, no negative changes in system behavior noted, If you need any additional testing dones on patchs like this please let me know.
comment:10 by , 13 years ago
Version: | R1/alpha3 → R1/Development |
---|
comment:11 by , 13 years ago
Patch applied in hrev44064. The FEATURE_6 names in the define could probably be more descriptive so maybe that should be changed.
comment:12 by , 13 years ago
patch: | 1 → 0 |
---|
comment:14 by , 8 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
To me the patch looks ok and done the right way. The ticket could have a better description though, like you do in #8445.