Re: [PATCH] tools/kvm_stat: fix out of date aarch64 kvm_exit reason definations

[Stefan Raspl]

On 4/3/21 11:17 AM, Zeng Tao wrote:

Aarch64 kvm exit reason defination is out of date for some time, so in
this patch:
1. Sync some newly introduced or missing EC definations.
2. Change the WFI to WFx.
3. Fix the comment.

Not all the definations are used or usable for aarch64 kvm, but it's
better to keep align across the whole kernel.

Signed-off-by: Zeng Tao 
---
  tools/kvm/kvm_stat/kvm_stat | 10 --
  1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/tools/kvm/kvm_stat/kvm_stat b/tools/kvm/kvm_stat/kvm_stat
index b0bf56c..63d87fd 100755
--- a/tools/kvm/kvm_stat/kvm_stat
+++ b/tools/kvm/kvm_stat/kvm_stat
@@ -154,17 +154,19 @@ SVM_EXIT_REASONS = {
  'NPF':0x400,
  }
  
-# EC definition of HSR (from arch/arm64/include/asm/kvm_arm.h)

+# EC definition of HSR (from arch/arm64/include/asm/esr.h)
  AARCH64_EXIT_REASONS = {
  'UNKNOWN':  0x00,
-'WFI':  0x01,
+'WFx':  0x01,
  'CP15_32':  0x03,
  'CP15_64':  0x04,
  'CP14_MR':  0x05,
  'CP14_LS':  0x06,
  'FP_ASIMD': 0x07,
  'CP10_ID':  0x08,
+'PAC':  0x09,
  'CP14_64':  0x0C,
+'BTI':  0x0D,
  'ILL_ISS':  0x0E,
  'SVC32':0x11,
  'HVC32':0x12,
@@ -173,6 +175,10 @@ AARCH64_EXIT_REASONS = {
  'HVC64':0x16,
  'SMC64':0x17,
  'SYS64':0x18,
+'SVE':  0x19,
+'ERET': 0x1a,
+'FPAC': 0x1c,
+'IMP_DEF':  0x1f,
  'IABT': 0x20,
  'IABT_HYP': 0x21,
  'PC_ALIGN': 0x22,



Reviewed-by: Stefan Raspl 


--

Mit freundlichen Grüßen / Kind regards

Stefan Raspl


Linux on Z
---
IBM Deutschland
Schoenaicher Str. 220
71032 Boeblingen
Phone: +49-7031-16-2177
E-Mail: stefan.ra...@de.ibm.com
---
IBM Deutschland Research & Development GmbH / Vorsitzender des Aufsichtsrats: 
Gregor Pillen

Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen / Registergericht: Amtsgericht Stuttgart, HRB 
243294

Re: [PATCH] perf kvm: Fix subcommands on s390

[Stefan Raspl]

On 12.07.2018 09:09, Thomas Richter wrote:
> With commit eca0fa28cd0d ("perf record: Provide detailed information on s390 
> CPU")
> s390 platform provides detailed type/model/capacitiy information
> in the CPU indentifier string instead of just "IBM/S390".
> 
> This breaks perf kvm support which uses hard coded string IBM/S390 to
> compare with the CPU identifier string. Fix this by changing the comparison.
> 
> Fixes: eca0fa28cd0d ("perf record: Provide detailed information on s390 CPU")
> Cc: Stefan Raspl 
> Cc:  # 4.17
> 
> Signed-off-by: Thomas Richter 
> Reviewed-by: Hendrik Brueckner 


Reported-and-tested-by: Stefan Raspl 

[PATCH] documentation: Add links to TRACE_EVENT documentation

[Stefan Raspl]

documentation: Add links to TRACE_EVENT documentation

Existing tracepoint documentation doesn't mention the popular
TRACE_EVENT
macro. Since an excellent series of articles on proper usage already
exists, respective links are added to the existing documentation.

Signed-off-by: Stefan Raspl 

---
 Documentation/trace/tracepoints.txt |5 +
 1 file changed, 5 insertions(+)

--- a/Documentation/trace/tracepoints.txt
+++ b/Documentation/trace/tracepoints.txt
@@ -99,3 +99,8 @@ core kernel image or in modules.
 If the tracepoint has to be used in kernel modules, an
 EXPORT_TRACEPOINT_SYMBOL_GPL() or EXPORT_TRACEPOINT_SYMBOL() can be
 used to export the defined tracepoints.
+
+Note: The convenience macro TRACE_EVENT provides an alternative way to
+  define tracepoints. Check http://lwn.net/Articles/379903,
+  http://lwn.net/Articles/381064 and http://lwn.net/Articles/383362
+  for a series of articles with more details.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/