Opened 8 years ago

Closed 20 months 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:
Has a Patch: no Platform: All

Description (last modified by umccullough)

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)

0001-x86-add-cpuid-feature-6-flags.patch (4.6 KB ) - added by yongcong 8 years ago.
0001-x86-add-cpuid-feature-6-flags.2.patch (4.6 KB ) - added by yongcong 8 years ago.
syslog (344.0 KB ) - added by SeanCollins 8 years ago.

Download all attachments as: .zip

Change History (18)

comment:1 by yongcong, 8 years ago

Has a Patch: set

comment:2 by tqh, 8 years ago

To me the patch looks ok and done the right way. The ticket could have a better description though, like you do in #8445.

comment:3 by yongcong, 8 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:4 by yongcong, 8 years ago

oh, it seems that I can't change the description?

comment:5 by umccullough, 8 years ago

Description: modified (diff)

Fix description.

comment:6 by yongcong, 8 years ago

thanks umccullough

comment:7 by SeanCollins, 8 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 ?

comment:8 by yongcong, 8 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 SeanCollins, 8 years ago

Attachment: syslog added

comment:9 by SeanCollins, 8 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 diver, 8 years ago

Version: R1/alpha3R1/Development

comment:11 by tqh, 8 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 tqh, 8 years ago

Has a Patch: unset

comment:13 by mmadia, 7 years ago

Is there any reason to keep this ticket open?

comment:14 by axeld, 3 years ago

Owner: changed from axeld to nobody
Status: newassigned

comment:15 by korli, 20 months ago

Resolution: fixed
Status: assignedclosed

Probably safe to close.

Note: See TracTickets for help on using tickets.