ssa.c: Commentary change

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
ssa.c: Change variable name in ssa_access_map_callback

for consistency with similar code in SSA_CONN_DONE message
handling in ssa_access_handler

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
acm.c: Add error return value to log message in acm_issue_query

Signed-off-by: Sasha Kotchubievsky <sashakot@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
acm.c: Cosmetic change to log message in acm_issue_query

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
ssa_extract.c: Add missing new line to end of a couple of log messages

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
plugin/ssa_extract.c: Increase log level for validate messages

Signed-off-by: Ilya Nelkenbaum <ilyan@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
ssa.c: In ssa_access_handler, recalculate PRDB when SMDB epoch has changed

and ref count object already exists when receiving SSA_CONN_DONE message

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
ssa_log.[h c]: Add ssa_get_log_level method

Signed-off-by: Ilya Nelkenbaum <ilyan@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
plugin/ssa_extract.c: Add log level checks in ssa_db_validate[_lft]

Signed-off-by: Ilya Nelkenbaum <ilyan@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
ssa.c: In ssa_upstream_initiate_conn, log source GID/LID for upstream connections

This improves ACM log readability.

Signed-off-by: Sasha Kotchubievsky <sashakot@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
acm: Add starting/ending cache update log messages to acm_parse_ssa_db

Signed-off-by: Ilya Nelkenbaum <ilyan@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
[core distrib].c: Add missing calls to ssa_close_log

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
acm/acme.c: Fix path records comparison

In case of verifying cached path record vs. path record
received from SA, compare only the relevant bits
(for rate & mtu those are 6 lsb bits).

Pointed out by Lenny Verkhovsky <lennyb@mellanox.com>,
during Kodiak cluster integration.

Signed-off-by: Ilya Nelkenbaum <ilyan@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
acm/acme.c: In show_path, fix mask for MTU, rate, and packet life time

Was using 0x1F rather than 0x3F as mask.
These fields are 6 rather than 5 bits.
More of spec conformance issue than practical issue as the
values fit into 5 bits.

Reported-by: Ilya Nelkenbaum <ilyan@mellanox.com>

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
[acm ssa].c: Add error message about another running ACM instance

Signed-off-by: Sasha Kotchubievsky <sashakot@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
distrib.c Add error message about another running ibssa instance

Signed-off-by: Sasha Kotchubievsky <sashakot@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
ssa.c: In ssa_upstream_handler, remove log message on upstream rsock error

which says "rsock should be but is not already closed".
This is normal occurence when upstream rclose's the connection
and is first detection of such event.

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
[distrib acm].c: Change to simplify maintenance

Rather than use explicit size of msg defined, use sizeof so
if msg array size changes, it is only changed in one place.

Signed-off-by: Hal Rosenstock <hal@mellanox.com>
acm.c Change log message in acm_issue_query to warning

Use ssa_log_warn rather than ssa_log and remove word ERROR
from log message.

Signed-off-by: Sasha Kotchubievsky <sashakot@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>
acm.c: Use short application name in "another instance" message

Signed-off-by: Sasha Kotchubievsky <sashakot@mellanox.com>
Signed-off-by: Hal Rosenstock <hal@mellanox.com>