8:27 p.m.
From: Jani Nikula <jani.nikula(a)intel.com>
Suggest enabling CONFIG_X86_MSR. Add \n at the end of the error message.
Signed-off-by: Jani Nikula <jani.nikula(a)intel.com>
---
Note: This breaks translations looking for msgid "msr reg not found". I'm
unsure how you want this kind of changes handled.
---
src/cpu/intel_cpus.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/cpu/intel_cpus.cpp b/src/cpu/intel_cpus.cpp
index 2b78d31..30c311f 100644
--- a/src/cpu/intel_cpus.cpp
+++ b/src/cpu/intel_cpus.cpp
@@ -58,7 +58,7 @@ static uint64_t get_msr(int cpu, uint64_t offset)
fd = sprintf(msr_path, "/dev/msr%d", cpu);
if (access(msr_path, R_OK) != 0){
- fprintf(stderr, _("msr reg not found"));
+ fprintf(stderr, _("Model-specific registers (MSR) not found (try enabling
CONFIG_X86_MSR).\n"));
exit(-2);
}
}
--
1.7.9.5
8:37 p.m.
New subject: [Powertop] [PATCH] Add a more informative error message about MSR not found
On (02/21/13 11:27), jani.nikula(a)linux.intel.com wrote:
From: Jani Nikula <jani.nikula(a)intel.com>
Suggest enabling CONFIG_X86_MSR. Add \n at the end of the error message.
Signed-off-by: Jani Nikula <jani.nikula(a)intel.com>
---
Note: This breaks translations looking for msgid "msr reg not found". I'm
unsure how you want this kind of changes handled.
---
src/cpu/intel_cpus.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/cpu/intel_cpus.cpp b/src/cpu/intel_cpus.cpp
index 2b78d31..30c311f 100644
--- a/src/cpu/intel_cpus.cpp
+++ b/src/cpu/intel_cpus.cpp
@@ -58,7 +58,7 @@ static uint64_t get_msr(int cpu, uint64_t offset)
fd = sprintf(msr_path, "/dev/msr%d", cpu);
if (access(msr_path, R_OK) != 0){
- fprintf(stderr, _("msr reg not found"));
+ fprintf(stderr, _("Model-specific registers (MSR) not found (try enabling
CONFIG_X86_MSR).\n"));
exit(-2);
}
}
--
I think we're beyond the `string freeze' date, most likely it will not get into
upcoming release.
-ss
1.7.9.5
_______________________________________________
PowerTop mailing list
PowerTop(a)lists.01.org
https://lists.01.org/mailman/listinfo/powertop
10:17 p.m.
New subject: [Powertop] [PATCH] Add a more informative error message about MSR not found
On Thu, 21 Feb 2013, Sergey Senozhatsky <sergey.senozhatsky(a)gmail.com> wrote:
I think we're beyond the `string freeze' date, most likely it
will not
get into upcoming release.
I'm ignorant about powertop release practices, and don't mind whether it
makes the next release or not; hopefully some release after that though.
I'm just trying to give a hint to the next developer hitting the issue,
so he won't have to first grep the powertop sources to figure out what
the error really is about, and then the kernel sources to figure out
who's responsible for creating /dev/cpu/%d/msr, and what config knob to
turn to enable that... The README patch I sent of course helps too.
BR,
Jani.
11:52 p.m.
New subject: [Powertop] [PATCH] Add a more informative error message about MSR not found
On (02/21/13 13:17), Jani Nikula wrote:
On Thu, 21 Feb 2013, Sergey Senozhatsky
<sergey.senozhatsky(a)gmail.com> wrote:
> I think we're beyond the `string freeze' date, most likely it will not
> get into upcoming release.
I'm ignorant about powertop release practices, and don't mind whether it
makes the next release or not; hopefully some release after that though.
I'm just trying to give a hint to the next developer hitting the issue,
so he won't have to first grep the powertop sources to figure out what
the error really is about, and then the kernel sources to figure out
who's responsible for creating /dev/cpu/%d/msr, and what config knob to
turn to enable that... The README patch I sent of course helps too.
sure, thanks.
-ss
BR,
Jani.
3496
days inactive
3496
days old
3 comments
3 participants
participants (3)
-
Jani Nikula -
jani.nikula＠linux.intel.com
-
Sergey Senozhatsky