To do it the proper way yes. I really hate to do it this way but given the timeframe for 2.4 we could nuke it later on.
Alin. -----Mesaj original----- De la: Nithin Raju [mailto:nit...@vmware.com] Trimis: Friday, June 19, 2015 7:36 PM Către: Alin Serdean Cc: dev@openvswitch.org Subiect: Re: [ovs-dev] [PATCH v2] dpif_probe_feature not suported in windows datapath hi Alin, Do you know what is required to complete the implementation of dpif_probe_feature() in the Windows datapath? Is it the flow validation logic? thanks, -- Nithin > On Jun 18, 2015, at 6:47 PM, Alin Serdean <aserd...@cloudbasesolutions.com> > wrote: > > This patch disables features which are not currently supported in the > windows datapath. > > Unfortunately we have to do it in userspace because dpif_probe_feature > is not treated accordingly in the windows the datapath. > > I opened the issue to track the feature for later implementations: > https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_openvs > witch_ovs-2Dissues_issues_85&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw- > YihVMNtXt-uEs&r=pNHQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=H9ymRGwE > SDpjH0qHp5EifPK6QimZgzvHEIyjM3QzRwg&s=-rOo7BG2BoxcGi2BlSQASDz8v5Hkhp4y > MoGbPZ-zQus&e= > --- > v2: Rebase > --- > ofproto/ofproto-dpif.c | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index > 0de8686..55d004e 100644 > --- a/ofproto/ofproto-dpif.c > +++ b/ofproto/ofproto-dpif.c > @@ -1014,8 +1014,14 @@ check_recirc(struct dpif_backer *backer) > > ofpbuf_use_stack(&key, &keybuf, sizeof keybuf); > odp_flow_key_from_flow(&odp_parms, &key); > +#ifdef _WIN32 > + /* XXX Force disable of datapath recirculation from userspace until the > + * dpif_probe_feature is properly implemented in the windows datapath */ > + enable_recirc = false; > +#else > enable_recirc = dpif_probe_feature(backer->dpif, "recirculation", &key, > NULL); > +#endif > > if (enable_recirc) { > VLOG_INFO("%s: Datapath supports recirculation", @@ -1052,7 > +1058,13 @@ check_ufid(struct dpif_backer *backer) > odp_flow_key_from_flow(&odp_parms, &key); > dpif_flow_hash(backer->dpif, key.data, key.size, &ufid); > > +#ifdef _WIN32 > + /* XXX Force disable of datapath recirculation from userspace until the > + * dpif_probe_feature is properly implemented in the windows datapath */ > + enable_ufid = false; > +#else > enable_ufid = dpif_probe_feature(backer->dpif, "UFID", &key, > &ufid); > +#endif > > if (enable_ufid) { > VLOG_INFO("%s: Datapath supports unique flow ids", @@ -1161,6 > +1173,11 @@ check_max_mpls_depth(struct dpif_backer *backer) > > ofpbuf_use_stack(&key, &keybuf, sizeof keybuf); > odp_flow_key_from_flow(&odp_parms, &key); > +#ifdef _WIN32 > + /* XXX Force disable of datapath recirculation from userspace until > the > + * dpif_probe_feature is properly implemented in the windows > datapath */ > + break; > +#endif > if (!dpif_probe_feature(backer->dpif, "MPLS", &key, NULL)) { > break; > } > -- > 1.9.5.msysgit.0 > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_ma > ilman_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt- > uEs&r=pNHQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=H9ymRGwESDpjH0qHp5 > EifPK6QimZgzvHEIyjM3QzRwg&s=voVGofZZJTP055uwJnpQBjFDxPbFyaiHEtEtt140MO > E&e= _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
