[kgtp] r948 committed - Remove some op to OP_SPECIAL_GETV, OP_SPECIAL_SETV, OP_SPECIAL_TRACEV

  • From: kgtp@xxxxxxxxxxxxxx
  • To: kgtp@xxxxxxxxxxxxx
  • Date: Fri, 02 Mar 2012 14:13:40 +0000

Revision: 948
Author:   teawater
Date:     Fri Mar  2 06:12:17 2012
Log: Remove some op to OP_SPECIAL_GETV, OP_SPECIAL_SETV, OP_SPECIAL_TRACEV

http://code.google.com/p/kgtp/source/detail?r=948

Modified:
 /trunk/gtp.c
 /trunk/gtp_for_review.patch

=======================================
--- /trunk/gtp.c        Fri Mar  2 05:50:56 2012
+++ /trunk/gtp.c        Fri Mar  2 06:12:17 2012
@@ -249,9 +249,9 @@

 #define INT2CHAR(h)            ((h) > 9 ? (h) + 'a' - 10 : (h) + '0')

-#define OP_GETV_SPECIAL                0xfa
-#define OP_SETV_SPECIAL                0xfb
-#define OP_TRACEV_SPECIAL      0xfc
+#define OP_SPECIAL_GETV                0xfa
+#define OP_SPECIAL_SETV                0xfb
+#define OP_SPECIAL_TRACEV      0xfc
 #define OP_TRACE_PRINTK                0xfd
 #define OP_TRACE_QUICK_PRINTK  0xfe
 #define OP_TRACEV_PRINTK       0xff
@@ -3159,7 +3159,7 @@
                                gtp_action_x_getv;
                                break;
                        /* getv_sepecial */
-                       case OP_GETV_SPECIAL:
+                       case OP_SPECIAL_GETV:
                                arg = ebuf[pc++];
                                arg = (arg << 8) + ebuf[pc++];
                                stack[sp++] = top;
@@ -3172,7 +3172,7 @@
                                gtp_action_x_setv;
                                break;
                        /* setv_sepecial */
-                       case OP_SETV_SPECIAL:
+                       case OP_SPECIAL_SETV:
                                arg = ebuf[pc++];
                                arg = (arg << 8) + ebuf[pc++];
                                gtp_set_var_special(gts, arg, top);
@@ -3184,7 +3184,7 @@
                                gtp_action_x_tracev;
                                break;
                        /* tracev_special */
-                       case OP_TRACEV_SPECIAL:
+                       case OP_SPECIAL_TRACEV:
                                arg = ebuf[pc++];
                                arg = (arg << 8) + ebuf[pc++];
                                gtp_collect_var_special(gts, arg);
@@ -3523,7 +3523,7 @@
                                gtp_action_x_getv;
                                break;
                        /* getv_sepecial */
-                       case OP_GETV_SPECIAL:
+                       case OP_SPECIAL_GETV:
                                arg = ebuf[pc++];
                                arg = (arg << 8) + ebuf[pc++];
                                stack[sp++] = top;
@@ -3536,7 +3536,7 @@
                                gtp_action_x_setv;
                                break;
                        /* setv_sepecial */
-                       case OP_SETV_SPECIAL:
+                       case OP_SPECIAL_SETV:
                                arg = ebuf[pc++];
                                arg = (arg << 8) + ebuf[pc++];
                                gtp_set_var_special(gts, arg, top);
@@ -3548,7 +3548,7 @@
                                gtp_action_x_tracev;
                                break;
                        /* tracev_special */
-                       case OP_TRACEV_SPECIAL:
+                       case OP_SPECIAL_TRACEV:
                                arg = ebuf[pc++];
                                arg = (arg << 8) + ebuf[pc++];
                                gtp_collect_var_special(gts, arg);
@@ -4828,7 +4828,7 @@
                                        else if (arg == GTP_VAR_COOKED_RDTSC_ID)
                                                gtp_access_cooked_rdtsc = 1;
 #endif
