From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 00/13] Gen7 batch buffer command parser Date: Wed, 29 Jan 2014 23:11:17 +0100 Message-ID: <20140129221117.GD17001@phenom.ffwll.local> References: <1385484699-51596-1-git-send-email-bradley.d.volkin@intel.com> <1391032514-19136-1-git-send-email-bradley.d.volkin@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ea0-f170.google.com (mail-ea0-f170.google.com [209.85.215.170]) by gabe.freedesktop.org (Postfix) with ESMTP id 55FAC11D8BC for ; Wed, 29 Jan 2014 14:11:25 -0800 (PST) Received: by mail-ea0-f170.google.com with SMTP id k10so1249606eaj.29 for ; Wed, 29 Jan 2014 14:11:21 -0800 (PST) Content-Disposition: inline In-Reply-To: <1391032514-19136-1-git-send-email-bradley.d.volkin@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: bradley.d.volkin@intel.com Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Wed, Jan 29, 2014 at 01:55:01PM -0800, bradley.d.volkin@intel.com wrote: > From: Brad Volkin > 3) Coherency. I've found a coherency issue on VLV when reading the batch buffer > from the CPU during execbuffer2. Userspace writes the batch via pwrite fast > path before calling execbuffer2. The parser reads stale data. This works fine > on IVB and HSW, so I believe it's an LLC vs. non-LLC issue. I'm just unclear > on what the correct flushing or synchronization is for this scenario. This > only matters if we get PPGTT working on VLV and enable the parser there. Hm, adopting the shmem_read clflushing didn't help for this? That would be fairly shocking, since it means our shmem read paths are broken. Which are used e.g. by the libva readback code for the encoded bitstream. One thing aside: When resending the complete series (even if it's just a subset) it's better to start a new thread. We tend to use in-reply-to only when resending individual patches, while the review discussion is still ongoing. That way the discussion stays together. But when there's been a bit a longer break it's imo better to start a new thread. Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch