Skip to content

Commit

Permalink
drm/i915/gvt: Don't dump partial state in cmd parser
Browse files Browse the repository at this point in the history
I have seen the cmd parser dump partial odd info. Stop that and only dump
the full verbose info when debug enabled.

Signed-off-by: Changbin Du <changbin.du@intel.com>
Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
  • Loading branch information
Changbin Du authored and Zhenyu Wang committed Nov 16, 2017
1 parent ffe2a50 commit e4aeba6
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions drivers/gpu/drm/i915/gvt/cmd_parser.c
Original file line number Diff line number Diff line change
Expand Up @@ -709,18 +709,13 @@ static void parser_exec_state_dump(struct parser_exec_state *s)

print_opcode(cmd_val(s, 0), s->ring_id);

/* print the whole page to trace */
pr_err(" ip_va=%p: %08x %08x %08x %08x\n",
s->ip_va, cmd_val(s, 0), cmd_val(s, 1),
cmd_val(s, 2), cmd_val(s, 3));

s->ip_va = (u32 *)((((u64)s->ip_va) >> 12) << 12);

while (cnt < 1024) {
pr_err("ip_va=%p: ", s->ip_va);
gvt_dbg_cmd("ip_va=%p: ", s->ip_va);
for (i = 0; i < 8; i++)
pr_err("%08x ", cmd_val(s, i));
pr_err("\n");
gvt_dbg_cmd("%08x ", cmd_val(s, i));
gvt_dbg_cmd("\n");

s->ip_va += 8 * sizeof(u32);
cnt += 8;
Expand Down

0 comments on commit e4aeba6

Please sign in to comment.