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_openvswitch_ovs-2Dissues_issues_85&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=pNHQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=H9ymRGwESDpjH0qHp5EifPK6QimZgzvHEIyjM3QzRwg&s=-rOo7BG2BoxcGi2BlSQASDz8v5Hkhp4yMoGbPZ-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_mailman_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=pNHQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=H9ymRGwESDpjH0qHp5EifPK6QimZgzvHEIyjM3QzRwg&s=voVGofZZJTP055uwJnpQBjFDxPbFyaiHEtEtt140MOE&e= > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
