Re: [Mesa-dev] [PATCH 2/4] i965: Use I915_EXEC_NO_RELOC

2017-07-07 Thread Chris Wilson
Quoting Daniel Vetter (2017-07-07 11:04:00) > On Mon, Jun 19, 2017 at 11:06:48AM +0100, Chris Wilson wrote: > > - if (target != batch->bo) > > - add_exec_bo(batch, target); > > + if (target != batch->bo) { > > + unsigned int index = add_exec_bo(batch, target); > > + struct drm_i9

Re: [Mesa-dev] [PATCH 2/4] i965: Use I915_EXEC_NO_RELOC

2017-07-07 Thread Daniel Vetter
On Mon, Jun 19, 2017 at 11:06:48AM +0100, Chris Wilson wrote: > If we correctly fill the batch with the right relocation value, and that > matches the expected location of the object, we can then tell the kernel > it can forgo checking each individual relocation by only checking > whether the objec

Re: [Mesa-dev] [PATCH 2/4] i965: Use I915_EXEC_NO_RELOC

2017-07-05 Thread Daniel Vetter
On Mon, Jun 19, 2017 at 08:53:33PM +0100, Chris Wilson wrote: > Quoting Kenneth Graunke (2017-06-19 20:28:31) > > On Monday, June 19, 2017 3:06:48 AM PDT Chris Wilson wrote: > > > - if (target != batch->bo) > > > - add_exec_bo(batch, target); > > > + if (target != batch->bo) { > > > +

Re: [Mesa-dev] [PATCH 2/4] i965: Use I915_EXEC_NO_RELOC

2017-06-19 Thread Chris Wilson
Quoting Jason Ekstrand (2017-06-19 22:00:45) > On Mon, Jun 19, 2017 at 12:53 PM, Chris Wilson > wrote: > > Quoting Kenneth Graunke (2017-06-19 20:28:31) > > On Monday, June 19, 2017 3:06:48 AM PDT Chris Wilson wrote: > > > -   if (target != batch->bo) > > > -      add_exec_bo(batc

Re: [Mesa-dev] [PATCH 2/4] i965: Use I915_EXEC_NO_RELOC

2017-06-19 Thread Jason Ekstrand
On Mon, Jun 19, 2017 at 12:53 PM, Chris Wilson wrote: > Quoting Kenneth Graunke (2017-06-19 20:28:31) > > On Monday, June 19, 2017 3:06:48 AM PDT Chris Wilson wrote: > > > - if (target != batch->bo) > > > - add_exec_bo(batch, target); > > > + if (target != batch->bo) { > > > + unsig

Re: [Mesa-dev] [PATCH 2/4] i965: Use I915_EXEC_NO_RELOC

2017-06-19 Thread Chris Wilson
Quoting Kenneth Graunke (2017-06-19 20:28:31) > On Monday, June 19, 2017 3:06:48 AM PDT Chris Wilson wrote: > > - if (target != batch->bo) > > - add_exec_bo(batch, target); > > + if (target != batch->bo) { > > + unsigned int index = add_exec_bo(batch, target); > > + struct drm_i9

Re: [Mesa-dev] [PATCH 2/4] i965: Use I915_EXEC_NO_RELOC

2017-06-19 Thread Kenneth Graunke
On Monday, June 19, 2017 3:06:48 AM PDT Chris Wilson wrote: > If we correctly fill the batch with the right relocation value, and that > matches the expected location of the object, we can then tell the kernel > it can forgo checking each individual relocation by only checking > whether the object

[Mesa-dev] [PATCH 2/4] i965: Use I915_EXEC_NO_RELOC

2017-06-19 Thread Chris Wilson
If we correctly fill the batch with the right relocation value, and that matches the expected location of the object, we can then tell the kernel it can forgo checking each individual relocation by only checking whether the object moved. v2: Rebase to apply ahead of I915_EXEC_HANDLE_LUT Signed-of