Re: [Intel-gfx] [PATCH] drm/i915: Batch copy_from_user for relocation processing

2012-03-26 Thread Daniel Vetter
On Sat, Mar 24, 2012 at 08:12:53PM +, Chris Wilson wrote: > Originally the code tried to allocate a large enough array to perform > the copy using vmalloc, performance wasn't great and throughput was > improved by processing each individual relocation entry separately. > This too is not as effi

[Intel-gfx] [PATCH] drm/i915: Batch copy_from_user for relocation processing

2012-03-24 Thread Chris Wilson
Originally the code tried to allocate a large enough array to perform the copy using vmalloc, performance wasn't great and throughput was improved by processing each individual relocation entry separately. This too is not as efficient as one would desire. A compromise would be to allocate a single

[Intel-gfx] [PATCH] drm/i915: Batch copy_from_user for relocation processing

2012-03-24 Thread Chris Wilson
Originally the code tried to allocate a large enough array to perform the copy using vmalloc, performance wasn't great and throughput was improved by processing each individual relocation entry separately. This too is not as efficient as one would desire. A compromise would be to allocate a single

Re: [Intel-gfx] [PATCH] drm/i915: Batch copy_from_user for relocation processing

2012-03-24 Thread Daniel Vetter
On Sat, Mar 24, 2012 at 12:20:24AM +, Chris Wilson wrote: > Originally the code tried to allocate a large enough array to perform > the copy using vmalloc, performance wasn't great and throughput was > improved by processing each individual relocation entry separately. > This too is not as effi

[Intel-gfx] [PATCH] drm/i915: Batch copy_from_user for relocation processing

2012-03-23 Thread Chris Wilson
Originally the code tried to allocate a large enough array to perform the copy using vmalloc, performance wasn't great and throughput was improved by processing each individual relocation entry separately. This too is not as efficient as one would desire. A compromise is then to allocate a single p