Lack of MTX_CMDID_NULL causes topaz fence to timeout
authorNaeem M. Afzal <naeem.m.afzal@intel.com>
Thu, 16 Feb 2012 17:04:07 +0000 (09:04 -0800)
committerMarkus Lehtonen <markus.lehtonen@linux.intel.com>
Tue, 3 Jul 2012 09:30:15 +0000 (12:30 +0300)
this timeout results in reset the hardware for every
jpeg frame encode. vaSyncSurface will not complete until
the timeout occurs, since it is still waiting for
MTX_CMID_NULL. This shows up as slow performance
(we have to wait for timeout).

Issue: ANDROID-1633

Signed-off-by: naeem.m.afzal@intel.com
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
drivers/staging/mrst/imgv/pnw_topaz.c

index b646fd7..5ada7b4 100644 (file)
@@ -521,6 +521,7 @@ pnw_topaz_send(struct drm_device *dev, void *cmd,
                case MTX_CMDID_RESET_ENCODE:
                case MTX_CMDID_ISSUEBUFF:
                case MTX_CMDID_SETUP:
+               case MTX_CMDID_NULL:
                        cur_cmd_header->seq = topaz_priv->topaz_cmd_count++;
                        cur_cmd_header->enable_interrupt = 0;
                        cur_cmd_size = 4;