Changeset View
Changeset View
Standalone View
Standalone View
sys/sys/systm.h
Show First 20 Lines • Show All 394 Lines • ▼ Show 20 Lines | |||||
sbintime_t cpu_idleclock(void); | sbintime_t cpu_idleclock(void); | ||||
void cpu_activeclock(void); | void cpu_activeclock(void); | ||||
void cpu_new_callout(int cpu, sbintime_t bt, sbintime_t bt_opt); | void cpu_new_callout(int cpu, sbintime_t bt, sbintime_t bt_opt); | ||||
void cpu_et_frequency(struct eventtimer *et, uint64_t newfreq); | void cpu_et_frequency(struct eventtimer *et, uint64_t newfreq); | ||||
extern int cpu_disable_c2_sleep; | extern int cpu_disable_c2_sleep; | ||||
extern int cpu_disable_c3_sleep; | extern int cpu_disable_c3_sleep; | ||||
extern void (*tcp_hpts_softclock)(void); | extern void (*tcp_hpts_softclock)(void); | ||||
extern volatile uint32_t __read_frequently hpts_that_need_softclock; | |||||
gallatin: There is an attribute (__read_frequently) that I think you should add to both of these… | |||||
Done Inline ActionsOk let me add that and update. rrs: Ok let me add that and update. | |||||
#define tcp_hpts_softclock() do { \ | #define tcp_hpts_softclock() do { \ | ||||
if (tcp_hpts_softclock != NULL) \ | if (hpts_that_need_softclock > 0) \ | ||||
tcp_hpts_softclock(); \ | tcp_hpts_softclock(); \ | ||||
} while (0) | } while (0) | ||||
char *kern_getenv(const char *name); | char *kern_getenv(const char *name); | ||||
void freeenv(char *env); | void freeenv(char *env); | ||||
int getenv_int(const char *name, int *data); | int getenv_int(const char *name, int *data); | ||||
int getenv_uint(const char *name, unsigned int *data); | int getenv_uint(const char *name, unsigned int *data); | ||||
int getenv_long(const char *name, long *data); | int getenv_long(const char *name, long *data); | ||||
▲ Show 20 Lines • Show All 182 Lines • Show Last 20 Lines |
There is an attribute (__read_frequently) that I think you should add to both of these variables. Otherwise, it looks good to me.