[ovs-dev] [PATCH v3] ovs-ofctl:'--bundle' option can be used with OpenFlow 1.3

2022-06-06 Thread yangchang
>From the commit 25070e045e, bundle option can be used with OpenFlow 1.3 Signed-off-by: yangchang Acked-by: Mike Pattrick --- Notes: v2: modify some compilation errors v3: modify message, change 'has' to 'can' utilities/ovs-ofctl.8.in

[ovs-dev] [PATCH v1] lacp: modify the comment misspelling

2022-06-23 Thread yangchang
change 'negotations' to 'negotiations' Signed-off-by: yangchang --- lib/lacp.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/lacp.h b/lib/lacp.h index d731ae9..3c14862 100644 --- a/lib/lacp.h +++ b/lib/lacp.h @@ -24,7 +24,7 @@ /* LACP Pro

[ovs-dev] [PATCH v1] flow: later fragment packets are set next header protocol

2022-04-24 Thread yangchang
If the flow contains protocol types, that the later fragment packets of UDP6 and ICMPv6 cannot hit the flow. So the later fragment packets should be set next header protocl types, too. Signed-off-by: yangchang --- lib/flow.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff

[ovs-dev] [PATCH v1] ovs-ofctl: '--bundle' option can be used with OpenFlow 1.3

2022-05-09 Thread yangchang
>From the commit 25070e045e, bundle option has be used with OpenFlow 1.3 Signed-off-by: yangchang --- utilities/ovs-ofctl.8.in | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/utilities/ovs-ofctl.8.in b/utilities/ovs-ofctl.8.in index e903ca12b..35f4dea80 100

[ovs-dev] [PATCH v2] ovs-ofctl: '--bundle' option can be used with OpenFlow 1.3

2022-05-09 Thread yangchang
>From the commit 25070e045e, bundle option has be used with OpenFlow 1.3 Signed-off-by: yangchang --- utilities/ovs-ofctl.8.in | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/utilities/ovs-ofctl.8.in b/utilities/ovs-ofctl.8.in index e903ca1..35f4dea 100644 --

[ovs-dev] [PATCH v4] ovs-ofctl:'--bundle' option can be used with OpenFlow 1.3

2022-11-03 Thread yangchang
>From the commit 25070e045e, bundle option can be used with OpenFlow 1.3 Signed-off-by: yangchang Acked-by: Mike Pattrick --- utilities/ovs-ofctl.8.in | 10 +- utilities/ovs-save | 6 +++--- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/utilities/ovs-ofctl.8

[ovs-dev] [PATCH] netdev-dpdk: make sure the mempool have enough mbufs for rxq

2022-12-06 Thread yangchang
For eth dev, dpdk XXX_rx_queue_start fuction will pre alloc mbuf for rxq, if n_mbufs < n_rxq * rxq_size, dev will start error becuse of it cannot allocate memory. When n_mbufs are not enough, the mempool is not necessary to create. Signed-off-by: yangchang --- lib/netdev-dpdk.c |

[ovs-dev] [PATCH v2] netdev-dpdk: make sure the mempool has enough mbufs for rxq

2022-12-06 Thread yangchang
For eth dev, dpdk X_rx_queue_start fuction will pre alloc mbuf for rxq, if n_mbufs < n_rxq * rxq_size, dev will start error becuse of it cannot allocate memory. If n_mbufs are not enough, the mempool is not necessary to create. Signed-off-by: yangchang --- lib/netdev-dpdk.c |

[ovs-dev] [PATCH v3] netdev-dpdk: make sure the mempool has enough mbufs for rxq

2022-12-07 Thread yangchang
For eth dev, dpdk X_rx_queue_start fuction will pre alloc mbuf for rxq, if n_mbufs < n_rxq * rxq_size, dev will start error becuse of it cannot allocate memory. If n_mbufs are not enough, the mempool is not necessary to create. Signed-off-by: yangchang --- lib/netdev-dpdk.c |

Re: [ovs-dev] [PATCH v3] netdev-dpdk: make sure the mempool has enough mbufs for rxq

2022-12-11 Thread yangchang
>>On 07/12/2022 11:53, yangchang wrote: >> For eth dev, dpdk X_rx_queue_start fuction will pre alloc mbuf >> for rxq, if n_mbufs < n_rxq * rxq_size, dev will start error >> becuse of it cannot allocate memory. If n_mbufs are not enough, >> the mempool is not n

[ovs-dev] [PATCH v4] netdev-dpdk: make sure the mempool has enough mbufs for rxq

2022-12-22 Thread yangchang
hanges. Signed-off-by: yangchang --- Notes: v3: - change some compilation errors v4: - add to determine whether mp is empty - correct spelling errors - change “break” to "return" lib/netdev-dpdk.c | 13 + 1

Re: [ovs-dev] [PATCH v3] netdev-dpdk: make sure the mempool has enough mbufs for rxq

2022-12-22 Thread yangchang
>On 12/12/2022 02:29, yangchang wrote: >> >>>> On 07/12/2022 11:53, yangchang wrote: >>>> For eth dev, dpdk X_rx_queue_start fuction will pre alloc mbuf >>>> for rxq, if n_mbufs < n_rxq * rxq_size, dev will start error >>>> becuse of it

[ovs-dev] [PATCH] bond: modify bond_link_status_update log

2022-08-17 Thread yangchang
bond link status change comes from carrier and lacp may_enable ,when carrir does not change and lacp change, prompt of LACP will be more accurate. Signed-off-by: yangchang --- lib/bond.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/lib/bond.c b/lib/bond.c index

[ovs-dev] [PATCH] bond: modify bond_link_status_update log

2022-08-17 Thread yangchang
bond link status change comes from carrier and lacp may_enable ,when carrir does not change and lacp change, prompt of LACP will be more accurate. Signed-off-by: yangchang --- ofproto/bond.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/ofproto/bond.c b/ofproto

[ovs-dev] [PATCH] bond: modify bond_link_status_update log

2022-08-17 Thread yangchang
bond link status change comes from carrier and LACP may_enable ,when carrier does not change and LACP change, prompt of LACP will be more accurate. Signed-off-by: yangchang --- ofproto/bond.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/ofproto/bond.c b/ofproto

[ovs-dev] [PATCH] bond: modify bond_link_status_update log

2022-08-17 Thread yangchang
bond link status change comes from carrier and LACP may_enable ,when carrier does not change and LACP change, prompt of LACP will be more accurate. Signed-off-by: yangchang --- ofproto/bond.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/ofproto/bond.c b/ofproto

Re: [ovs-dev] [PATCH] bond: modify bond_link_status_update log

2022-08-17 Thread yangchang
Thanks for your review, It is indeed similar to the patch, I will testing and reviewing the change. yangch...@chinatelecom.cn From: Ilya Maximets Date: 2022-08-17 20:15 To: yangchang; dev CC: i.maximets; Michael Pattrick Subject: Re: [ovs-dev] [PATCH] bond: modify bond_link_status_update log

[ovs-dev] [PATCH] netdev-dpdk: set netdev_dpdk dev attached flag

2022-08-24 Thread yangchang
If the attached flag is not set, the dev is not detached when it is desruct, and other dpdk processes will report an error when using it. Signed-off-by: yangchang --- lib/netdev-dpdk.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c

[ovs-dev] [PATCH] bond: modify bond crash error

2022-10-14 Thread yangchang
dp_netdev_process_rxq_port (lib/dpif-netdev.c:5337) 15 pmd_thread_main (lib/dpif-netdev.c:6944) 16 ovsthread_wrapper (lib/ovs-thread.c:422) 17 ?? (/lib64/libpthread.so.0) 18 clone (/lib64/libc.so.6) Signed-off-by: yangchang --- ofproto/bond.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[ovs-dev] [PATCH] userspace: modify the width of tpa and spa

2023-05-25 Thread yangchang
Arp_spa and arp_tpa are IP addresses, their width should be 32 bits. Signed-off-by: yangchang --- lib/meta-flow.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/meta-flow.xml b/lib/meta-flow.xml index a1a2036..bdd12f6 100644 --- a/lib/meta-flow.xml +++ b/lib/meta

[ovs-dev] [PATCH] ovs-tcpdump:Solve the problem of residual mirror port

2024-03-21 Thread yangchang
Before creating an mirror port, it is necessary to first check whether the original port exists. Otherwise, when the original port does not exist, the created mirror port will remain. Signed-off-by: yangchang --- utilities/ovs-tcpdump.in | 6 +++--- 1 file changed, 3 insertions(+), 3

Re: [ovs-dev] [PATCH] ovs-tcpdump:Solve the problem of residual mirror port

2024-04-09 Thread yangchang
>On Thu, Mar 21, 2024 at 08:33:37PM +0800, yangchang wrote: >> Before creating an mirror port, it is necessary to first check whether the >> original port exists. >> Otherwise, when the original port does not exist, the created mirror port >> will remain. >