Re: [Xen-devel] [PATCH v2] x86/hyperv: change hv_tlb_flush_ex to fix clang build

[Paul Durrant <pdurrant@xxxxxxxxx>]

On Mon, 23 Dec 2019 at 11:03, Wei Liu <wl@xxxxxxx> wrote:
>
> Clang complains:
>
> In file included from synic.c:15:
> /builds/xen-project/xen/xen/include/asm/guest/hyperv-tlfs.h:900:18: error: 
> field 'hv_vp_set' with variable sized type 'struct hv_vpset' not at the end 
> of a struct or class is a GNU extension 
> [-Werror,-Wgnu-variable-sized-type-not-at-end]
>         struct hv_vpset hv_vp_set;
>                         ^
> 1 error generated.
> /builds/xen-project/xen/xen/Rules.mk:198: recipe for target 'synic.o' failed
> make[6]: *** [synic.o] Error 1
>
> Comment out the last variable size array from hv_tlb_flush_ex to fix
> clang builds.
>
> Signed-off-by: Wei Liu <liuwe@xxxxxxxxxxxxx>
> Cc: Paul Durrant <paul@xxxxxxx>

Reviewed-by: Paul Durrant <paul@xxxxxxx>

> ---
>  xen/include/asm-x86/guest/hyperv-tlfs.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/xen/include/asm-x86/guest/hyperv-tlfs.h 
> b/xen/include/asm-x86/guest/hyperv-tlfs.h
> index fe9fb232d0..5b43f99de8 100644
> --- a/xen/include/asm-x86/guest/hyperv-tlfs.h
> +++ b/xen/include/asm-x86/guest/hyperv-tlfs.h
> @@ -908,12 +908,12 @@ struct hv_tlb_flush {
>         u64 gva_list[];
>  } __packed;
>
> -/* HvFlushVirtualAddressSpaceEx, HvFlushVirtualAddressListEx hypercalls */
> +/* HvFlushVirtualAddressSpaceEx hypercall */
>  struct hv_tlb_flush_ex {
>         u64 address_space;
>         u64 flags;
>         struct hv_vpset hv_vp_set;
> -       u64 gva_list[];
> +       /* u64 gva_list[]; */
>  } __packed;
>
>  struct hv_partition_assist_pg {
> --
> 2.20.1
>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel