On Wed, Dec 17, 2014 at 04:45:46PM +0000, Thomas Wood wrote:
> Signed-off-by: Thomas Wood <[email protected]>
> ---
>  lib/igt_core.c | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> diff --git a/lib/igt_core.c b/lib/igt_core.c
> index b98cb41..925d78c 100644
> --- a/lib/igt_core.c
> +++ b/lib/igt_core.c
> @@ -366,6 +366,23 @@ static void low_mem_killer_disable(bool disable)
>  }
>  
>  #define write_log_msg(x) write(STDERR_FILENO, x, strlen(x))
> +static void clear_log(void)
> +{
> +     uint8_t i;
> +
> +     pthread_mutex_lock(&log_buffer_mutex);
> +
> +     i = 0;
> +     do {
> +             free(log_buffer.entries[i]);
> +             log_buffer.entries[i] = NULL;
> +             i++;
> +     } while (i != 0);
> +     log_buffer.start = 0;
> +     log_buffer.end = 0;
> +
> +     pthread_mutex_unlock(&log_buffer_mutex);

Too much work. Just log_buffer.start = log_buffer.end under the mutex.
The freeing of old entries will be done by new writers, or by process
termination.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to