On 07/09/2012 11:18 AM, Kenneth Graunke wrote: > In one workload, 45% of CPU time was spent in pwrite(), primarily > uploading large amounts of VBO data. This was in fact the hottest path > in the application, according to sysprof. Removing the system memory > buffer and just storing VBOs in actual BOs cut the application's CPU > usage by about half, and moved pwrite down to about 2.5% of CPU time. > > Signed-off-by: Kenneth Graunke <kenn...@whitecape.org> > --- > src/mesa/drivers/dri/intel/intel_buffer_objects.c | 117 > +-------------------- > src/mesa/drivers/dri/intel/intel_buffer_objects.h | 3 - > 2 files changed, 4 insertions(+), 116 deletions(-) > > Here's my version of the same patch. It's quite a bit more invasive, > but it deletes all the code made dead by the change in Eric's patch.
Eric rightly pointed out that this code is still needed by i915. So, NAK on my patch 1/2, and Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> on Eric's patch. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
