Re: [PATCH] update-linux-headers: Inject VIRTIO_RING_NO_LEGACY in virtio_ring.h

["Michael S. Tsirkin" <mst@redhat.com>]

On Mon, May 11, 2026 at 01:19:13PM +0200, Cédric Le Goater wrote:
> The kernel commit 3c4629b68dbe ("virtio: uapi: avoid usage of libc
> types") changed the virtio_ring.h header and this breaks the build on
> Windows which requires the uintptr_t type to cast from pointer to
> integer.
> 
> Inject '#define VIRTIO_RING_NO_LEGACY' at the top of the synced header
> via the update script after the include guard. This discards the code
> section incompatible with Windows.
> 
> Signed-off-by: Cédric Le Goater <clg@redhat.com>

Acked-by: Michael S. Tsirkin <mst@redhat.com>

> ---
>  scripts/update-linux-headers.sh | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh
> index da367acee790657e33c71c10bc118b085774f45d..3c3a0e9394d277766e613dd5ef4e9c7407ec0033 100755
> --- a/scripts/update-linux-headers.sh
> +++ b/scripts/update-linux-headers.sh
> @@ -104,6 +104,7 @@ cp_portable() {
>          -e 's/__kernel_ulong_t/unsigned long/' \
>          -e 's/struct ethhdr/struct eth_header/' \
>          -e '/\#define _LINUX_ETHTOOL_H/a \\n\#include "net/eth.h"' \
> +        -e '/\#define _LINUX_VIRTIO_RING_H/a \\n\#define VIRTIO_RING_NO_LEGACY' \
>          "$f" > "$to/$header";
>  }
>  
> -- 
> 2.54.0