From: Junkyeong Kim Date: Thu, 28 May 2020 04:14:02 +0000 (+0900) Subject: e_hwc_windows: change pp_commit flag false setting location X-Git-Tag: submit/tizen/20200528.224542~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8c2ec7406d20ed13dc8cc6c0ad09ba87643e528e;p=platform%2Fupstream%2Fenlightenment.git e_hwc_windows: change pp_commit flag false setting location fix pp destroy fail error. sometimes pp destroy is cannot be called cause of inappropriate pp_commit flag false set location. Change-Id: I8019d43aefebdf9abe0d75a916cf93d8f8d6ea97 Signed-off-by: Junkyeong Kim --- diff --git a/src/bin/e_hwc_windows.c b/src/bin/e_hwc_windows.c index 2acf17059d..d11b1f9b86 100644 --- a/src/bin/e_hwc_windows.c +++ b/src/bin/e_hwc_windows.c @@ -1365,7 +1365,6 @@ _e_hwc_windows_pp_output_commit_handler(tdm_output *toutput, unsigned int sequen _e_hwc_windows_commit_data_release(hwc, sequence, tv_sec, tv_usec); hwc->wait_commit = EINA_FALSE; - hwc->pp_commit = EINA_FALSE; /* layer already resetted */ if (hwc->pp_output_commit_data) @@ -1462,7 +1461,6 @@ fail: _e_hwc_windows_commit_data_release(hwc, 0, 0, 0); hwc->wait_commit = EINA_FALSE; - hwc->pp_commit = EINA_FALSE; return EINA_FALSE; } @@ -1526,6 +1524,8 @@ _e_hwc_windows_pp_commit_handler(tdm_pp *pp, tbm_surface_h tsurface_src, tbm_sur EHWSTRACE("!!!!!!!! HWC PP Commit Handler !!!!!!!!", NULL, hwc); EHWSTRACE(" tsurface src(%p) dst(%p)", NULL, hwc, tsurface_src, tsurface_dst); + hwc->pp_commit = EINA_FALSE; + /* if pp_set is false, skip the commit */ if (!hwc->pp_set) { @@ -1536,7 +1536,6 @@ _e_hwc_windows_pp_commit_handler(tdm_pp *pp, tbm_surface_h tsurface_src, tbm_sur } _e_hwc_windows_commit_data_release(hwc, 0, 0, 0); hwc->wait_commit = EINA_FALSE; - hwc->pp_commit = EINA_FALSE; goto done; } @@ -1546,9 +1545,7 @@ _e_hwc_windows_pp_commit_handler(tdm_pp *pp, tbm_surface_h tsurface_src, tbm_sur { tbm_surface_queue_release(hwc->pp_tqueue, tsurface_dst); _e_hwc_windows_commit_data_release(hwc, 0, 0, 0); - hwc->wait_commit = EINA_FALSE; - hwc->pp_commit = EINA_FALSE; goto done; }