[PATCH v2] drm/radeon/kms: remove benchmarks shorter than one page

2012-01-31 Thread Alex Deucher
On Tue, Jan 31, 2012 at 9:27 AM, Ilija Hadzic wrote: > > > On Tue, 31 Jan 2012, Alex Deucher wrote: > >>> >>> Signed-off-by: Ilija Hadzic >> >> >> Reviewed-by: Alex Deucher >> > > Thanks. There will be a v3, though to address one tirvial comment > (whitespace between binary '*' operator) that I

[PATCH v2] drm/radeon/kms: remove benchmarks shorter than one page

2012-01-31 Thread Alex Deucher
On Mon, Jan 30, 2012 at 11:10 PM, Ilija Hadzic wrote: > copy_blit operation works only on integral number of pages > so benchmarks shorter than one page size (4K) do not make sense > > v2: use RADEON_GPU_PAGE_SIZE instead of "magic" 1024 number and > ? ?sweep sizes between 1x to 16x doubling > ?

[PATCH v2] drm/radeon/kms: remove benchmarks shorter than one page

2012-01-31 Thread Ilija Hadzic
On Tue, 31 Jan 2012, Alex Deucher wrote: >> >> Signed-off-by: Ilija Hadzic > > Reviewed-by: Alex Deucher > Thanks. There will be a v3, though to address one tirvial comment (whitespace between binary '*' operator) that I received in E-mail sent directly to me. I guess the diff v2/v3 will

Re: [PATCH v2] drm/radeon/kms: remove benchmarks shorter than one page

2012-01-31 Thread Alex Deucher
On Mon, Jan 30, 2012 at 11:10 PM, Ilija Hadzic ihad...@research.bell-labs.com wrote: copy_blit operation works only on integral number of pages so benchmarks shorter than one page size (4K) do not make sense v2: use RADEON_GPU_PAGE_SIZE instead of magic 1024 number and    sweep sizes between

Re: [PATCH v2] drm/radeon/kms: remove benchmarks shorter than one page

2012-01-31 Thread Ilija Hadzic
On Tue, 31 Jan 2012, Alex Deucher wrote: Signed-off-by: Ilija Hadzic ihad...@research.bell-labs.com Reviewed-by: Alex Deucher alexander.deuc...@amd.com Thanks. There will be a v3, though to address one tirvial comment (whitespace between binary '*' operator) that I received in E-mail

Re: [PATCH v2] drm/radeon/kms: remove benchmarks shorter than one page

2012-01-31 Thread Alex Deucher
On Tue, Jan 31, 2012 at 9:27 AM, Ilija Hadzic ihad...@research.bell-labs.com wrote: On Tue, 31 Jan 2012, Alex Deucher wrote: Signed-off-by: Ilija Hadzic ihad...@research.bell-labs.com Reviewed-by: Alex Deucher alexander.deuc...@amd.com Thanks. There will be a v3, though to address one

[PATCH v2] drm/radeon/kms: remove benchmarks shorter than one page

2012-01-30 Thread Ilija Hadzic
copy_blit operation works only on integral number of pages so benchmarks shorter than one page size (4K) do not make sense v2: use RADEON_GPU_PAGE_SIZE instead of "magic" 1024 number and sweep sizes between 1x to 16x doubling the size in each iteration; we get the same coverage, as in

[PATCH v2] drm/radeon/kms: remove benchmarks shorter than one page

2012-01-30 Thread Ilija Hadzic
copy_blit operation works only on integral number of pages so benchmarks shorter than one page size (4K) do not make sense v2: use RADEON_GPU_PAGE_SIZE instead of magic 1024 number and sweep sizes between 1xpage_size to 16xpage_size doubling the size in each iteration; we get the same