Skip to content

Commit

Permalink
x86: MSR: add a struct representation of an MSR
Browse files Browse the repository at this point in the history
Add a struct representing a 64bit MSR pair consisting of a low and high
register part and convert msr_info to use it. Also, rename msr-on-cpu.c
to msr.c.

Side note: Put the cpumask.h include in __KERNEL__ space thus fixing an
allmodconfig build failure in the headers_check target.

CC: H. Peter Anvin <hpa@zytor.com>
Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
  • Loading branch information
Borislav Petkov committed Jun 10, 2009
1 parent 07a2039 commit 6bc1096
Show file tree
Hide file tree
Showing 3 changed files with 25 additions and 14 deletions.
11 changes: 11 additions & 0 deletions arch/x86/include/asm/msr.h
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,17 @@

#include <asm/asm.h>
#include <asm/errno.h>
#include <asm/cpumask.h>

struct msr {
union {
struct {
u32 l;
u32 h;
};
u64 q;
};
};

static inline unsigned long long native_read_tscp(unsigned int *aux)
{
Expand Down
2 changes: 1 addition & 1 deletion arch/x86/lib/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# Makefile for x86 specific library files.
#

obj-$(CONFIG_SMP) := msr-on-cpu.o
obj-$(CONFIG_SMP) := msr.o

lib-y := delay.o
lib-y += thunk_$(BITS).o
Expand Down
26 changes: 13 additions & 13 deletions arch/x86/lib/msr-on-cpu.c → arch/x86/lib/msr.c
Original file line number Diff line number Diff line change
Expand Up @@ -5,22 +5,22 @@

struct msr_info {
u32 msr_no;
u32 l, h;
struct msr reg;
int err;
};

static void __rdmsr_on_cpu(void *info)
{
struct msr_info *rv = info;

rdmsr(rv->msr_no, rv->l, rv->h);
rdmsr(rv->msr_no, rv->reg.l, rv->reg.h);
}

static void __wrmsr_on_cpu(void *info)
{
struct msr_info *rv = info;

wrmsr(rv->msr_no, rv->l, rv->h);
wrmsr(rv->msr_no, rv->reg.l, rv->reg.h);
}

int rdmsr_on_cpu(unsigned int cpu, u32 msr_no, u32 *l, u32 *h)
Expand All @@ -30,8 +30,8 @@ int rdmsr_on_cpu(unsigned int cpu, u32 msr_no, u32 *l, u32 *h)

rv.msr_no = msr_no;
err = smp_call_function_single(cpu, __rdmsr_on_cpu, &rv, 1);
*l = rv.l;
*h = rv.h;
*l = rv.reg.l;
*h = rv.reg.h;

return err;
}
Expand All @@ -42,8 +42,8 @@ int wrmsr_on_cpu(unsigned int cpu, u32 msr_no, u32 l, u32 h)
struct msr_info rv;

rv.msr_no = msr_no;
rv.l = l;
rv.h = h;
rv.reg.l = l;
rv.reg.h = h;
err = smp_call_function_single(cpu, __wrmsr_on_cpu, &rv, 1);

return err;
Expand All @@ -55,14 +55,14 @@ static void __rdmsr_safe_on_cpu(void *info)
{
struct msr_info *rv = info;

rv->err = rdmsr_safe(rv->msr_no, &rv->l, &rv->h);
rv->err = rdmsr_safe(rv->msr_no, &rv->reg.l, &rv->reg.h);
}

static void __wrmsr_safe_on_cpu(void *info)
{
struct msr_info *rv = info;

rv->err = wrmsr_safe(rv->msr_no, rv->l, rv->h);
rv->err = wrmsr_safe(rv->msr_no, rv->reg.l, rv->reg.h);
}

int rdmsr_safe_on_cpu(unsigned int cpu, u32 msr_no, u32 *l, u32 *h)
Expand All @@ -72,8 +72,8 @@ int rdmsr_safe_on_cpu(unsigned int cpu, u32 msr_no, u32 *l, u32 *h)

rv.msr_no = msr_no;
err = smp_call_function_single(cpu, __rdmsr_safe_on_cpu, &rv, 1);
*l = rv.l;
*h = rv.h;
*l = rv.reg.l;
*h = rv.reg.h;

return err ? err : rv.err;
}
Expand All @@ -84,8 +84,8 @@ int wrmsr_safe_on_cpu(unsigned int cpu, u32 msr_no, u32 l, u32 h)
struct msr_info rv;

rv.msr_no = msr_no;
rv.l = l;
rv.h = h;
rv.reg.l = l;
rv.reg.h = h;
err = smp_call_function_single(cpu, __wrmsr_safe_on_cpu, &rv, 1);

return err ? err : rv.err;
Expand Down

0 comments on commit 6bc1096

Please sign in to comment.