x86, ptrace: use jiffies for BTS timestamps

Replace sched_clock() with jiffies for BTS timestamps.

Signed-off-by: Markus Metzger <markus.t.metzger@intel.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>

diff --git a/arch/x86/kernel/ds.c b/arch/x86/kernel/ds.c
index 996a7c4f5963230ea36dfcfc7b1cbe4c2f135354..e7855def97c3e62d6a37c504d100c08ddf7005be 100644 (file)
--- a/arch/x86/kernel/ds.c
+++ b/arch/x86/kernel/ds.c
@@ -167,23 +167,13 @@ static inline void set_info_type(char *base, unsigned char value)
 {
        (*(unsigned char *)(base + ds_cfg.info_type.offset)) = value;
 }
-/*
- * The info data might overlap with the info type on some architectures.
- * We therefore read and write the exact number of bytes.
- */
-static inline unsigned long long get_info_data(char *base)
+static inline unsigned long get_info_data(char *base)
 {
-       unsigned long long value = 0;
-       memcpy(&value,
-              base + ds_cfg.info_data.offset,
-              ds_cfg.info_data.size);
-       return value;
+       return *(unsigned long *)(base + ds_cfg.info_data.offset);
 }
-static inline void set_info_data(char *base, unsigned long long value)
+static inline void set_info_data(char *base, unsigned long value)
 {
-       memcpy(base + ds_cfg.info_data.offset,
-              &value,
-              ds_cfg.info_data.size);
+       (*(unsigned long *)(base + ds_cfg.info_data.offset)) = value;
 }
 
 
@@ -282,8 +272,8 @@ int ds_read_bts(void *ds, size_t index, struct bts_struct *out)
 
        memset(out, 0, sizeof(*out));
        if (get_from_ip(bts) == BTS_ESCAPE_ADDRESS) {
-               out->qualifier         = get_info_type(bts);
-               out->variant.timestamp = get_info_data(bts);
+               out->qualifier       = get_info_type(bts);
+               out->variant.jiffies = get_info_data(bts);
        } else {
                out->qualifier = BTS_BRANCH;
                out->variant.lbr.from_ip = get_from_ip(bts);
@@ -319,7 +309,7 @@ int ds_write_bts(void *ds, const struct bts_struct *in)
        case BTS_TASK_DEPARTS:
                set_from_ip(bts, BTS_ESCAPE_ADDRESS);
                set_info_type(bts, in->qualifier);
-               set_info_data(bts, in->variant.timestamp);
+               set_info_data(bts, in->variant.jiffies);
                break;
 
        default:
@@ -350,7 +340,7 @@ static const struct ds_configuration ds_cfg_netburst = {
        .from_ip = { 0, 4 },
        .to_ip = { 4, 4 },
        .info_type = { 4, 1 },
-       .info_data = { 5, 7 },
+       .info_data = { 8, 4 },
        .debugctl_mask = (1<<2)|(1<<3)
 };
 
@@ -364,7 +354,7 @@ static const struct ds_configuration ds_cfg_pentium_m = {
        .from_ip = { 0, 4 },
        .to_ip = { 4, 4 },
        .info_type = { 4, 1 },
-       .info_data = { 5, 7 },
+       .info_data = { 8, 4 },
        .debugctl_mask = (1<<6)|(1<<7)
 };
 #endif /* _i386_ */
@@ -379,7 +369,7 @@ static const struct ds_configuration ds_cfg_core2 = {
        .from_ip = { 0, 8 },
        .to_ip = { 8, 8 },
        .info_type = { 8, 1 },
-       .info_data = { 9, 7 },
+       .info_data = { 16, 8 },
        .debugctl_mask = (1<<6)|(1<<7)|(1<<9)
 };
 

diff --git a/arch/x86/kernel/ptrace.c b/arch/x86/kernel/ptrace.c
index 8d0dd8b5effec6791d272669ad01f31199c7bb65..ec86abaab530fb45e0a5ef15ca65c9fd16836442 100644 (file)
--- a/arch/x86/kernel/ptrace.c
+++ b/arch/x86/kernel/ptrace.c
@@ -616,7 +616,7 @@ void ptrace_bts_take_timestamp(struct task_struct *tsk,
 {
        struct bts_struct rec = {
                .qualifier = qualifier,
-               .variant.timestamp = sched_clock()
+               .variant.jiffies = jiffies
        };
 
        if (ptrace_bts_get_buffer_size(tsk) <= 0)

diff --git a/include/asm-x86/ds.h b/include/asm-x86/ds.h
index edd8467740a63b9481de92a3a24ec3ccd441494e..c9e15381dc7f14ce0e79cfd57ddacf508d332120 100644 (file)
--- a/include/asm-x86/ds.h
+++ b/include/asm-x86/ds.h
@@ -48,7 +48,7 @@ struct bts_struct {
                } lbr;
                /* BTS_TASK_ARRIVES or
                   BTS_TASK_DEPARTS */
-               unsigned long long timestamp;
+               unsigned long jiffies;
        } variant;
 };