Re: [Intel-gfx] [PATCH v3] drm/i915: Eliminate vmap overhead for cmd parser

2015-11-25 Thread Ville Syrjälä
On Wed, Nov 25, 2015 at 08:13:43PM +, Chris Wilson wrote: > On Wed, Nov 25, 2015 at 09:51:08PM +0200, Ville Syrjälä wrote: > > On Fri, Nov 20, 2015 at 03:31:23PM +, Chris Wilson wrote: > > > @@ -1097,6 +1003,7 @@ static bool check_cmd(const struct intel_engine_cs > > > *ring, > > > } > >

Re: [Intel-gfx] [PATCH v3] drm/i915: Eliminate vmap overhead for cmd parser

2015-11-25 Thread Chris Wilson
On Wed, Nov 25, 2015 at 09:51:08PM +0200, Ville Syrjälä wrote: > On Fri, Nov 20, 2015 at 03:31:23PM +, Chris Wilson wrote: > > @@ -1097,6 +1003,7 @@ static bool check_cmd(const struct intel_engine_cs > > *ring, > > } > > > > #define LENGTH_BIAS 2 > > +#define MAX_PARTIAL 256 > > There see

Re: [Intel-gfx] [PATCH v3] drm/i915: Eliminate vmap overhead for cmd parser

2015-11-25 Thread Ville Syrjälä
On Fri, Nov 20, 2015 at 03:31:23PM +, Chris Wilson wrote: > With a little complexity to handle cmds straddling page boundaries, we > can completely avoiding having to vmap the batch and the shadow batch > objects whilst running the command parser. > > On ivb i7-3720MQ: > > x11perf -dot before

[Intel-gfx] [PATCH v3] drm/i915: Eliminate vmap overhead for cmd parser

2015-11-20 Thread Chris Wilson
With a little complexity to handle cmds straddling page boundaries, we can completely avoiding having to vmap the batch and the shadow batch objects whilst running the command parser. On ivb i7-3720MQ: x11perf -dot before 54.3M, after 53.2M (max 203M) glxgears before 7110 fps, after 7300 fps (max