x86: Show symbol name if insn decoder test failed
authorMasami Hiramatsu <mhiramat@redhat.com>
Mon, 16 Nov 2009 23:06:24 +0000 (18:06 -0500)
committerIngo Molnar <mingo@elte.hu>
Tue, 17 Nov 2009 06:16:50 +0000 (07:16 +0100)
Show symbol name if insn decoder test find a difference.
This will help us to find out where the issue is.

Signed-off-by: Masami Hiramatsu <mhiramat@redhat.com>
Cc: systemtap <systemtap@sources.redhat.com>
Cc: DLE <dle-develop@lists.sourceforge.net>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Randy Dunlap <rdunlap@xenotime.net>
Cc: Jim Keniston <jkenisto@us.ibm.com>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
LKML-Reference: <20091116230624.5250.49813.stgit@harusame>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/tools/distill.awk
arch/x86/tools/test_get_len.c

index d433619bb866279e438f18fc9d44decd96234690..c13c0ee48ab421a9a13e9170420dce1fe3e3d796 100644 (file)
@@ -15,6 +15,11 @@ BEGIN {
        fwait_str="9b\tfwait"
 }
 
+/^ *[0-9a-f]+ <[^>]*>:/ {
+       # Symbol entry
+       printf("%s%s\n", $2, $1)
+}
+
 /^ *[0-9a-f]+:/ {
        if (split($0, field, "\t") < 3) {
                # This is a continuation of the same insn.
index 5743e5128d352ea5697254d71ea91f95bb91772f..af75e07217ba5616f8cebed4951624bc6787594d 100644 (file)
@@ -110,7 +110,7 @@ static void parse_args(int argc, char **argv)
 
 int main(int argc, char **argv)
 {
-       char line[BUFSIZE];
+       char line[BUFSIZE], sym[BUFSIZE] = "<unknown>";
        unsigned char insn_buf[16];
        struct insn insn;
        int insns = 0, c;
@@ -122,6 +122,12 @@ int main(int argc, char **argv)
                int nb = 0;
                unsigned int b;
 
+               if (line[0] == '<') {
+                       /* Symbol line */
+                       strcpy(sym, line);
+                       continue;
+               }
+
                insns++;
                memset(insn_buf, 0, 16);
                strcpy(copy, line);
@@ -145,6 +151,8 @@ int main(int argc, char **argv)
                insn_init(&insn, insn_buf, x86_64);
                insn_get_length(&insn);
                if (insn.length != nb) {
+                       fprintf(stderr, "Error: %s found a difference at %s\n",
+                               prog, sym);
                        fprintf(stderr, "Error: %s", line);
                        fprintf(stderr, "Error: objdump says %d bytes, but "
                                "insn_get_length() says %d\n", nb,