Re: [PATCH 0/5] Finish gen6/7 ppgtt dynamic page allocations

[Mika Kuoppala <mika.kuoppala@linux.intel.com>]

Daniel Vetter <daniel@ffwll.ch> writes:

> On Thu, Mar 26, 2015 at 05:43:51PM +0200, Mika Kuoppala wrote:
>> Michel Thierry <michel.thierry@intel.com> writes:
>> 
>> > The first 2 patches are fixes from the previous patchset, reported by static
>> > analysis tools, while the last 2 patches complete the required work for gen6/7.
>> >
>> > I've also started changing the authorship of the patches as suggested by Daniel.
>> >
>> > Michel Thierry (5):
>> >   drm/i915: Prevent out of range pt in gen6_for_each_pde
>> >   drm/i915: i915_dma_map_single returns positive error codes
>> >   drm/i915: Remove unnecessary gen6_ppgtt_unmap_pages
>> >   drm/i915: Finish gen6/7 dynamic page table allocation
>> >   drm/i915: Add dynamic page trace events
>> >
>> 
>> Daniel,
>> 
>> We have tossed the Finish gen6/7 one around in irc quite a lot.
>> 
>> Patches 1, 2v3, 3, 4, 5:
>> Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>
>
> All merged, thanks for patches&review.
>
> btw since you're having all these discussions: Are you taking notes for
> some nice kerneldoc polish on top of all these patches?
>

I have something saved yes. However mostly they are irc logs where
I am chasing my own tail in a strong coffee induced stupor.

But after the gen8 parts, including 48bit have landed, we should
see if any naming/cleanups are needed and after that do
the kerneldoc polish.

I feel like I have overused my naming rebase quota already at
this stage.

- Mika

> Thanks, Daniel
>> 
>> >  drivers/gpu/drm/i915/i915_gem.c     |   2 +
>> >  drivers/gpu/drm/i915/i915_gem_gtt.c | 164 +++++++++++++++++++++++++++++-------
>> >  drivers/gpu/drm/i915/i915_gem_gtt.h |   8 +-
>> >  drivers/gpu/drm/i915/i915_trace.h   |  99 ++++++++++++++++++++++
>> >  4 files changed, 239 insertions(+), 34 deletions(-)
>> >
>> > -- 
>> > 2.1.1
>> >
>> > _______________________________________________
>> > Intel-gfx mailing list
>> > Intel-gfx@lists.freedesktop.org
>> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
> -- 
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx