>               else if (sinfo->gso_type & SKB_GSO_TCPV6)
>                       hdr->gso_type = VIRTIO_NET_HDR_GSO_TCPV6;
> -             else
> -                     return -EINVAL;
> +             else {
> +                     if (skb->data_len == 0)
> +                             hdr->gso_type = VIRTIO_NET_HDR_GSO_NONE;


maybe use "else if" like in the before? yes, it's a different type of
condition, but braces look a bit unnatural here to me at least

johannes


_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

Reply via email to