-                                       ebuf[pc - 3] = OP_GETV_SPECIAL;
+                                       ebuf[pc - 3] = OP_SPECIAL_GETV;
                                }
                        }
                        if (ae->type == 'X') {
@@ -4868,7 +4868,7 @@
                                        if (arg == GTP_VAR_PRINTK_LEVEL_ID)
                                                tpe->have_printk = 1;

-                                       ebuf[pc - 3] = OP_SETV_SPECIAL;
+                                       ebuf[pc - 3] = OP_SPECIAL_SETV;
                                }
                        }
                        break;
@@ -4902,7 +4902,7 @@
                                        else if (arg == GTP_VAR_COOKED_RDTSC_ID)
                                                gtp_access_cooked_rdtsc = 1;
 #endif
-                                       ebuf[pc - 3] = OP_TRACEV_SPECIAL;
+                                       ebuf[pc - 3] = OP_SPECIAL_TRACEV;
                                }
                        }
                        break;
=======================================
--- /trunk/gtp_for_review.patch Fri Mar  2 05:50:56 2012
+++ /trunk/gtp_for_review.patch Fri Mar  2 06:12:17 2012
@@ -8,12 +8,12 @@
  kernel/events/core.c             |   14
  lib/Kconfig.debug                |   10
  lib/Makefile                     |    4
- lib/gtp.c | 8665 +++++++++++++++++++++++++++++++++++++++ + lib/gtp.c | 8666 +++++++++++++++++++++++++++++++++++++++
  lib/gtp_rb.c                     |  495 ++
  scripts/gtp/add-ons/pe.py        |  729 +++
  scripts/gtp/getgtprsp.pl         |  137
  scripts/gtp/getmod.py            |  132
- 14 files changed, 12005 insertions(+), 2 deletions(-)
+ 14 files changed, 12006 insertions(+), 2 deletions(-)

 --- /dev/null
 +++ b/Documentation/gtp/howto.txt
@@ -1952,7 +1952,7 @@
 +gtp.o: gtp_rb.c
 --- /dev/null
 +++ b/lib/gtp.c
-@@ -0,0 +1,8665 @@
+@@ -0,0 +1,8666 @@
 +/*
 + * Kernel GDB tracepoint module.
 + *
@@ -2056,9 +2056,9 @@
 +
 +#define INT2CHAR(h)           ((h) > 9 ? (h) + 'a' - 10 : (h) + '0')
 +
-+#define OP_GETV_SPECIAL               0xfa
-+#define OP_SETV_SPECIAL               0xfb
-+#define OP_TRACEV_SPECIAL     0xfc
++#define OP_SPECIAL_GETV               0xfa
++#define OP_SPECIAL_SETV               0xfb
++#define OP_SPECIAL_TRACEV     0xfc
 +#define OP_TRACE_PRINTK               0xfd
 +#define OP_TRACE_QUICK_PRINTK 0xfe
 +#define OP_TRACEV_PRINTK      0xff
@@ -4584,7 +4584,7 @@
 +                              gtp_action_x_getv;
 +                              break;
 +                      /* getv_sepecial */
-+                      case OP_GETV_SPECIAL:
++                      case OP_SPECIAL_GETV:
 +                              arg = ebuf[pc++];
 +                              arg = (arg << 8) + ebuf[pc++];
 +                              stack[sp++] = top;
@@ -4597,7 +4597,7 @@
 +                              gtp_action_x_setv;
 +                              break;
 +                      /* setv_sepecial */
-+                      case OP_SETV_SPECIAL:
++                      case OP_SPECIAL_SETV:
 +                              arg = ebuf[pc++];
 +                              arg = (arg << 8) + ebuf[pc++];
 +                              gtp_set_var_special(gts, arg, top);
@@ -4609,7 +4609,7 @@
 +                              gtp_action_x_tracev;
 +                              break;
 +                      /* tracev_special */
-+                      case OP_TRACEV_SPECIAL:
++                      case OP_SPECIAL_TRACEV:
 +                              arg = ebuf[pc++];
 +                              arg = (arg << 8) + ebuf[pc++];
 +                              gtp_collect_var_special(gts, arg);
@@ -4948,7 +4948,7 @@
 +                              gtp_action_x_getv;
 +                              break;
 +                      /* getv_sepecial */
-+                      case OP_GETV_SPECIAL:
++                      case OP_SPECIAL_GETV:
 +                              arg = ebuf[pc++];
 +                              arg = (arg << 8) + ebuf[pc++];
 +                              stack[sp++] = top;
@@ -4961,7 +4961,7 @@
 +                              gtp_action_x_setv;
 +                              break;
 +                      /* setv_sepecial */
-+                      case OP_SETV_SPECIAL:
++                      case OP_SPECIAL_SETV:
 +                              arg = ebuf[pc++];
 +                              arg = (arg << 8) + ebuf[pc++];
 +                              gtp_set_var_special(gts, arg, top);
@@ -4973,7 +4973,7 @@
 +                              gtp_action_x_tracev;
 +                              break;
 +                      /* tracev_special */
-+                      case OP_TRACEV_SPECIAL:
++                      case OP_SPECIAL_TRACEV:
 +                              arg = ebuf[pc++];
 +                              arg = (arg << 8) + ebuf[pc++];
 +                              gtp_collect_var_special(gts, arg);
@@ -6230,7 +6230,7 @@
 +                                      else if (arg == GTP_VAR_COOKED_RDTSC_ID)
 +                                              gtp_access_cooked_rdtsc = 1;
 +#endif
-+                                      ebuf[pc - 3] = OP_GETV_SPECIAL;
++                                      ebuf[pc - 3] = OP_SPECIAL_GETV;
 +                              }
 +                      }
 +                      if (ae->type == 'X') {
@@ -6249,27 +6249,28 @@
 +                              arg = ebuf[pc++];
 +                              arg = (arg << 8) + ebuf[pc++];
 +
-+                              if (arg == GTP_VAR_NO_SELF_TRACE_ID) {
-+                                      tpe->no_self_trace = 1;
-+                                      ret = 1;
-+                                      goto release_out;
-+                              } else if (arg == GTP_VAR_KRET_ID) {
-+                                      /* XXX: still not set it
-+                                         value to maxactive.  */
-+                                      tpe->is_kretprobe = 1;
-+                                      ret = 1;
-+                                      goto release_out;
-+                              }
-+
-+
-+                              if (arg == GTP_VAR_PRINTK_LEVEL_ID)
-+                                      tpe->have_printk = 1;
-+
 +                              if (!GTP_VAR_IS_SPECIAL(arg)) {
 +                                      if (gtp_x_var_add(&vlist, arg, 2)) {
 +                                              ret = -ENOMEM;
 +                                              goto release_out;
 +                                      }
++                              } else {
++                                      if (arg == GTP_VAR_NO_SELF_TRACE_ID) {
++                                              tpe->no_self_trace = 1;
++                                              ret = 1;
++                                              goto release_out;
++                                      } else if (arg == GTP_VAR_KRET_ID) {
++                                              /* XXX: still not set it
++                                              value to maxactive.  */
++                                              tpe->is_kretprobe = 1;
++                                              ret = 1;
++                                              goto release_out;
++                                      }
++
++                                      if (arg == GTP_VAR_PRINTK_LEVEL_ID)
++                                              tpe->have_printk = 1;
++
++                                      ebuf[pc - 3] = OP_SPECIAL_SETV;
 +                              }
 +                      }
 +                      break;
@@ -6303,7 +6304,7 @@
 +                                      else if (arg == GTP_VAR_COOKED_RDTSC_ID)
 +                                              gtp_access_cooked_rdtsc = 1;
 +#endif
-+                                      ebuf[pc - 3] = OP_TRACEV_SPECIAL;
++                                      ebuf[pc - 3] = OP_SPECIAL_TRACEV;
 +                              }
 +                      }
 +                      break;

Other related posts:

  • » [kgtp] r948 committed - Remove some op to OP_SPECIAL_GETV, OP_SPECIAL_SETV, OP_SPECIAL_TRACEV - kgtp