From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Huth <thuth@redhat.com>
Subject: Re: [kvm-unit-tests PATCH v1] lib: fix compilation warning
Date: Fri, 12 May 2017 14:22:52 +0200
Message-ID: <46d2a449-f61d-331d-2d18-4cf565236d67@redhat.com>
References: <20170512104723.9469-1-david@redhat.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Cc: Paolo Bonzini <pbonzini@redhat.com>,
        =?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= <rkrcmar@redhat.com>,
        David Matlack <dmatlack@google.com>
To: David Hildenbrand <david@redhat.com>, kvm@vger.kernel.org
Return-path: <kvm-owner@vger.kernel.org>
Received: from mx1.redhat.com ([209.132.183.28]:33814 "EHLO mx1.redhat.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1755400AbdELMXD (ORCPT <rfc822;kvm@vger.kernel.org>);
        Fri, 12 May 2017 08:23:03 -0400
In-Reply-To: <20170512104723.9469-1-david@redhat.com>
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

On 12.05.2017 12:47, David Hildenbrand wrote:
> GCC doesn't like comparison of signed and unsigned values.
> 
> Signed-off-by: David Hildenbrand <david@redhat.com>
> ---
>  lib/report.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/report.c b/lib/report.c
> index aa23b65..f7af596 100644
> --- a/lib/report.c
> +++ b/lib/report.c
> @@ -29,7 +29,7 @@ void report_pass(void)
>  void report_prefix_pushf(const char *prefix_fmt, ...)
>  {
>  	va_list va;
> -	int len;
> +	unsigned int len;
>  	int start;
>  
>  	spin_lock(&lock);

This fixes the problem for me, too:

Tested-by: Thomas Huth <thuth@redhat.com>