[ovs-dev] [PATCH v3] datapath-windows: Add support to configure ct zone limits
Anand Kumar
kumaranand at vmware.com
Wed Sep 19 18:42:28 UTC 2018
Hi Ben,
Thanks for the review. I think this bug got introduced when addressing review comments.
I have fixed it and sent out a v4.
Regards,
Anand Kumar
On 9/18/18, 1:28 AM, "Ben Pfaff" <blp at ovn.org> wrote:
I didn't really review this but a glance at it showed one possible
issue.
Missing () around definition here:
> +#define CT_MAX_ZONE UINT16_MAX + 1
Bang?
> + zoneInfo = OvsAllocateMemoryWithTag(sizeof(OVS_CT_ZONE_INFO) *
> + CT_MAX_ZONE, OVS_CT_POOL_TAG);
Thanks,
Ben.
More information about the dev
mailing list