[devel] [PATCH 1 of 1] cpsv: set dtaBuff to NULL and readSize to zero in TrackCallback [#733]

2014-06-03 Thread mahesh . valla
osaf/services/saf/cpsv/cpnd/cpnd_proc.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) Now in the TrackCallback() the ioVector dtaBuff is set to NULL, readSize needs to be set to zero diff --git a/osaf/services/saf/cpsv/cpnd/cpnd_proc.c b/osaf/services/saf/cpsv/cpnd/cpnd_proc.c ---

[devel] [PATCH 0 of 1] Review Request for cpsv: set dtaBuff to NULL and readSize to zero in TrackCallback [#733]

2014-06-03 Thread mahesh . valla
Summary:cpsv: set dtaBuff to NULL and readSize to zero in TrackCallback [#733] Review request for Trac Ticket(s):#733 Peer Reviewer(s): Sirisha/Ramesh Pull request to: <> Affected branch(es): default & 4.4 Development branch: default Impacted area Impact y/

[devel] [PATCH 1 of 1] cpsv: set dataSize to zero in case of section deletion in the TrackCallback [#734]

2014-06-03 Thread mahesh . valla
osaf/services/saf/cpsv/cpnd/cpnd_evt.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) In case of section deletion dataSize set to zero in the TrackCallback() diff --git a/osaf/services/saf/cpsv/cpnd/cpnd_evt.c b/osaf/services/saf/cpsv/cpnd/cpnd_evt.c --- a/osaf/services/saf/cpsv/cp

[devel] [PATCH 0 of 1] Review Request for cpsv: set dataSize to zero in case of section deletion in the TrackCallback [#734]

2014-06-03 Thread mahesh . valla
Summary:cpsv: set dataSize to zero in case of section deletion in the TrackCallback [#734] Review request for Trac Ticket(s): #734 Peer Reviewer(s): Sirisha/Ramesh Pull request to: <> Affected branch(es): default & 4.4 Development branch: default Impacted area

[devel] [PATCH 0 of 1] Review Request for log: Cleanup some findings [#924]

2014-06-03 Thread Lennart Lund
Summary: log: Cleanup some findings Review request for Trac Ticket(s): #924 Peer Reviewer(s): mathi.naic...@oracle.com, anders.wid...@ericsson.com Pull request to: <> Affected branch(es): 4.4, 4.5 Development branch: <> Impacted area Impact y/n --

[devel] [PATCH 1 of 1] log: Cleanup some findings [#924]

2014-06-03 Thread Lennart Lund
osaf/services/saf/logsv/lgs/lgs_file.c| 4 ++-- osaf/services/saf/logsv/lgs/lgs_filehdl.c | 1 - osaf/services/saf/logsv/lgs/lgs_stream.c | 8 ++-- 3 files changed, 8 insertions(+), 5 deletions(-) lgs_filehdl: make_log_dir_hdl() strncpy replaced by snprintf but the strncpy is not rem

Re: [devel] [PATCH 0 of 1] Review Request for v2 amfd: Send notifications when cold sync is completed [#860]

2014-06-03 Thread Mathivanan Naickan Palanivelu
Hi, Just curious, how would this notification be useful to the end-user? I mean, is there an usecase where this notification would be consumed by the end-user? (Or is it just for testing purposes? If so, I think just logging would be enough!) Thanks, Mathi. >-Original Message- >From: M

Re: [devel] [PATCH 0 of 1] Review Request for v2 amfd: Send notifications when cold sync is completed [#860]

2014-06-03 Thread Hans Feldt
As it says in the ticket. Mainly for test to know when the standby controller is ready. I think SMF has some similar problem when it rolls over nodes and needs to be poll. Thanks, Hans > -Original Message- > From: Mathivanan Naickan Palanivelu [mailto:mathi.naic...@oracle.com] > Sent: de

Re: [devel] [PATCH 1 of 7] amfd: add support for different key types in db_template v4 [#713]

2014-06-03 Thread Nagendra Kumar
Ack for all the patches. Thanks -Nagu > -Original Message- > From: Hans Nordeback [mailto:hans.nordeb...@ericsson.com] > Sent: 02 June 2014 17:25 > To: hans.fe...@ericsson.com; Nagendra Kumar; Praveen Malviya > Cc: opensaf-devel@lists.sourceforge.net > Subject: [PATCH 1 of 7] amfd: add su

Re: [devel] [PATCH 1 of 1] amfd: Send notification when cold sync is completed [#860]

2014-06-03 Thread praveen malviya
Here it is standby controller that is sending the notification. Are there any other notifications which standby sends ? Thanks Praveen On 03-Jun-14 10:57 AM, Minh Hon Chau wrote: > osaf/services/saf/amf/amfd/chkop.cc | 1 + > osaf/services/saf/amf/amfd/include/ntf.h | 14 +++- > os

Re: [devel] [PATCH 1 of 1] amfd: Send notification when cold sync is completed [#860]

2014-06-03 Thread Hans Feldt
Guess not and I this one might as well be sent by the active amfd. It has the knowledge Skickat från min Sony Xperia™-smartphone praveen malviya skrev Here it is standby controller that is sending the notification. Are there any other notifications which standby sends ? Thanks Prave

[devel] [PATCH 1 of 2] amfd: use simple decode in App ckpt [#713]

2014-06-03 Thread Hans Nordeback
osaf/services/saf/amf/amfd/ckpt_dec.cc | 51 ++ 1 files changed, 15 insertions(+), 36 deletions(-) diff --git a/osaf/services/saf/amf/amfd/ckpt_dec.cc b/osaf/services/saf/amf/amfd/ckpt_dec.cc --- a/osaf/services/saf/amf/amfd/ckpt_dec.cc +++ b/osaf/services/saf/a

[devel] [PATCH 0 of 2] Review Request for AMF: remove EDU use in App [#713]

2014-06-03 Thread Hans Nordeback
Summary: AMF: remove EDU use in App Review request for Trac Ticket(s): [#713] Peer Reviewer(s): HansF, Nagu, Praveen Pull request to: Affected branch(es): default Development branch: default Impacted area Impact y/n Docs

[devel] [PATCH 2 of 2] amfd: use simple encode in App ckpt [#713]

2014-06-03 Thread Hans Nordeback
osaf/services/saf/amf/amfd/ckpt_dec.cc| 17 +++- osaf/services/saf/amf/amfd/ckpt_edu.cc| 51 +-- osaf/services/saf/amf/amfd/ckpt_enc.cc| 46 +--- osaf/services/saf/amf/amfd/include/ckpt_edu.h | 3 - 4 files changed, 24 in