Hi,

I think this is just enhancement, not an urgent fix.
Then we should make it better if possible.

About #2859, I am not reviewer at that time.
But I would not agree that solution as we can see service keep restart
if service still start in middle of one CCB many operations.

Best Regards,
ThuanTr

-----Original Message-----
From: Thang Duc Nguyen <thang.d.ngu...@dektech.com.au> 
Sent: Thursday, April 16, 2020 10:51 AM
To: Thuan Tran <thuan.t...@dektech.com.au>; Minh Hon Chau 
<minh.c...@dektech.com.au>
Cc: opensaf-devel@lists.sourceforge.net
Subject: RE: [PATCH 1/1] ntf: restart ntfimcnd if it fails to get operation 
invoke name [#3178]

Hi Thuan,

Thanks for your comment.
First this issue happen only in specific situation. And I think restart it is 
no cause big issue.
And the ccb is internal data based mange by ntf/ntfimcnd. After ntfimcnd 
restart, it reinitialize 
CcbUtilCcbData and operation invoke name is empty.

Moreover, in current code in ntfimcn_imm.c, there are many place use 
imcn_exit(EXIT_FAILURE)
when detect the error. Example for this is #2859.
We consider to open a new ticket to consider your suggestion by refactor/change 
current behavior of ntfimcnd.

B.R/Thang

-----Original Message-----
From: Thuan Tran <thuan.t...@dektech.com.au> 
Sent: Thursday, April 16, 2020 10:16 AM
To: Thang Duc Nguyen <thang.d.ngu...@dektech.com.au>; Minh Hon Chau 
<minh.c...@dektech.com.au>
Cc: opensaf-devel@lists.sourceforge.net
Subject: RE: [PATCH 1/1] ntf: restart ntfimcnd if it fails to get operation 
invoke name [#3178]

Hi Thang,

>From reproduce method, with solution after exit (instead of crash), user 
>continue input another operation then service exit again.
The point is why we cannot get admin owner or object implementer via 2nd imm 
modify callback in this scenario?
Is it an IMM limit that don't include admin owner or object implementer from 
2nd modify callback?

If limit, can we use another way to get admin owner or object implementer base 
on object name?
By this, we can avoid continuous exit if user keep going on operations by same 
CCB.

Best Regards,
ThuanTr

-----Original Message-----
From: Thang Duc Nguyen <thang.d.ngu...@dektech.com.au>
Sent: Wednesday, April 15, 2020 3:43 PM
To: Minh Hon Chau <minh.c...@dektech.com.au>; Thuan Tran 
<thuan.t...@dektech.com.au>
Cc: opensaf-devel@lists.sourceforge.net; Thang Duc Nguyen 
<thang.d.ngu...@dektech.com.au>
Subject: [PATCH 1/1] ntf: restart ntfimcnd if it fails to get operation invoke 
name [#3178]

If ntfimcnd is restarted during ccb modify, it will initialize ccbUtilCcbData 
that not contain operation invoke name.
This causes ntfimcnd crashed due to operation invoke name not existed.

The fix is to restart ntfimcnd instead of raising the coredump.
---
 src/ntf/ntfimcnd/ntfimcn_imm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/ntf/ntfimcnd/ntfimcn_imm.c b/src/ntf/ntfimcnd/ntfimcn_imm.c 
index 3c0a8c02a..3563a2264 100644
--- a/src/ntf/ntfimcnd/ntfimcn_imm.c
+++ b/src/ntf/ntfimcnd/ntfimcn_imm.c
@@ -376,9 +376,9 @@ get_operation_invoke_name_modify(SaImmOiCcbIdT ccbId,
                        goto done;
                }
        }
-       /* If we get here no name is found! */
+       /* ntfimcnd was restarted during ccb midify */
        LOG_ER("%s no name was found", __FUNCTION__);
-       osafassert(0);
+       imcn_exit(EXIT_FAILURE);
 
 done:
        TRACE_LEAVE();
--
2.17.1



_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to