Hi Tai,

forAddRemove  we can have the following cases:
1.  only  deactivationUnit
2. only  activationUnit
3. both

For case 1 and 2 we can not optimize for node/Su/Comp.
For case 3 we can optimize for node/Su/comp.

I think the published patch needs to be corrected.
i.e if node/SU/Comp is present in both activation and deactivation then 
only optimize, otherwise do not optimize.

Thanks,
Neel.



On 2016/12/02 05:44 PM, Tai Chi DINH wrote:
>
> Hi Neel,
>
> I think we also need to remove any duplication under SU level and 
> Component Level also.
> Example we have the original campaign that have:
> - Rolling on SCs
> - ForModify on SU1, SU2 that are hosted on PLs
> - ForAddRemoved on SU1, SU2.
>
> Which this patch, the result campaign will have AU/DU on 
> SCs/SU1/SU2/SU1/SU2.
> Which means we have redundant/unnecessary lock/unlock of SU1/SU2 (it's 
> enough to just lock/unlock them only once).
>
> How do you think?
>
> /Tai
> Quoting Neelakanta Reddy <reddy.neelaka...@oracle.com 
> <mailto:reddy.neelaka...@oracle.com>>:
>
>> Hi All,
>>
>> Here the defect no is #2209 not #2214
>>
>> Thanks,
>> Neel.
>>
>> On 2016/12/02 04:22 PM, reddy.neelaka...@oracle.com 
>> <mailto:reddy.neelaka...@oracle.com> wrote:
>>> Summary: smf:Allow optimization at node level forAddRemove in 
>>> mergeStepIntoSingle[#2214]
>>> Review request for Trac Ticket(s): 2214
>>> Peer Reviewer(s): Rafael, Lennart, tai
>>> Affected branch(es): 5.0.x, 5.1.x, default
>>> Development branch:default
>>>
>>> --------------------------------
>>> Impacted area       Impact y/n
>>> --------------------------------
>>> Docs                    n
>>> Build system            n
>>> RPM/packaging           n
>>> Configuration files     n
>>> Startup scripts         n
>>> SAF services            y
>>> OpenSAF services        n
>>> Core libraries          n
>>> Samples                 n
>>> Tests                   n
>>> Other                   n
>>>
>>>
>>> Comments (indicate scope for each "y" above):
>>> ---------------------------------------------
>>>
>>> changeset 2becbe07a7f92d70f928e23dcd6b0a6576c8e22a
>>> Author:        Neelakanta Reddy <reddy.neelaka...@oracle.com 
>>> <mailto:reddy.neelaka...@oracle.com>>
>>> Date:        Fri, 02 Dec 2016 16:16:33 +0530
>>>
>>>         smf:Allow optimization at node level forAddRemove in
>>>         mergeStepIntoSingle[#2214]
>>>
>>>
>>> Complete diffstat:
>>> ------------------
>>> osaf/services/saf/smfsv/smfd/SmfUpgradeProcedure.cc |  40 
>>> +++++++++++++++++++++++++++++++++++++++-
>>> 1 files changed, 39 insertions(+), 1 deletions(-)
>>>
>>>
>>> Testing Commands:
>>> -----------------
>>> campaign must contain rolling and singlestep upgrade with AU/SU node 
>>> level
>>>
>>> Testing, Expected Results:
>>> --------------------------
>>> Campaign should not fail
>>>
>>> Conditions of Submission:
>>> -------------------------
>>> Ack from Reviewers
>>>
>>> Arch      Built     Started    Linux distro
>>> -------------------------------------------
>>> mips        n          n
>>> mips64      n          n
>>> x86         n          n
>>> x86_64      y          y
>>> powerpc     n          n
>>> powerpc64   n          n
>>>
>>>
>>> Reviewer Checklist:
>>> -------------------
>>> [Submitters: make sure that your review doesn't trigger any checkmarks!]
>>>
>>>
>>> Your checkin has not passed review because (see checked entries):
>>>
>>> ___ Your RR template is generally incomplete; it has too many blank 
>>> entries
>>>     that need proper data filled in.
>>>
>>> ___ You have failed to nominate the proper persons for review and push.
>>>
>>> ___ Your patches do not have proper short+long header
>>>
>>> ___ You have grammar/spelling in your header that is unacceptable.
>>>
>>> ___ You have exceeded a sensible line length in your 
>>> headers/comments/text.
>>>
>>> ___ You have failed to put in a proper Trac Ticket # into your commits.
>>>
>>> ___ You have incorrectly put/left internal data in your comments/files
>>>     (i.e. internal bug tracking tool IDs, product names etc)
>>>
>>> ___ You have not given any evidence of testing beyond basic build tests.
>>>     Demonstrate some level of runtime or other sanity testing.
>>>
>>> ___ You have ^M present in some of your files. These have to be removed.
>>>
>>> ___ You have needlessly changed whitespace or added whitespace crimes
>>>     like trailing spaces, or spaces before tabs.
>>>
>>> ___ You have mixed real technical changes with whitespace and other
>>>     cosmetic code cleanup changes. These have to be separate commits.
>>>
>>> ___ You need to refactor your submission into logical chunks; there is
>>>     too much content into a single commit.
>>>
>>> ___ You have extraneous garbage in your review (merge commits etc)
>>>
>>> ___ You have giant attachments which should never have been sent;
>>>     Instead you should place your content in a public tree to be pulled.
>>>
>>> ___ You have too many commits attached to an e-mail; resend as threaded
>>>     commits, or place in a public tree for a pull.
>>>
>>> ___ You have resent this content multiple times without a clear 
>>> indication
>>>     of what has changed between each re-send.
>>>
>>> ___ You have failed to adequately and individually address all of the
>>>     comments and change requests that were proposed in the initial 
>>> review.
>>>
>>> ___ You have a misconfigured ~/.hgrc file (i.e. username, email etc)
>>>
>>> ___ Your computer have a badly configured date and time; confusing the
>>>     the threaded patch review.
>>>
>>> ___ Your changes affect IPC mechanism, and you don't present any results
>>>     for in-service upgradability test.
>>>
>>> ___ Your changes affect user manual and documentation, your patch series
>>>     do not contain the patch that updates the Doxygen manual.
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> Check out the vibrant tech community on one of the world's most
>>> engaging tech sites, SlashDot.org! http://sdm.link/slashdot
>>> _______________________________________________
>>> Opensaf-devel mailing list
>>> Opensaf-devel@lists.sourceforge 
>>> <mailto:Opensaf-devel@lists.sourceforge>.nethttps://lists.sourceforge.net/lists/listinfo/opensaf-devel
>>

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to