From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 17/48] drm/i915: Track which ring a context ran on Date: Tue, 22 Apr 2014 15:54:04 +0100 Message-ID: <20140422145404.GB4493@nuc-i3427.alporthouse.com> References: <20131206215521.GA6922@bwidawsk.net> <1386367941-7131-1-git-send-email-benjamin.widawsky@intel.com> <1386367941-7131-17-git-send-email-benjamin.widawsky@intel.com> <20140418095146.GA7270@nuc-i3427.alporthouse.com> <20140422142522.GR10722@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from fireflyinternet.com (mail.fireflyinternet.com [87.106.93.118]) by gabe.freedesktop.org (Postfix) with ESMTP id E761B6E8BA for ; Tue, 22 Apr 2014 07:54:12 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140422142522.GR10722@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: Intel GFX , Ben Widawsky , Ben Widawsky List-Id: intel-gfx@lists.freedesktop.org On Tue, Apr 22, 2014 at 04:25:22PM +0200, Daniel Vetter wrote: > I think this doesn't have the possibility to have broken anything yet > since we don't allow the same context on multiple rings. Except the > default one, but mesa creates new contexts anyway afaik. Or am I wrong? It's a minor regression in that we reload the context before every batch; a brief wtf when doing perf monitoring. -Chris -- Chris Wilson, Intel Open Source Technology Centre