Re: [PATCH] app/test: fix LACP handshake overtime

2024-04-17 Thread Ferruh Yigit
On 8/18/2022 6:43 AM, Ke Zhang wrote: > Increase the loop count so that there is a longer > threshold time for the LACP handshake process. > > Fixes: 5e41ab250dfa ("app/test: unit tests for bonding mode 4") > Cc: sta...@dpdk.org > > Signed-off-by: Ke Zhang > Patch seems to fix unit test where L

RE: [PATCH] app/test: fix LACP handshake overtime

2022-10-21 Thread Zhang, Ke1X
riday, September 30, 2022 9:28 AM > To: humin (Q) ; ch...@att.com; dev@dpdk.org > Cc: sta...@dpdk.org > Subject: RE: [PATCH] app/test: fix LACP handshake overtime > > Hi humin: > > Thanks for your comments, I test the case, I try to change > > for (i = 0; i < 30 &a

RE: [PATCH] app/test: fix LACP handshake overtime

2022-09-29 Thread Zhang, Ke1X
.c (app\test): rte_delay_ms(conf.update_timeout_ms); So I didn't change the value, please confirming , many thanks. BR Zhangke > -Original Message- > From: humin (Q) > Sent: Friday, September 23, 2022 8:58 AM > To: Zhang, Ke1X ; ch...@att.com; dev@dpdk.org >

Re: [PATCH] app/test: fix LACP handshake overtime

2022-09-22 Thread humin (Q)
Hi, Zhang,     Why LACP handshake "overtime" in your test?     Maybe in other occasions, 60 * delay is also not enough.     BTW, how about config "conf.update_timeout_ms" for the test? 在 2022/8/18 13:43, Ke Zhang 写道: Increase the loop count so that there is a longer threshold time for the LAC