From: Kitae Kim <kt920.kim@samsung.com>
Date: Thu, 14 Aug 2014 07:16:54 +0000 (+0900)
Subject: brillcodec: remove useless memory copy.
X-Git-Tag: Tizen_Studio_1.3_Release_p2.3.1~264^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e6092fdf8f5da7b9898517f8a191f257d1eafe0c;p=sdk%2Femulator%2Fqemu.git

brillcodec: remove useless memory copy.

Unnecessary memcpy operations occur while decoding video.
In addition to this, it causes performance degradation in emulator.
After this commit, cpu usage on Host decreases 10 ~ 15%, especially HD Video.

Change-Id: If619b69c30e2587df839e108b962a7c8e9674d77
Signed-off-by: Kitae Kim <kt920.kim@samsung.com>
---

diff --git a/tizen/src/hw/maru_brill_codec.c b/tizen/src/hw/maru_brill_codec.c
index 8b60f5cc28..0ac55bb8c0 100644
--- a/tizen/src/hw/maru_brill_codec.c
+++ b/tizen/src/hw/maru_brill_codec.c
@@ -692,7 +692,7 @@ static int maru_brill_codec_get_picture_size(AVPicture *picture, uint8_t *ptr,
 
         if (!encode && !ptr) {
             TRACE("allocate a buffer for a decoded picture.\n");
-            ptr = av_mallocz(fsize);
+            ptr = g_malloc(fsize);
             if (!ptr) {
                 ERR("[%d] failed to allocate memory.\n", __LINE__);
                 return -1;
@@ -1373,7 +1373,7 @@ static bool codec_decode_video(MaruBrillCodecState *s, int ctx_id, void *data_bu
     tempbuf = g_malloc(tempbuf_size);
     if (!tempbuf) {
         ERR("failed to allocate decoded audio buffer\n");
-        // FIXME: how to handle this case?
+        tempbuf_size = 0;
     } else {
         struct video_data video;
 
@@ -1399,7 +1399,7 @@ static bool codec_picture_copy (MaruBrillCodecState *s, int ctx_id, void *elem)
     AVCodecContext *avctx = NULL;
     AVPicture *src = NULL;
     AVPicture dst;
-    uint8_t *out_buffer = NULL, *tempbuf = NULL;
+    uint8_t *tempbuf = NULL;
     int pict_size = 0;
     bool ret = true;
 
@@ -1433,14 +1433,7 @@ static bool codec_picture_copy (MaruBrillCodecState *s, int ctx_id, void *elem)
             av_picture_copy(&dst, src, avctx->pix_fmt,
                             avctx->width, avctx->height);
 
-            tempbuf = g_malloc0(pict_size);
-            if (!tempbuf) {
-                ERR("failed to allocate a picture buffer. size: %d\n", pict_size);
-            } else {
-                out_buffer = dst.data[0];
-                memcpy(tempbuf, out_buffer, pict_size);
-            }
-            av_free(out_buffer);
+            tempbuf = dst.data[0];
         }
     }