Re: [devel] [PATCH 1/1] mds: Avoid message re-allocation [#3089]

2019-11-25 Thread Minh Hon Chau
Hi Vu, Thuan, The patch misses the error cases and the kDisabled state. I rework for the V2. Thanks Minh On 25/11/19 6:44 pm, Nguyen Minh Vu wrote: Hi Minh, Ack with comments inline. Regards, Vu On 11/25/19 1:12 PM, Minh Chau wrote: The patch avoids message reallocation if enable

Re: [devel] [PATCH 1/1] mds: Avoid message re-allocation [#3089]

2019-11-24 Thread Nguyen Minh Vu
Hi Minh, Ack with comments inline. Regards, Vu On 11/25/19 1:12 PM, Minh Chau wrote: The patch avoids message reallocation if enable MDS_TIPC_FCTRL_ENABLED --- src/mds/mds_dt_tipc.c| 27 --- src/mds/mds_tipc_fctrl_msg.cc| 2 +-

Re: [devel] [PATCH 1/1] mds: Avoid message re-allocation [#3089]

2019-11-24 Thread Tran Thuan
Hi Minh, ACK from me. Best Regards, ThuanTr -Original Message- From: Minh Chau Sent: Monday, November 25, 2019 1:13 PM To: thuan.t...@dektech.com.au; gary@dektech.com.au; vu.m.ngu...@dektech.com.au Cc: opensaf-devel@lists.sourceforge.net; Minh Chau Subject: [PATCH 1/1] mds:

[devel] [PATCH 1/1] mds: Avoid message re-allocation [#3089]

2019-11-24 Thread Minh Chau
The patch avoids message reallocation if enable MDS_TIPC_FCTRL_ENABLED --- src/mds/mds_dt_tipc.c| 27 --- src/mds/mds_tipc_fctrl_msg.cc| 2 +- src/mds/mds_tipc_fctrl_portid.cc | 9 +++-- 3 files changed, 24 insertions(+), 14 deletions(-) diff --git