Re: [PATCH igt 12/24] igt/gem_ringfill: Prime execbuf before measuring ring size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 14, 2017 at 09:18:36PM +0100, Chris Wilson wrote:
> Make sure that our buffers are ready and loaded to reduce the initial
> stall.

Reviewed-by: Michał Winiarski <michal.winiarski@xxxxxxxxx>

Already looked at this one before:
https://lists.freedesktop.org/archives/intel-gfx/2017-July/134344.html
(as well as couple of other patches in this series)

-Michał

> ---
>  tests/gem_ringfill.c | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/gem_ringfill.c b/tests/gem_ringfill.c
> index b52996a4..01cbd0a9 100644
> --- a/tests/gem_ringfill.c
> +++ b/tests/gem_ringfill.c
> @@ -288,10 +288,15 @@ static unsigned int measure_ring_size(int fd)
>  	obj[1].handle = gem_create(fd, 4096);
>  	gem_write(fd, obj[1].handle, 0, &bbe, sizeof(bbe));
>  
> +	memset(&execbuf, 0, sizeof(execbuf));
> +	execbuf.buffers_ptr = to_user_pointer(obj + 1);
> +	execbuf.buffer_count = 1;
> +	gem_execbuf(fd, &execbuf);
> +	gem_sync(fd, obj[1].handle);
> +
>  	plug(fd, &c);
>  	obj[0].handle = c.handle;
>  
> -	memset(&execbuf, 0, sizeof(execbuf));
>  	execbuf.buffers_ptr = to_user_pointer(obj);
>  	execbuf.buffer_count = 2;
>  
> @@ -302,7 +307,8 @@ static unsigned int measure_ring_size(int fd)
>  	itv.it_value.tv_usec = 1000;
>  	setitimer(ITIMER_REAL, &itv, NULL);
>  
> -	last = count = 0;
> +	last = -1;
> +	count = 0;
>  	do {
>  		if (__execbuf(fd, &execbuf) == 0) {
>  			count++;
> -- 
> 2.13.3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux