From 4b558281e04e4d4febfc361632a90f8a45080c49 Mon Sep 17 00:00:00 2001 From: Chris Wilson Date: Sun, 2 Sep 2012 12:23:59 +0100 Subject: [PATCH] sna: Don't retire when searching for a snoopable buffer without hw support If the hw/kernel doesn't support snoopable buffers, then it makes little sense to search for one, and force a retire in the certainty of not finding any. Signed-off-by: Chris Wilson --- src/sna/kgem.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/sna/kgem.c b/src/sna/kgem.c index 6df5a2e5..cb0c82a7 100644 --- a/src/sna/kgem.c +++ b/src/sna/kgem.c @@ -1375,6 +1375,9 @@ search_snoop_cache(struct kgem *kgem, unsigned int num_pages, unsigned flags) DBG(("%s: num_pages=%d, flags=%x\n", __FUNCTION__, num_pages, flags)); + if ((kgem->has_cacheing | kgem->has_userptr) == 0) + return NULL; + if (list_is_empty(&kgem->snoop)) { DBG(("%s: inactive and cache empty\n", __FUNCTION__)); if (!__kgem_throttle_retire(kgem, flags)) {