[Intel-gfx] [PATCH] dim: robustify drm-rerere update logic

Daniel Vetter daniel.vetter at ffwll.ch
Fri Sep 8 16:26:16 UTC 2017


We've now had two reports (from Chris and Sean each one) that dim
stumbled over uncommitted state in drm-rerere/. In both cases a bunch
of git rm have been staged, but not committed.

I didn't really figure out what exactly happend, but best explanation
is that one of the commands failed, but silently, since we &>
/dev/null the output. Make sure no matter what happens, we get at
least to the commit step to not leave a broken state behind and add a
|| true everywhere it's missing.

Cc: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Sean Paul <seanpaul at chromium.org>
Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
---
 dim | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dim b/dim
index b4e46c698345..3647d7168a36 100755
--- a/dim
+++ b/dim
@@ -542,10 +542,10 @@ function commit_rerere_cache
 		git add ./*.patch >& /dev/null || true
 		for file  in $(git ls-files); do
 			if ! git log --since="60 days ago" --name-only -- $file | grep $file &> /dev/null; then
-				git rm $file &> /dev/null
+				git rm $file &> /dev/null || true
 			fi
 		done
-		find rr-cache/ -mtime -1 -type f -not -name "thisimage*" -print0 | xargs -0 git add > /dev/null
+		find rr-cache/ -mtime -1 -type f -not -name "thisimage*" -print0 | xargs -0 git add > /dev/null || true
 		git rm rr-cache/rr-cache &> /dev/null || true
 		if git commit -m "$time: $integration_branch rerere cache update" >& /dev/null; then
 			echo -n "New commit. "
-- 
2.14.1



More information about the Intel-gfx mailing list