On 09/13/2018 01:45 PM, Ian Stokes wrote:
> On 9/13/2018 1:34 PM, Ophir Munk wrote:
>> Hi Sugesh,
>> I can update documentation. Is it enough to update dpdk.rst.txt only?
>> If not - can you please send a link to additional files with
>> references to 17.11 to be updated?
>> I will wait 1-2 days to get more reviews then will send v3.
>
> Hi Ophir, I've just sent a patch to update master branch to use DPDK
> 17.11.4, you can follow the same docs that are modified in that patch
> for the v3 of this. Docs modified and link to the patch itself below.
>
> .travis/linux-build.sh | 2 +-
> Documentation/faq/releases.rst | 6 +++---
> Documentation/intro/install/dpdk.rst | 8 ++++----
> Documentation/topics/dpdk/vhost-user.rst | 6 +++---
>
> https://mail.openvswitch.org/pipermail/ovs-dev/2018-September/352104.html
>
> Thanks
> Ian
>>
>> Regards,
>> Ophir
>>
>>> -----Original Message-----
>>> From: Chandran, Sugesh [mailto:[email protected]]
>>> Sent: Thursday, September 13, 2018 2:58 PM
>>> To: Ophir Munk <[email protected]>; '[email protected]'
>>> <[email protected]>
>>> Cc: Asaf Penso <[email protected]>; Stokes, Ian
>>> <[email protected]>; 'Ben Pfaff' <[email protected]>; Shahaf Shuler
>>> <[email protected]>; Thomas Monjalon <[email protected]>;
>>> Olga Shern <[email protected]>; 'Kevin Traynor' <[email protected]>
>>> Subject: RE: [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk v18.08.0
>>>
>>> Hi Ophir,
>>>
>>> Also a quick grep show there are number of places in documentation still
>>> pointing to DPDK17.11, I guess its worth while to change them as well.
>>> This will avoid confusion for people who wanted to use this branch,
>>> What do
>>> you think?
>>>
>>>
>>>
>>> Regards
>>> _Sugesh
>>>
>>>
>>>> -----Original Message-----
>>>> From: Chandran, Sugesh
>>>> Sent: Wednesday, September 12, 2018 5:05 PM
>>>> To: Ophir Munk <[email protected]>; [email protected]
>>>> Cc: Asaf Penso <[email protected]>; Stokes, Ian
>>>> <[email protected]>; Ben Pfaff <[email protected]>; Shahaf Shuler
>>>> <[email protected]>; Thomas Monjalon <[email protected]>;
>>> Olga
>>>> Shern <[email protected]>; Kevin Traynor <[email protected]>
>>>> Subject: RE: [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk
>>>> v18.08.0
>>>>
>>>> Hi Ophir,
>>>> I verified this patch in our test lab and looks fine to me.
>>>> I have couple of minor comments in the V1 series. Could you please
>>>> take care of them as well?
>>>>
>>>>
>>>> Regards
>>>> _Sugesh
>>>>
>>>>> -----Original Message-----
>>>>> From: Ophir Munk [mailto:[email protected]]
>>>>> Sent: Tuesday, September 11, 2018 12:05 AM
>>>>> To: [email protected]
>>>>> Cc: Asaf Penso <[email protected]>; Chandran, Sugesh
>>>>> <[email protected]>; Stokes, Ian <[email protected]>; Ben
>>>>> Pfaff <[email protected]>; Shahaf Shuler <[email protected]>; Thomas
>>>>> Monjalon <[email protected]>; Olga Shern <[email protected]>;
>>>> Ophir
>>>>> Munk <[email protected]>; Kevin Traynor
>>> <[email protected]>
>>>>> Subject: [dpdk-howl PATCH v2] netdev-dpdk: Upgrade to dpdk v18.08.0
>>>>>
>>>>> 1. Enable compilation and linkage with dpdk 18.08.0 The following
>>>>> dpdk commits which were introduced after dpdk 17.11.x require OVS
>>>>> updates to accommodate to the dpdk changes.
>>>>> - ce17eddefc20 ("ethdev: introduce Rx queue offloads API")
>>>>> - ab3ce1e0c193 ("ethdev: remove old offload API")
>>>>> - c06ddf9698e0 ("meter: add configuration profile")
>>>>> - e58638c32411 ("ethdev: fix TPID handling in flow API")
>>>>> - cd8c7c7ce241 ("ethdev: replace bus specific struct with generic
>>>>> dev")
>>>>> - ac8d22de2394 ("ethdev: flatten RSS configuration in flow API")
>>>>>
>>>>> 2. Limit configured rss hash functions to only those supported by
>>>>> the eth
>>>> device.
>>>>>
>>>>> 3. Update references to DPDK version in Documentation
>>>>>
>>>>> 4. Update DPDK version in travis' linux-build script
>>>>>
>>>>> Signed-off-by: Ophir Munk <[email protected]>
>>>>> ---
>>>>> v1:
>>>>> First version
>>>>>
>>>>> v2:
>>>>> Avoid seg faults cases as described in
>>>>>
>>> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fp
>>>>>
>>> atchwork.ozlabs.org%2Fpatch%2F965451%2F&data=02%7C01%7Cophi
>>> rmu%4
>>>>>
>>> 0mellanox.com%7Ccc4dcac1d427416c8dc508d619703178%7Ca652971c7d2
>>> e4d9ba
>>>>>
>>> 6a4d149256f461b%7C0%7C0%7C636724366985653825&sdata=CFZjVD
>>> Rd6w0TP
>>>>> x7lKWOkUVEh28aoyrpl%2Fop635Vrlx0%3D&reserved=0
>>>>> by using the patch in:
>>>>>
>>> https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fg
>>>>> ithub.com%2Fkevintraynor%2Fovs-dpdk-
>>> &data=02%7C01%7Cophirmu%40me
>>>>>
>>> llanox.com%7Ccc4dcac1d427416c8dc508d619703178%7Ca652971c7d2e4d9
>>> ba6a4
>>>>>
>>> d149256f461b%7C0%7C0%7C636724366985653825&sdata=%2F2B4uH
>>> HAHGma%2
>>>>> F48VS2I3QsMm%2ByO3LMZdaHlxVKJ9JYg%3D&reserved=0
>>>>> master/commit/88f46cc5ab338eb4f3ca5db1eacd0effefe4fa0c
>>>>>
>>>>> .travis/linux-build.sh | 2 +-
>>>>> Documentation/faq/releases.rst | 2 +-
>>>>> lib/netdev-dpdk.c | 142
>>>>> +++++++++++++++++++++++++++++-------
>>> -----
>>>>> 3 files changed, 104 insertions(+), 42 deletions(-)
>>>>>
>>>>> diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh index
>>>>> 4b9fc4a..c60ac71
>>>>> 100755
>>>>> --- a/.travis/linux-build.sh
>>>>> +++ b/.travis/linux-build.sh
>>>>> @@ -83,7 +83,7 @@ fi
>>>>>
>>>>> if [ "$DPDK" ]; then
>>>>> if [ -z "$DPDK_VER" ]; then
>>>>> - DPDK_VER="17.11.3"
>>>>> + DPDK_VER="18.08.0"
>>>>> fi
>>>>> install_dpdk $DPDK_VER
>>>>> if [ "$CC" = "clang" ]; then
>>>>> diff --git a/Documentation/faq/releases.rst
>>>>> b/Documentation/faq/releases.rst index 41d41e3..646ae09 100644
>>>>> --- a/Documentation/faq/releases.rst
>>>>> +++ b/Documentation/faq/releases.rst
>>>>> @@ -168,7 +168,7 @@ Q: What DPDK version does each Open vSwitch
>>>>> release work with?
>>>>> 2.7.x 16.11.7
>>>>> 2.8.x 17.05.2
>>>>> 2.9.x 17.11.3
>>>>> - 2.10.x 17.11.3
>>>>> + 2.10.x 18.08.0
Hi - on updating versions in docs, this one should not change as the
released/to be released versions of OVS 2.10.x stick with 17.11.x. We
can just leave this unmodified for now and add an entry for 2.11.x
closer to the release time.
Kevin.
>>>>> ============ =======
>>>>>
>>>>> Q: Are all the DPDK releases that OVS versions work with maintained?
>>>>> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index
>>>>> f91aa27..6879e9a
>>>>> 100644
>>>>> --- a/lib/netdev-dpdk.c
>>>>> +++ b/lib/netdev-dpdk.c
>>>>> @@ -168,11 +168,7 @@ static const struct rte_eth_conf port_conf = {
>>>>> .rxmode = {
>>>>> .mq_mode = ETH_MQ_RX_RSS,
>>>>> .split_hdr_size = 0,
>>>>> - .header_split = 0, /* Header Split disabled */
>>>>> - .hw_ip_checksum = 0, /* IP checksum offload disabled */
>>>>> - .hw_vlan_filter = 0, /* VLAN filtering disabled */
>>>>> - .jumbo_frame = 0, /* Jumbo Frame Support disabled */
>>>>> - .hw_strip_crc = 0,
>>>>> + .offloads = 0,
>>>>> },
>>>>> .rx_adv_conf = {
>>>>> .rss_conf = {
>>>>> @@ -364,6 +360,7 @@ struct dpdk_ring { struct ingress_policer {
>>>>> struct rte_meter_srtcm_params app_srtcm_params;
>>>>> struct rte_meter_srtcm in_policer;
>>>>> + struct rte_meter_srtcm_profile in_prof;
>>>>> rte_spinlock_t policer_lock;
>>>>> };
>>>>>
>>>>> @@ -894,6 +891,8 @@ dpdk_eth_dev_port_config(struct netdev_dpdk
>>>>> *dev, int n_rxq, int n_txq)
>>>>> struct rte_eth_dev_info info;
>>>>> uint16_t conf_mtu;
>>>>>
>>>>> + rte_eth_dev_info_get(dev->port_id, &info);
>>>>> +
>>>>> /* As of DPDK 17.11.1 a few PMDs require to explicitly enable
>>>>> * scatter to support jumbo RX. Checking the offload
>>>>> capabilities
>>>>> * is not an option as PMDs are not required yet to report @@
>>>>> -901,20
>>>>> +900,24 @@ dpdk_eth_dev_port_config(struct netdev_dpdk *dev, int
>>>>> +n_rxq, int
>>>>> n_txq)
>>>>> * (testing or code review). Listing all such PMDs feels harder
>>>>> * than highlighting the one known not to need scatter */
>>>>> if (dev->mtu > ETHER_MTU) {
>>>>> - rte_eth_dev_info_get(dev->port_id, &info);
>>>>> if (strncmp(info.driver_name, "net_nfp", 7)) {
>>>>> - conf.rxmode.enable_scatter = 1;
>>>>> + conf.rxmode.offloads |= DEV_RX_OFFLOAD_SCATTER;
>>>>> }
>>>>> }
>>>>>
>>>>> conf.intr_conf.lsc = dev->lsc_interrupt_mode;
>>>>> - conf.rxmode.hw_ip_checksum = (dev->hw_ol_features &
>>>>> - NETDEV_RX_CHECKSUM_OFFLOAD) != 0;
>>>>> + conf.rxmode.offloads |= ((dev->hw_ol_features &
>>>>> + NETDEV_RX_CHECKSUM_OFFLOAD) != 0) ?
>>>>> + DEV_RX_OFFLOAD_CHECKSUM : 0;
>>>>>
>>>>> if (dev->hw_ol_features & NETDEV_RX_HW_CRC_STRIP) {
>>>>> - conf.rxmode.hw_strip_crc = 1;
>>>>> + conf.rxmode.offloads |= DEV_RX_OFFLOAD_CRC_STRIP;
>>>>> }
>>>>>
>>>>> + /* Limit configured rss hash functions to only those supported
>>>>> + * by the eth device. */
>>>>> + conf.rx_adv_conf.rss_conf.rss_hf &=
>>>>> + info.flow_type_rss_offloads;
>>>>> +
>>>>> /* A device may report more queues than it makes available
>>>>> (this has
>>>>> * been observed for Intel xl710, which reserves some of them
>>>>> for
>>>>> * SRIOV): rte_eth_*_queue_setup will fail if a queue is not
>>>>> @@
>>>>> -1932,16
>>>>> +1935,18 @@ netdev_dpdk_eth_tx_burst(struct netdev_dpdk *dev, int
>>>>> +qid,
>>>>>
>>>>> static inline bool
>>>>> netdev_dpdk_policer_pkt_handle(struct rte_meter_srtcm *meter,
>>>>> + struct rte_meter_srtcm_profile
>>>>> + *profile,
>>>>> struct rte_mbuf *pkt, uint64_t
>>>>> time) {
>>>>> uint32_t pkt_len = rte_pktmbuf_pkt_len(pkt) - sizeof(struct
>>>>> ether_hdr);
>>>>>
>>>>> - return rte_meter_srtcm_color_blind_check(meter, time, pkt_len) ==
>>>>> + return rte_meter_srtcm_color_blind_check(meter, profile, time,
>>>>> + pkt_len) ==
>>>>> e_RTE_METER_GREEN;
>>>>> }
>>>>>
>>>>> static int
>>>>> netdev_dpdk_policer_run(struct rte_meter_srtcm *meter,
>>>>> + struct rte_meter_srtcm_profile *profile,
>>>>> struct rte_mbuf **pkts, int pkt_cnt,
>>>>> bool should_steal) { @@ -1953,7 +1958,8 @@
>>>>> netdev_dpdk_policer_run(struct rte_meter_srtcm *meter,
>>>>> for (i = 0; i < pkt_cnt; i++) {
>>>>> pkt = pkts[i];
>>>>> /* Handle current packet */
>>>>> - if (netdev_dpdk_policer_pkt_handle(meter, pkt,
>>>>> current_time)) {
>>>>> + if (netdev_dpdk_policer_pkt_handle(meter, profile, pkt,
>>>>> + current_time)) {
>>>>> if (cnt != i) {
>>>>> pkts[cnt] = pkt;
>>>>> }
>>>>> @@ -1975,8 +1981,8 @@ ingress_policer_run(struct ingress_policer
>>>>> *policer, struct rte_mbuf **pkts,
>>>>> int cnt = 0;
>>>>>
>>>>> rte_spinlock_lock(&policer->policer_lock);
>>>>> - cnt = netdev_dpdk_policer_run(&policer->in_policer, pkts,
>>>>> - pkt_cnt, should_steal);
>>>>> + cnt = netdev_dpdk_policer_run(&policer->in_policer, &policer-
>>>> in_prof,
>>>>> + pkts, pkt_cnt, should_steal);
>>>>> rte_spinlock_unlock(&policer->policer_lock);
>>>>>
>>>>> return cnt;
>>>>> @@ -2767,8 +2773,15 @@ netdev_dpdk_policer_construct(uint32_t
>>> rate,
>>>>> uint32_t burst)
>>>>> policer->app_srtcm_params.cir = rate_bytes;
>>>>> policer->app_srtcm_params.cbs = burst_bytes;
>>>>> policer->app_srtcm_params.ebs = 0;
>>>>> + err = rte_meter_srtcm_profile_config(&policer->in_prof,
>>>>> + &policer->app_srtcm_params);
>>>>> + if (err) {
>>>>> + VLOG_ERR("Could not create rte meter profile for ingress
>>>>> policer");
>>>>> + free(policer);
>>>>> + return NULL;
>>>>> + }
>>>>> err = rte_meter_srtcm_config(&policer->in_policer,
>>>>> - &policer->app_srtcm_params);
>>>>> + &policer->in_prof);
>>>>> if (err) {
>>>>> VLOG_ERR("Could not create rte meter for ingress policer");
>>>>> free(policer);
>>>>> @@ -3043,13 +3056,18 @@ netdev_dpdk_get_status(const struct
>>> netdev
>>>>> *netdev, struct smap *args)
>>>>> smap_add_format(args, "if_descr", "%s %s", rte_version(),
>>>>>
>>>>> dev_info.driver_name);
>>>>>
>>>>> - if (dev_info.pci_dev) {
>>>>> - smap_add_format(args, "pci-vendor_id", "0x%x",
>>>>> - dev_info.pci_dev->id.vendor_id);
>>>>> - smap_add_format(args, "pci-device_id", "0x%x",
>>>>> - dev_info.pci_dev->id.device_id);
>>>>> + const struct rte_bus *bus;
>>>>> + const struct rte_pci_device *pci_dev;
>>>>> + bus = rte_bus_find_by_device(dev_info.device);
>>>>> + if (bus && !strcmp(bus->name, "pci")) {
>>>>> + pci_dev = RTE_DEV_TO_PCI(dev_info.device);
>>>>> + if (pci_dev) {
>>>>> + smap_add_format(args, "pci-vendor_id", "0x%u",
>>>>> + pci_dev->id.vendor_id);
>>>>> + smap_add_format(args, "pci-device_id", "0x%x",
>>>>> + pci_dev->id.device_id);
>>>>> + }
>>>>> }
>>>>> -
>>>>> return 0;
>>>>> }
>>>>>
>>>>> @@ -3727,6 +3745,7 @@ struct egress_policer {
>>>>> struct qos_conf qos_conf;
>>>>> struct rte_meter_srtcm_params app_srtcm_params;
>>>>> struct rte_meter_srtcm egress_meter;
>>>>> + struct rte_meter_srtcm_profile egress_prof;
>>>>> };
>>>>>
>>>>> static void
>>>>> @@ -3749,8 +3768,16 @@ egress_policer_qos_construct(const struct
>>>>> smap *details,
>>>>> policer = xmalloc(sizeof *policer);
>>>>> qos_conf_init(&policer->qos_conf, &egress_policer_ops);
>>>>> egress_policer_details_to_param(details,
>>>>> &policer->app_srtcm_params);
>>>>> + err = rte_meter_srtcm_profile_config(&policer->egress_prof,
>>>>> + &policer->app_srtcm_params);
>>>>> + if (err) {
>>>>> + free(policer);
>>>>> + *conf = NULL;
>>>>> + return -err;
>>>>> + }
>>>>> err = rte_meter_srtcm_config(&policer->egress_meter,
>>>>> - &policer->app_srtcm_params);
>>>>> + &policer->egress_prof);
>>>>> +
>>>>> if (!err) {
>>>>> *conf = &policer->qos_conf;
>>>>> } else {
>>>>> @@ -3803,7 +3830,8 @@ egress_policer_run(struct qos_conf *conf,
>>>>> struct rte_mbuf **pkts, int pkt_cnt,
>>>>> struct egress_policer *policer =
>>>>> CONTAINER_OF(conf, struct egress_policer, qos_conf);
>>>>>
>>>>> - cnt = netdev_dpdk_policer_run(&policer->egress_meter, pkts,
>>>>> + cnt = netdev_dpdk_policer_run(&policer->egress_meter,
>>>>> + &policer->egress_prof, pkts,
>>>>> pkt_cnt, should_steal);
>>>>>
>>>>> return cnt;
>>>>> @@ -4103,15 +4131,15 @@ dump_flow_pattern(struct rte_flow_item
>>>>> *item)
>>>>>
>>>>> VLOG_DBG("rte flow vlan pattern:\n");
>>>>> if (vlan_spec) {
>>>>> - VLOG_DBG(" Spec: tpid=0x%"PRIx16", tci=0x%"PRIx16"\n",
>>>>> - ntohs(vlan_spec->tpid), ntohs(vlan_spec->tci));
>>>>> + VLOG_DBG(" Spec: inner_type=0x%"PRIx16",
>>> tci=0x%"PRIx16"\n",
>>>>> + ntohs(vlan_spec->inner_type),
>>>>> + ntohs(vlan_spec->tci));
>>>>> } else {
>>>>> VLOG_DBG(" Spec = null\n");
>>>>> }
>>>>>
>>>>> if (vlan_mask) {
>>>>> - VLOG_DBG(" Mask: tpid=0x%"PRIx16", tci=0x%"PRIx16"\n",
>>>>> - vlan_mask->tpid, vlan_mask->tci);
>>>>> + VLOG_DBG(" Mask: inner_type=0x%"PRIx16",
>>> tci=0x%"PRIx16"\n",
>>>>> + vlan_mask->inner_type, vlan_mask->tci);
>>>>> } else {
>>>>> VLOG_DBG(" Mask = null\n");
>>>>> }
>>>>> @@ -4281,27 +4309,56 @@ add_flow_action(struct flow_actions
>>>>> *actions, enum rte_flow_action_type type,
>>>>> actions->cnt++;
>>>>> }
>>>>>
>>>>> +/*
>>>>> + * Storage for struct rte_flow_action_rss
>>>>> + * including storage for key and queue array */ #define
>>>>> +MAX_RSS_HASH_KEY_LENGTH 128 #define
>>>> MAX_ACTION_RSS_QUEUE_NUM
>>>>> 128
>>>>> +struct action_rss_data {
>>>>> + struct rte_flow_action_rss conf;
>>>>> + uint16_t queue[MAX_ACTION_RSS_QUEUE_NUM];
>>>>> + uint8_t key[MAX_RSS_HASH_KEY_LENGTH]; };
>>>>> +
>>>>> static struct rte_flow_action_rss * add_flow_rss_action(struct
>>>>> flow_actions *actions,
>>>>> struct netdev *netdev) {
>>>>> int i;
>>>>> - struct rte_flow_action_rss *rss;
>>>>> + struct action_rss_data *rss_data;
>>>>>
>>>>> - rss = xmalloc(sizeof(*rss) + sizeof(uint16_t) * netdev->n_rxq);
>>>>> - /*
>>>>> - * Setting it to NULL will let the driver use the default RSS
>>>>> - * configuration we have set: &port_conf.rx_adv_conf.rss_conf.
>>>>> - */
>>>>> - rss->rss_conf = NULL;
>>>>> - rss->num = netdev->n_rxq;
>>>>> + if (netdev->n_rxq > MAX_ACTION_RSS_QUEUE_NUM) {
>>>>> + VLOG_ERR("Num of rxq (%u) must not be greater " \
>>>>> + "than max rss num of queues (%u)",
>>>>> + netdev->n_rxq, MAX_ACTION_RSS_QUEUE_NUM);
>>>>> + return NULL;
>>>>> + }
>>>>>
>>>>> - for (i = 0; i < rss->num; i++) {
>>>>> - rss->queue[i] = i;
>>>>> + rss_data = xmalloc(sizeof(struct action_rss_data));
>>>>> + *rss_data = (struct action_rss_data){
>>>>> + .conf = (struct rte_flow_action_rss){
>>>>> + .func = RTE_ETH_HASH_FUNCTION_DEFAULT,
>>>>> + .level = 0,
>>>>> + .types = ETH_RSS_IP,
>>>>> + .key_len = 0,
>>>>> + .queue_num = netdev->n_rxq,
>>>>> + .queue = rss_data->queue,
>>>>> + .key = rss_data->key,
>>>>> + },
>>>>> + .key = { 0 },
>>>>> + .queue = { 0 },
>>>>> + };
>>>>> +
>>>>> + /* TODO: Override key with default */
>>>>> +
>>>>> + /* Override queue array with default */
>>>>> + for (i = 0; i < rss_data->conf.queue_num; i++) {
>>>>> + rss_data->queue[i] = i;
>>>>> }
>>>>>
>>>>> - add_flow_action(actions, RTE_FLOW_ACTION_TYPE_RSS, rss);
>>>>> + add_flow_action(actions, RTE_FLOW_ACTION_TYPE_RSS,
>>>>> + &rss_data->conf);
>>>>>
>>>>> - return rss;
>>>>> + return &rss_data->conf;
>>>>> }
>>>>>
>>>>> static int
>>>>> @@ -4365,7 +4422,7 @@ netdev_dpdk_add_rte_flow_offload(struct
>>> netdev
>>>>> *netdev,
>>>>> vlan_mask.tci = match->wc.masks.vlans[0].tci &
>>>>> ~htons(VLAN_CFI);
>>>>>
>>>>> /* match any protocols */
>>>>> - vlan_mask.tpid = 0;
>>>>> + vlan_mask.inner_type = 0;
>>>>>
>>>>> add_flow_pattern(&patterns, RTE_FLOW_ITEM_TYPE_VLAN,
>>>>> &vlan_spec, &vlan_mask); @@ -4516,6
>>>>> +4573,11 @@
>>>>> end_proto_check:
>>>>>
>>>>> struct rte_flow_action_rss *rss;
>>>>> rss = add_flow_rss_action(&actions, netdev);
>>>>> + if (!rss) {
>>>>> + VLOG_ERR("add_flow_rss_action error.\n");
>>>>> + ret = -1;
>>>>> + goto out;
>>>>> + }
>>>>> add_flow_action(&actions, RTE_FLOW_ACTION_TYPE_END, NULL);
>>>>>
>>>>> flow = rte_flow_create(dev->port_id, &flow_attr,
>>>>> patterns.items,
>>>>> --
>>>>> 1.8.3.1
>>
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
