Summary: amfd: Remove asserts from validation routines [#849]
Review request for Trac Ticket(s): 849
Peer Reviewer(s): Hans F, Hans N, Nagendra, Praveen
Pull request to:
Affected branch(es): all
Development branch: default
Impacted area Impact y/n
osaf/services/saf/amf/amfd/app.cc| 18 +++---
osaf/services/saf/amf/amfd/comp.cc | 5 +++--
osaf/services/saf/amf/amfd/compcstype.cc | 5 +++--
osaf/services/saf/amf/amfd/hlt.cc| 10 ++
osaf/services/saf/amf/amfd/sg.cc | 9 +
Will push this tomorrow if there are no objections.
/AndersBj
-Original Message-
From: Anders Bjornerstedt [mailto:anders.bjornerst...@ericsson.com]
Sent: den 29 april 2014 16:37
To: reddy.neelaka...@oracle.com; Zoran Milinkovic
Cc: opensaf-devel@lists.sourceforge.net
Subject: [devel]
osaf/libs/common/immsv/immsv_evt.c | 56 ++-
osaf/libs/common/immsv/include/immsv_evt.h | 4 +
osaf/libs/common/immsv/include/immsv_evt_model.h | 1 +
osaf/services/saf/immsv/immd/immd_evt.c | 9 ++-
osaf/services/saf/immsv/immnd/ImmModel.cc
tests/immsv/implementer/test_cleanup.c |1 +
tests/immsv/implementer/test_saImmOiImplementerSet.c | 287 +++
2 files changed, 288 insertions(+), 0 deletions(-)
Test cases cover OI callback timeout and timeout on searching for an object
with RTA
diff --git
osaf/libs/agents/saf/imma/imma_cb.h | 1 +
osaf/libs/agents/saf/imma/imma_def.h| 1 +
osaf/libs/agents/saf/imma/imma_oi_api.c | 23 ++-
3 files changed, 24 insertions(+), 1 deletions(-)
IMMA uses IMMA_OI_CALLBACK_TIMEOUT for sending OI timeout to IMM service
Should we change value for saAmfSGAutoRepair (in
osaf/services/saf/amf/config/amf_objects.xml) to have the value True since we
now actually support this?
Thanks,
Hans
--
Is your legacy SCM system holding you back? Join
According to MDS documentation COPY support is needed when sending messages to
the same MDS core which I interpret as the same linux process.
Since AMF does not do that (correct me if I am wrong) this code can/should be
removed. I added an assert in the COPY callback and nothing happens...
osaf/libs/core/common/ncs_main_pub.c | 2 +-
osaf/libs/core/common/ncs_sprr.c | 2 +-
osaf/libs/core/include/ncssysf_def.h | 4 ++--
osaf/libs/core/leap/hj_dec.c | 4 ++--
osaf/libs/core/leap/hj_enc.c | 8
Yes that should ALSO be done. :-) There are probably several more places
where we have printf.
This patch just fixes the missing parentheses. I could fix the printf
too, while touching that line.
/ Anders Widell
On 05/08/2014 03:33 PM, Hans Feldt wrote:
Why not just replace with trace?
Ack on these patches with the following comments:
---
1)
In the imma client you have added:
+#define IMMSV_OI_CALLBACK_WAIT_TIME 6 /* Default wait time for OI
callback to reply in seconds = 6 sec*/
This default timeout is set in the client if the client has not explicitly
osaf/services/saf/immsv/README | 22 ++
1 files changed, 22 insertions(+), 0 deletions(-)
See osaf/services/saf/immsv/README for details on IMMA_OI_CALLBACK_TIMEOUT.
diff --git a/osaf/services/saf/immsv/README b/osaf/services/saf/immsv/README
---
12 matches
Mail list logo