> On Aug 4, 2015, at 1:45 AM, Sorin Vinturis <svintu...@cloudbasesolutions.com>
> wrote:
>
> Nithin, that check (if (nbl->SourceHandle == context->ovsPool.ndisHandle) ..)
> at L564 was added by my patch. Please see here:
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_openvswitch_ovs_commit_d50b88bf830c401636085ecd5048e929c80fd4d2&d=BQIFAg&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=pNHQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=K8LEl4falwGybwMNhkjlfV6n2yOWVSgOrlRCDE698NU&s=SpfcoxkTPEFep3bMt-EGs375BRou-fDDT7BwtEvrvx0&e=
>
yes, I agree. It was good catch.
My question is, do you need the additional patch you are proposing in this
patch. I’ve copy pasted it for convenience below:
> + ctx = (POVS_BUFFER_CONTEXT)NET_BUFFER_LIST_CONTEXT_DATA_START(nbl);
> + if (ctx && ctx->magic == OVS_CTX_MAGIC) {
> + OVS_LOG_INFO("nbl context already initialized/allocated by OVS.");
> + return ctx;
> + }
thanks,
-- Ntihin
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
