Re: [devel] test

2014-10-19 Thread Suryanarayana Garlapati
i am not getting mails which i am sending to the list. On 20-10-2014 10:18, A V Mahesh wrote: > Test passed :) > > -AVM > > On 10/20/2014 10:15 AM, Suryanarayana Garlapati wrote: > -- Suryanarayana Garlapati | Senior Solutions Architect GlobalLogic P +91.120.406.2257 M +91.8

[devel] test

2014-10-19 Thread Suryanarayana Garlapati
-- Suryanarayana Garlapati | Senior Solutions Architect GlobalLogic P +91.120.406.2257 M +91.874.400.0585 Skype surya_g10 www.globallogic.com http://www.globallogic.com/email_disclaimer.txt -- Comprehensive Server

Re: [devel] [PATCH 1 of 1] mds: change tipc_or_tcp variable from string to bool to avoid strcmp [#998]

2014-10-16 Thread Suryanarayana Garlapati
is_tipc = true; >>> + } >>> >>> - if (strcmp(tipc_or_tcp, "TIPC") == 0) { >>> + if (is_tipc) { >>> mds_mdtm_init = mdtm_tipc_init; >>> mds_mdtm_destroy = mdtm_tipc_destroy; >>> mds

Re: [devel] [PATCH 1 of 1] mds: change tipc_or_tcp variable from string to bool to avoid strcmp [#998]

2014-10-16 Thread Suryanarayana Garlapati
cho message on opensaf start `opensafd: > Starting OpenSAF Services (Using TIPC)` [Surya] If we are getting any other transport variable other than TIPC or TCP, we should exit. Similarly for the intranode communication as well. > > -AVM > > On 10/16/2014 2:41 PM, Suryanarayana Garlap

Re: [devel] [PATCH 1 of 1] test:tune the MDS test code according to new MDS Fragment SIZE [959]

2014-07-16 Thread SuryaNarayana Garlapati
Comments inline. On Thursday 17 July 2014 11:31 AM, mahesh.va...@oracle.com wrote: > tests/mds/mdstipc.h | 2 +- > tests/mds/mdstipc_api.c | 40 > 2 files changed, 21 insertions(+), 21 deletions(-) > > > The patch tunes the MDS test code (/tests/

Re: [devel] Upcoming OpenSAF 4.5 Transport (MDS) Performance Measurements

2014-05-28 Thread SuryaNarayana Garlapati
Good Work guys. On Thursday 29 May 2014 10:32 AM, A V Mahesh wrote: > > Attaching the Table in PDF ( few people doesn't receive image in > correct format ) > > -AVM > > On 5/29/2014 9:22 AM, A V Mahesh wrote: >> Hi all , >> >> Following are the Upcoming OpenSAF 4.5 Transport (MDS**TIPC/TCP) >>

Re: [devel] [PATCH 1 of 3] dtm: upgrade the fragment size to MDS_DIRECT_BUF_MAXSIZE [#654]

2014-05-19 Thread SuryaNarayana Garlapati
On Tuesday 20 May 2014 12:08 PM, A V Mahesh wrote: > Hi, > > On 5/20/2014 11:47 AM, SuryaNarayana Garlapati wrote: >> >> On Tuesday 20 May 2014 12:56 AM, Hans Feldt wrote: >>> - I guess the commit message prefix should be "mds: " >>> - I miss the

Re: [devel] [PATCH 1 of 3] dtm: upgrade the fragment size to MDS_DIRECT_BUF_MAXSIZE [#654]

2014-05-19 Thread SuryaNarayana Garlapati
On Tuesday 20 May 2014 11:52 AM, Hans Feldt wrote: > >> -Original Message- >> From: SuryaNarayana Garlapati [mailto:suryanarayana.garlap...@oracle.com] >> Sent: den 20 maj 2014 08:17 >> To: Hans Feldt; mahesh.va...@oracle.com; de...@list.opens

Re: [devel] [PATCH 1 of 3] dtm: upgrade the fragment size to MDS_DIRECT_BUF_MAXSIZE [#654]

2014-05-19 Thread SuryaNarayana Garlapati
On Tuesday 20 May 2014 12:56 AM, Hans Feldt wrote: > - I guess the commit message prefix should be "mds: " > - I miss the use case / benefit of the proposed change in the commit message > > I don't understand the need for setting a default of rcvbuf and sndbuf for > unix domain sockets. This make

Re: [devel] [PATCH 3 of 4] mds: use TIPC segmentation/reassembly [#654]

2014-05-06 Thread SuryaNarayana Garlapati
is decreased. Regards Surya On Wednesday 07 May 2014 08:44 AM, A V Mahesh wrote: > Surya, > > Thank for reiterating arch_word of the MDS feature ,we all in sync. > > On 5/6/2014 3:55 PM, SuryaNarayana Garlapati wrote: >>> MDS version unless we get alternate bits

Re: [devel] [PATCH 3 of 4] mds: use TIPC segmentation/reassembly [#654]

2014-05-06 Thread SuryaNarayana Garlapati
ode install & subscribe` option for Mds version (My > previously published approach) and fix > all the ENC_TYPE_FLAT issue with in-service upgrade and expose > `same arc message optimization` > feature to OpenSAF users. > > > -AVM > > On 5/1/2014 2:45 PM, SuryaN

Re: [devel] [PATCH 3 of 4] mds: use TIPC segmentation/reassembly [#654]

2014-05-01 Thread SuryaNarayana Garlapati
m. I don't see why we should. [Surya] This is not limited to even 32/64 bit. Its how the encoding is handled for the destinations. > > Thanks, > Hans > >> -Original Message- >> From: SuryaNarayana Garlapati [mailto:suryanarayana.garlap...@oracle.com] >

Re: [devel] [PATCH 3 of 4] mds: use TIPC segmentation/reassembly [#654]

2014-05-01 Thread SuryaNarayana Garlapati
On Thursday 01 May 2014 01:04 PM, A V Mahesh wrote: > On 5/1/2014 10:16 AM, SuryaNarayana Garlapati wrote: >> The archtype is not a legacy one, it was introduced to have the >> optimization for the message encoding by the applications. > > Passing compile flags `mds

Re: [devel] [PATCH 3 of 4] mds: use TIPC segmentation/reassembly [#654]

2014-04-30 Thread SuryaNarayana Garlapati
On Thursday 01 May 2014 09:56 AM, A V Mahesh wrote: > Hi Surya, > > On 5/1/2014 7:28 AM, Suryanarayana Garlapati wrote: >> Nack. >> MDS ARCHWORD is used for deciding the factor whether the message >> which is being sent to the destination should be encoded >> fu

Re: [devel] [PATCH 1 of 4] mds: use TIPC defined max msg size for rec buffer [#654]

2014-04-30 Thread SuryaNarayana Garlapati
On Tuesday 29 April 2014 10:58 AM, Hans Feldt wrote: > osaf/libs/core/mds/include/mds_dt.h | 3 --- > osaf/libs/core/mds/mds_dt_tipc.c| 2 +- > 2 files changed, 1 insertions(+), 4 deletions(-) > > > Using connection-less TIPC messaging the largest message that can be sent and > received

Re: [devel] [PATCH 3 of 4] mds: use TIPC segmentation/reassembly [#654]

2014-04-30 Thread Suryanarayana Garlapati
Nack. MDS ARCHWORD is used for deciding the factor whether the message which is being sent to the destination should be encoded full/flat/copied. This bits were introduced to do a optimization in the way the callbacks encode full/flat are called. For example, if there are two processes(can be pre

Re: [devel] [PATCH 1 of 3] avsv: include and use sirank in SUSI msg [#574]

2014-04-18 Thread SuryaNarayana Garlapati
On Friday 11 April 2014 12:52 PM, Hans Feldt wrote: > On 8 April 2014 11:55, SuryaNarayana Garlapati > wrote: >> I Tested your patch and it works because of the bug in the existing code >> where the AVD subpart version is incremented but not the Max version(in >> prev

Re: [devel] [PATCH 1 of 3] avsv: include and use sirank in SUSI msg [#574]

2014-04-08 Thread SuryaNarayana Garlapati
avd also this kind of mapping should be added. Regards Surya On Monday 07 April 2014 07:11 PM, SuryaNarayana Garlapati wrote: > After further testing, i found that si rank is always being read from > IMM. Will update you more on this. > As of now, please hold for some time. > > On

Re: [devel] [PATCH 1 of 3] avsv: include and use sirank in SUSI msg [#574]

2014-04-07 Thread SuryaNarayana Garlapati
After further testing, i found that si rank is always being read from IMM. Will update you more on this. As of now, please hold for some time. On Monday 07 April 2014 06:12 PM, SuryaNarayana Garlapati wrote: > Nack. > > Following points needs to be considered. > > 1. As there

Re: [devel] [PATCH 1 of 3] avsv: include and use sirank in SUSI msg [#574]

2014-04-07 Thread SuryaNarayana Garlapati
Nack. Following points needs to be considered. 1. As there is a change in messaging change with the AVD, You should bump up the AVD MDS Sub part version. 2. As there is no change in AVND messaging, its MDS sub part version should not be changed. Attached is modified patch. I had tested 4.3 a

Re: [devel] [PATCH 2 of 5] amfnd: store amfd mds ver [#574]

2014-04-04 Thread SuryaNarayana Garlapati
See inline > Thanks, > Hans > >> -Original Message- >> From: SuryaNarayana Garlapati [mailto:suryanarayana.garlap...@oracle.com] >> Sent: den 4 april 2014 08:29 >> To: Hans Feldt >> Cc: Hans Feldt; Hans Nordebäck; praveen malviya; nagendr...@oracle.com; >> o

Re: [devel] [PATCH 2 of 5] amfnd: store amfd mds ver [#574]

2014-04-03 Thread SuryaNarayana Garlapati
ited it to 8 bits for the SVC install. > > Thanks, > Hans > > On 3 April 2014 13:13, SuryaNarayana Garlapati > wrote: >> Comments: >> 1. First of all, the mds sub part version should not be stored/used. You >> should depend on the message format version as SI ra

Re: [devel] [PATCH 2 of 5] amfnd: store amfd mds ver [#574]

2014-04-03 Thread SuryaNarayana Garlapati
Comments: 1. First of all, the mds sub part version should not be stored/used. You should depend on the message format version as SI rank is being embedded in the new message(which is being sent from AVD). If you have a requirement for usage of message version, you can get it at runtime when y

Re: [devel] [PATCH 1 of 3] MDS: TIPC performance improvement [#654]

2014-02-05 Thread SuryaNarayana Garlapati
On Wednesday 05 February 2014 05:09 PM, Hans Feldt wrote: > Great to see the performance improvement but seriously... > > This patch does the same thing my previous patch did in a more intrusive, > more complex and non scalable way. [Surya] This includes TCP as well, which your patch was missing.

Re: [devel] [PATCH 0 of 2] Review Request for MDS #654

2014-01-05 Thread SuryaNarayana Garlapati
This patch set doesn’t fully implement the suggestions/improvements that are mentioned in the Ticket. Can you please elaborate in detail the purpose of Patch 1 of 2? For the patch 2 of 2. We have a more better way of detecting the packet loss at the service level and the changes are present in

Re: [devel] [PATCH 0 of 2] Review Request for MDS #654

2013-12-19 Thread SuryaNarayana Garlapati
i am in the process of reviewing the patch. Will send the comments. On Monday 16 December 2013 03:23 PM, Hans Feldt wrote: > Summary: MDS improvements > Review request for Trac Ticket(s): 654 > Peer Reviewer(s): Mahesh, Surya > Pull request to: <> > Affected branch(es): default > Development branc

Re: [devel] [PATCH 1 of 1] amfnd: Reboot payload when link between Controller and Payload flickers [#600]

2013-11-18 Thread SuryaNarayana Garlapati
Original Message- >> From: SuryaNarayana Garlapati [mailto:suryanarayana.garlap...@oracle.com] >> Sent: den 18 november 2013 09:41 >> To: Nagendra Kumar; Hans Feldt; Hans Nordebäck; Praveen Malviya; Mathivanan >> Naickan Palanivelu >> Cc: opensaf-devel@lists.sourcefor

Re: [devel] [PATCH 1 of 1] amfnd: Reboot payload when link between Controller and Payload flickers [#600]

2013-11-18 Thread SuryaNarayana Garlapati
es like an orphan as Amfd will not > entertain any requests from Amfnd. > > Thanks > -Nagu > > -Original Message- > From: Hans Feldt [mailto:hans.fe...@ericsson.com] > Sent: 18 November 2013 12:56 > To: Nagendra Kumar; Suryanarayana Garlapati; Hans Nordebäck; Prave

Re: [devel] [PATCH 1 of 1] amfnd: Reboot payload when link between Controller and Payload flickers [#600]

2013-10-22 Thread Suryanarayana Garlapati
ecause of problem in payload, > others payload is impacted because of Act controller failover. > > Thanks > -Nagu > -Original Message----- > From: Suryanarayana Garlapati > Sent: 21 October 2013 18:22 > To: Nagendra Kumar; hans.fe...@ericsson.com; hans.nordeb...@ericsson.co

Re: [devel] [PATCH 1 of 1] amfnd: Reboot payload when link between Controller and Payload flickers [#600]

2013-10-21 Thread Suryanarayana Garlapati
Hi Nagu, I am not comfortable with this approach. I think its better to reboot the active controller if link flaps and not the payload node. If the link flaps between the active controller and payload nodes, then there will total payload cluster reset which we can avoid by just rebooting the act

Re: [devel] [PATCH 1 of 1] amfd: exit at MDS quiesced event in wrong state [#516]

2013-09-15 Thread Suryanarayana Garlapati
CTIVE detected, > exiting" instead of the "Controller split brain detected, exiting" [Surya] Ok. > > /Hans > > On 09/13/2013 12:52 PM, Suryanarayana Garlapati wrote: >> In the case where opensaf is shutting down and took more time to >> shutdown >> Some

Re: [devel] [PATCH 1 of 1] amfd: exit at MDS quiesced event in wrong state [#516]

2013-09-13 Thread Suryanarayana Garlapati
hursday 05 September 2013 03:09 PM, Hans Feldt wrote: > Any suggestion? > /Hans > >> -Original Message----- >> From: Suryanarayana Garlapati [mailto:suryanarayana.garlap...@oracle.com] >> Sent: den 5 september 2013 11:06 >> To: praveen malviya >> Cc: Han

Re: [devel] [PATCH 1 of 1] amfd: exit at MDS quiesced event in wrong state [#516]

2013-09-05 Thread Suryanarayana Garlapati
Change the following log accordingly: + LOG_CR("Controller split brain detected, exiting"); This is always not true. The above is true in Tipc link flap. On Thursday 05 September 2013 02:34 PM, praveen malviya wrote: > Ack, > > Thanks, > Praveen > On 16-Aug-13 7:03 PM, Hans Feldt

Re: [devel] [PATCH 1 of 1] amfd: exit at MDS quiesced event in wrong state [#516]

2013-08-19 Thread Suryanarayana Garlapati
In my perspective, this should be done at the Active Only and standby should just drop it. Atleast we will have the standby which gets promoted to active and continues to provide the service. We should not be performing a cluster reset. Thoughts?. On Friday 16 August 2013 07:03 PM, Hans Feldt