[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