From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 2/2] drm/i915: properly clflush pwrites to phys objects Date: Tue, 29 Nov 2011 15:35:54 +0000 Message-ID: References: <1322579369-26954-1-git-send-email-daniel.vetter@ffwll.ch> <1322579369-26954-2-git-send-email-daniel.vetter@ffwll.ch> Return-path: In-Reply-To: <1322579369-26954-2-git-send-email-daniel.vetter@ffwll.ch> Sender: linux-kernel-owner@vger.kernel.org To: intel-gfx@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Daniel Vetter , stable@kernel.org List-Id: dri-devel@lists.freedesktop.org On Tue, 29 Nov 2011 16:09:29 +0100, Daniel Vetter wrote: > Usually results in (rare) cursor corruptions on platforms > requiring physically addressed cursors. So the phys cursor pages are set to WC upon creation, are we just missing the mb()? Or more likely the CPUs don't have PAT and we are being lazy in not detecting the error. Anyway as this is reported to fix the issue on 8xx at least, overkill is fine. :) Reviewed-by: Chris Wilson -Chris -- Chris Wilson, Intel Open Source Technology Centre