From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:36723)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <thuth@redhat.com>) id 1ZUEbp-00024J-Io
	for qemu-devel@nongnu.org; Tue, 25 Aug 2015 09:49:54 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <thuth@redhat.com>) id 1ZUEbm-0001EP-9i
	for qemu-devel@nongnu.org; Tue, 25 Aug 2015 09:49:53 -0400
Message-ID: <55DC7277.8020807@redhat.com>
Date: Tue, 25 Aug 2015 15:49:43 +0200
From: Thomas Huth <thuth@redhat.com>
MIME-Version: 1.0
References: <1440417809-21069-1-git-send-email-gwshan@linux.vnet.ibm.com>
	<1440417809-21069-2-git-send-email-gwshan@linux.vnet.ibm.com>
In-Reply-To: <1440417809-21069-2-git-send-email-gwshan@linux.vnet.ibm.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH v6 1/8] scripts: Allow include "stdint.h"
	in virtio headers
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Gavin Shan <gwshan@linux.vnet.ibm.com>, qemu-devel@nongnu.org
Cc: aik@ozlabs.ru, peter.maydell@linaro.org, qemu-ppc@nongnu.org, david@gibson.dropbear.id.au

On 24/08/15 14:03, Gavin Shan wrote:
> This allows to include "stdint.h" in virtio header files. Otherwise,
> scripts/update-linux-headers.sh fails when updating headers from
> Linux 4.2.rc8 kernel. include/uapi/linux/virtio_ring.h starts to
> include "stdint.h" from commit d768f32a ("virtio: Fix typecast of
> pointer in vring_init()").
> 
> Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>
> ---
>  scripts/update-linux-headers.sh | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh
> index f0e830c..2fddf2e 100755
> --- a/scripts/update-linux-headers.sh
> +++ b/scripts/update-linux-headers.sh
> @@ -37,7 +37,8 @@ cp_virtio() {
>          mkdir -p "$to"
>          for f in $virtio; do
>              if
> -                grep '#include' "$f" | grep -v -e 'linux/virtio' \
> +                grep '#include' "$f" | grep -v -e 'stdint' \
> +                                             -e 'linux/virtio' \
>                                               -e 'linux/types' \
>                                               -e 'linux/if_ether' \
>                                               -e 'sys/' \
> 

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