[devel] [PATCH 1 of 1] mds: Remove unnecessary syslog message about TIPC importance [#755]

2014-01-30 Thread Anders Widell
 osaf/libs/core/mds/mds_dt_tipc.c |  3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)


Remove an unnecessary log message regarding the configured TIPC importance,
that was introduced in ticket [#589]. The information in the log message is
unnecessary since the same information can be found in the README file.

diff --git a/osaf/libs/core/mds/mds_dt_tipc.c b/osaf/libs/core/mds/mds_dt_tipc.c
--- a/osaf/libs/core/mds/mds_dt_tipc.c
+++ b/osaf/libs/core/mds/mds_dt_tipc.c
@@ -1306,9 +1306,6 @@ uint32_t mds_mdtm_svc_install_tipc(PW_EN
int imp = 0;
imp = TIPC_HIGH_IMPORTANCE;
 
-   if (TIPCIMPORTANCE == TIPC_HIGH_IMPORTANCE)
-   LOG_WA("Default TIPC importance set to same level as 
AVND (TIPC_HIGH_IMPORTANCE). A non preferred and untested option");
-
if (setsockopt(tipc_cb.BSRsock, SOL_TIPC, TIPC_IMPORTANCE, 
&imp, sizeof(imp)) != 0) {
m_MDS_LOG_ERR("MDTM: Can't set socket option TIPC_IMP 
err :%s\n", strerror(errno));
assert(0);

--
WatchGuard Dimension instantly turns raw network data into actionable 
security intelligence. It gives you real-time visual feedback on key
security issues and trends.  Skip the complicated setup - simply import
a virtual appliance and go from zero to informed in seconds.
http://pubads.g.doubleclick.net/gampad/clk?id=123612991&iu=/4140/ostg.clktrk
___
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel


Re: [devel] [PATCH 1 of 1] mds: Remove unnecessary syslog message about TIPC importance [#755]

2014-02-02 Thread A V Mahesh
ACK from  me.

-AVM

On 1/30/2014 9:19 PM, Anders Widell wrote:
>   osaf/libs/core/mds/mds_dt_tipc.c |  3 ---
>   1 files changed, 0 insertions(+), 3 deletions(-)
>
>
> Remove an unnecessary log message regarding the configured TIPC importance,
> that was introduced in ticket [#589]. The information in the log message is
> unnecessary since the same information can be found in the README file.
>
> diff --git a/osaf/libs/core/mds/mds_dt_tipc.c 
> b/osaf/libs/core/mds/mds_dt_tipc.c
> --- a/osaf/libs/core/mds/mds_dt_tipc.c
> +++ b/osaf/libs/core/mds/mds_dt_tipc.c
> @@ -1306,9 +1306,6 @@ uint32_t mds_mdtm_svc_install_tipc(PW_EN
>   int imp = 0;
>   imp = TIPC_HIGH_IMPORTANCE;
>   
> - if (TIPCIMPORTANCE == TIPC_HIGH_IMPORTANCE)
> - LOG_WA("Default TIPC importance set to same level as 
> AVND (TIPC_HIGH_IMPORTANCE). A non preferred and untested option");
> -
>   if (setsockopt(tipc_cb.BSRsock, SOL_TIPC, TIPC_IMPORTANCE, 
> &imp, sizeof(imp)) != 0) {
>   m_MDS_LOG_ERR("MDTM: Can't set socket option TIPC_IMP 
> err :%s\n", strerror(errno));
>   assert(0);


--
Managing the Performance of Cloud-Based Applications
Take advantage of what the Cloud has to offer - Avoid Common Pitfalls.
Read the Whitepaper.
http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk
___
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel