Re: [devel] [PATCH 1 of 1] amfd: assign only one csi per component for NPI SU [#856]

2014-04-16 Thread Nagendra Kumar
Ack. Thanks -Nagu -Original Message- From: Praveen Malviya Sent: 15 April 2014 16:19 To: hans.fe...@ericsson.com; Nagendra Kumar Cc: opensaf-devel@lists.sourceforge.net Subject: [PATCH 1 of 1] amfd: assign only one csi per component for NPI SU [#856]

Re: [devel] [PATCH 2 of 2] amfd: Remove unnecessary traces [#817]

2014-04-16 Thread Gary Lee
Thanks Hans and Anders. I'll post a new patch... On 11/04/14 17:20, Hans Feldt wrote: Minor comments that can be fixed at the same time: - in ImmObjCreate::exec TRACE_ENTER2 becomes anonymous just printing exec as the function name, so we should add Create in the text and some suitable args.

[devel] [PATCH 0 of 2] Review Request for amfd: use job queue for IMM admin response [#817]

2014-04-16 Thread Gary Lee
Summary: amfd: use job queue for IMM admin response [#817] V2 Review request for Trac Ticket(s): 817 Peer Reviewer(s): Hans F, Hans N, Nagendra, Praveen Pull request to: Affected branch(es): default Development branch: default Impacted area Impact y/n

[devel] [PATCH 1 of 2] amfd: use job queue for IMM admin response [#817]

2014-04-16 Thread Gary Lee
osaf/services/saf/amf/amfd/imm.cc| 45 +++ osaf/services/saf/amf/amfd/include/imm.h | 16 +++ 2 files changed, 54 insertions(+), 7 deletions(-) Currently, admin operation responses from amfd are not synchronised with IMM changes that may be

Re: [devel] [PATCH 0 of 2] Review Request for amfd: use job queue for IMM admin response [#817]

2014-04-16 Thread Hans Feldt
Ack from me for patch series. I will push this by end tomorrow unless I hear something else. Thanks, Hans -Original Message- From: Gary Lee [mailto:gary@dektech.com.au] Sent: den 16 april 2014 09:23 To: Hans Feldt; Hans Nordebäck; nagendr...@oracle.com Cc:

Re: [devel] [PATCH 0 of 2] Review Request for IMM: add error strings for CCB operations for ERR_NOT_EXIST error [#459]

2014-04-16 Thread Neelakanta Reddy
Hi zoran, Reviewed and tested the patch. Ack. /Neel. On Monday 14 April 2014 05:40 PM, Zoran Milinkovic wrote: Summary: IMM: add error strings for CCB operations for ERR_NOT_EXIST error [#459] Review request for Trac Ticket(s): 459 Peer Reviewer(s): Anders, Neelakanta Pull request to: