Re: [Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-12-24 Thread Rodrigo Vivi
to be honest I got lost on the discussion here. I like the simplicity Zhipeng used so I'm keeping it on collector and still +1 for the merge up. We can improve later and provide ratio usage for userspace to take care of balance work, but I don't see why to block progress here. On Wed, Dec 10, 20

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-12-10 Thread Zhao, Yakui
On Wed, 2014-12-10 at 08:55 -0700, Dave Gordon wrote: > On 10/12/14 09:11, Daniel Vetter wrote: > > On Wed, Dec 10, 2014 at 02:18:15AM +, Gong, Zhipeng wrote: > >> On Tue, 2014-12-09 at 10:46 +0100, Daniel Vetter wrote: > >>> On Mon, Dec 08, 2014 at 01:55:56PM -0800, Rodrigo Vivi wrote: > > [s

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-12-10 Thread Dave Gordon
On 10/12/14 09:11, Daniel Vetter wrote: > On Wed, Dec 10, 2014 at 02:18:15AM +, Gong, Zhipeng wrote: >> On Tue, 2014-12-09 at 10:46 +0100, Daniel Vetter wrote: >>> On Mon, Dec 08, 2014 at 01:55:56PM -0800, Rodrigo Vivi wrote: [snip] diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-12-10 Thread Daniel Vetter
On Wed, Dec 10, 2014 at 02:18:15AM +, Gong, Zhipeng wrote: > On Tue, 2014-12-09 at 10:46 +0100, Daniel Vetter wrote: > > On Mon, Dec 08, 2014 at 01:55:56PM -0800, Rodrigo Vivi wrote: > > > On Tue, Nov 25, 2014 at 5:04 AM, Daniel Vetter wrote: > > > > On Mon, Nov 24, 2014 at 08:29:40AM -0800, R

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-12-09 Thread Gong, Zhipeng
On Tue, 2014-12-09 at 10:46 +0100, Daniel Vetter wrote: > On Mon, Dec 08, 2014 at 01:55:56PM -0800, Rodrigo Vivi wrote: > > On Tue, Nov 25, 2014 at 5:04 AM, Daniel Vetter wrote: > > > On Mon, Nov 24, 2014 at 08:29:40AM -0800, Rodrigo Vivi wrote: > > >> From: Zhipeng Gong > > >> > > >> On Broadwel

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-12-09 Thread Daniel Vetter
On Mon, Dec 08, 2014 at 01:55:56PM -0800, Rodrigo Vivi wrote: > On Tue, Nov 25, 2014 at 5:04 AM, Daniel Vetter wrote: > > On Mon, Nov 24, 2014 at 08:29:40AM -0800, Rodrigo Vivi wrote: > >> From: Zhipeng Gong > >> > >> On Broadwell GT3 we have 2 Video Command Streamers (VCS), but userspace > >> ha

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-12-08 Thread Rodrigo Vivi
On Tue, Nov 25, 2014 at 5:04 AM, Daniel Vetter wrote: > On Mon, Nov 24, 2014 at 08:29:40AM -0800, Rodrigo Vivi wrote: >> From: Zhipeng Gong >> >> On Broadwell GT3 we have 2 Video Command Streamers (VCS), but userspace >> has no control when using VCS1 or VCS2. This patch introduces a mechanism >>

Re: [Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-11-25 Thread Daniel Vetter
On Mon, Nov 24, 2014 at 08:29:40AM -0800, Rodrigo Vivi wrote: > From: Zhipeng Gong > > On Broadwell GT3 we have 2 Video Command Streamers (VCS), but userspace > has no control when using VCS1 or VCS2. This patch introduces a mechanism > to avoid the default ping-pong mode and use one specific rin

[Intel-gfx] [PATCH 1/7] drm/i915: Specify bsd rings through exec flag

2014-11-24 Thread Rodrigo Vivi
From: Zhipeng Gong On Broadwell GT3 we have 2 Video Command Streamers (VCS), but userspace has no control when using VCS1 or VCS2. This patch introduces a mechanism to avoid the default ping-pong mode and use one specific ring through execution flag. v2: fix whitespace (Rodrigo) v3: remove incor