On Fri, 21 Sep 2018 18:20:35 +0000
Haiyang Zhang <haiya...@linuxonhyperv.com> wrote:

Overall, this looks good. Some minor suggestions.

> +struct nvsc_rsc {
> +     const struct ndis_pkt_8021q_info *vlan;
> +     const struct ndis_tcp_ip_checksum_info *csum_info;
> +     u8 is_last; /* last RNDIS msg in a vmtransfer_page */
> +     u32 cnt; /* #fragments in an RSC packet */
> +     u32 pktlen; /* Full packet length */
> +     void *data[NVSP_RSC_MAX];
> +     u32 len[NVSP_RSC_MAX];
> +};
> +

This new state structure is state on a per-channel basis.
Do you really need this to be persistent across packets?

Could this be on stack or do you need it to handle split packets
arriving in different polls? Or is the stack space a problem?

Also, maybe data and length could be in one structure since they
are related.
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to