drm/i915/gvt: Make MI_USER_INTERRUPT nop in cmd parser

Zhipeng Gong · zhenyw · commit 5da795b061a1 · 2018-03-28T16:14:59.000+08:00
GVT-g dispatches request to host i915 and depends on i915 notify
ring interrupt mechanism to check completion of request.
For now MI_USER_INTERRUPT in guest requests is passed through
in GVT-g cmd parser and i915 does not use it, which causes
unnecessary interrupt handling in i915.
On the other hand, if several requests from guest are combined into
one request in and contain MI_USER_INTERRUPT in the middle of
combined request. GVT-g still has to wait on the whole request to
complete to inject user interrupts to guest.

This patch makes all the MI_USER_INTERRUPT nop to save some interrupt
handling.

Here is test result to run glmark2 on guest for 10 seconds:
host master interrupts number is reduced from 16021 to 11162
host user interrupts number is reduced from 7936 to 3536

v2:
- revise commit message. (Kevin)

Reviewed-by: Kevin Tian &lt;kevin.tian@intel.com&gt;
Signed-off-by: Zhipeng Gong &lt;zhipeng.gong@intel.com&gt;
Signed-off-by: Zhenyu Wang &lt;zhenyuw@linux.intel.com&gt;
diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c b/drivers/gpu/drm/i915/gvt/cmd_parser.c
@@ -1079,6 +1079,7 @@ static int cmd_handler_mi_user_interrupt(struct parser_exec_state *s)
 {
 	set_bit(cmd_interrupt_events[s->ring_id].mi_user_interrupt,
 			s->workload->pending_events);
+	patch_value(s, cmd_ptr(s, 0), MI_NOOP);
 	return 0;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1079,6 +1079,7 @@ static int cmd_handler_mi_user_interrupt(struct parser_exec_state *s)`
`1079`	`1079`	`{`
`1080`	`1080`	`set_bit(cmd_interrupt_events[s->ring_id].mi_user_interrupt,`
`1081`	`1081`	`s->workload->pending_events);`
	`1082`	`+ patch_value(s, cmd_ptr(s, 0), MI_NOOP);`
`1082`	`1083`	`return 0;`
`1083`	`1084`	`}`
`1084`	`1085`