Re: [PATCH v3 4/4] drm/i915: Use partial view in mmap fault handler

From: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
To: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
	intel-gfx <intel-gfx@lists.freedesktop.org>
Subject: Re: [PATCH v3 4/4] drm/i915: Use partial view in mmap fault handler
Date: Wed, 06 May 2015 13:33:33 +0100	[thread overview]
Message-ID: <554A0A1D.8040901@linux.intel.com> (raw)
In-Reply-To: <1430912169.9311.9.camel@jlahtine-mobl1>


On 05/06/2015 12:36 PM, Joonas Lahtinen wrote:
>
> Use partial view for huge BOs (bigger than half the mappable aperture)
> in fault handler so that they can be accessed withough trying to make
> room for them by evicting other objects.
>
> v2:
> - Only use partial views in the case where early rejection was
>    previously done.
> - Account variable type changes from previous reroll.
> v3:
> - Add a comment about overwriting existing page entries.
>    (Tvrtko Ursulin)
> - Whitespace fixes.

Again I wish two vm_insert_pfn loops could be unified by making partial 
view a special case of normal, but as this is another area where 
imminent change has been hinted, and it doesn't matter hugely anyway:

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

Regards,

Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx