***SPAM*** Re: [ofa-general] Re: [PATCHv2][TRIVIAL] OpenSM: Display port number in decimal in log messages

Hal Rosenstock hal.rosenstock at gmail.com
Wed Oct 8 04:04:05 PDT 2008


Sasha,

On Tue, Oct 7, 2008 at 7:53 PM, Sasha Khapyorsky <sashak at voltaire.com> wrote:
> Hi Hal,
>
> On 14:07 Fri 03 Oct     , Hal Rosenstock wrote:
>>
>> Cosmetic patch attached. Found some more cases...
>
> The patch is broken (it has parts from previous patch) and doesn't apply
> clearly.So I was need to apply this to another branch and rebase.

v2 means a replacement patch not built on top of the previous one.

>Also please next time add 'Sign-off-by'.

Sorry; forgot that this time.

-- Hal

> Sasha
>
>>
>> -- Hal
>
>> diff --git a/opensm/opensm/osm_drop_mgr.c b/opensm/opensm/osm_drop_mgr.c
>> index e827c26..8c6e7fb 100644
>> --- a/opensm/opensm/osm_drop_mgr.c
>> +++ b/opensm/opensm/osm_drop_mgr.c
>> @@ -103,7 +103,7 @@ static void drop_mgr_clean_physp(osm_sm_t * sm, IN osm_physp_t * p_physp)
>>                           IB_LINK_ACTIVE) {
>>                               OSM_LOG(sm->p_log, OSM_LOG_VERBOSE,
>>                                       "Forcing new heavy sweep. Remote "
>> -                                     "port 0x%016" PRIx64 " port num: 0x%X "
>> +                                     "port 0x%016" PRIx64 " port num: %u "
>>                                       "was recognized in ACTIVE state\n",
>>                                       cl_ntoh64(p_remote_physp->port_guid),
>>                                       p_remote_physp->port_num);
>> @@ -117,7 +117,7 @@ static void drop_mgr_clean_physp(osm_sm_t * sm, IN osm_physp_t * p_physp)
>>                               p_remote_port->discovery_count = 0;
>>                               OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>>                                       "Resetting discovery count of node: "
>> -                                     "0x%016" PRIx64 " port num:0x%X\n",
>> +                                     "0x%016" PRIx64 " port num:%u\n",
>>                                       cl_ntoh64(osm_node_get_node_guid
>>                                                 (p_remote_physp->p_node)),
>>                                       p_remote_physp->port_num);
>> @@ -125,9 +125,9 @@ static void drop_mgr_clean_physp(osm_sm_t * sm, IN osm_physp_t * p_physp)
>>               }
>>
>>               OSM_LOG(sm->p_log, OSM_LOG_VERBOSE,
>> -                     "Unlinking local node 0x%016" PRIx64 ", port 0x%X"
>> +                     "Unlinking local node 0x%016" PRIx64 ", port %u"
>>                       "\n\t\t\t\tand remote node 0x%016" PRIx64
>> -                     ", port 0x%X\n",
>> +                     ", port %u\n",
>>                       cl_ntoh64(osm_node_get_node_guid(p_physp->p_node)),
>>                       p_physp->port_num,
>>                       cl_ntoh64(osm_node_get_node_guid
>> @@ -139,7 +139,7 @@ static void drop_mgr_clean_physp(osm_sm_t * sm, IN osm_physp_t * p_physp)
>>       }
>>
>>       OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>> -             "Clearing node 0x%016" PRIx64 " physical port number 0x%X\n",
>> +             "Clearing node 0x%016" PRIx64 " physical port number %u\n",
>>               cl_ntoh64(osm_node_get_node_guid(p_physp->p_node)),
>>               p_physp->port_num);
>>
>> @@ -186,7 +186,8 @@ static void __osm_drop_mgr_remove_port(osm_sm_t * sm, IN osm_port_t * p_port)
>>       if (p_sm != (osm_remote_sm_t *) cl_qmap_end(p_sm_guid_tbl)) {
>>               /* need to remove this item */
>>               OSM_LOG(sm->p_log, OSM_LOG_VERBOSE,
>> -                     "Cleaned SM for port guid\n");
>> +                     "Cleaned SM for port guid 0x%016" PRIx64 "\n",
>> +                     cl_ntoh64(port_guid));
>>
>>               free(p_sm);
>>       }
>> diff --git a/opensm/opensm/osm_helper.c b/opensm/opensm/osm_helper.c
>> index 4c67c3f..45a100e 100644
>> --- a/opensm/opensm/osm_helper.c
>> +++ b/opensm/opensm/osm_helper.c
>> @@ -780,7 +780,7 @@ osm_dump_port_info(IN osm_log_t * const p_log,
>>
>>               osm_log(p_log, log_level,
>>                       "PortInfo dump:\n"
>> -                     "\t\t\t\tport number.............0x%X\n"
>> +                     "\t\t\t\tport number.............%u\n"
>>                       "\t\t\t\tnode_guid...............0x%016" PRIx64 "\n"
>>                       "\t\t\t\tport_guid...............0x%016" PRIx64 "\n"
>>                       "\t\t\t\tm_key...................0x%016" PRIx64 "\n"
>> @@ -790,7 +790,7 @@ osm_dump_port_info(IN osm_log_t * const p_log,
>>                       "\t\t\t\tcapability_mask.........0x%X\n"
>>                       "\t\t\t\tdiag_code...............0x%X\n"
>>                       "\t\t\t\tm_key_lease_period......0x%X\n"
>> -                     "\t\t\t\tlocal_port_num..........0x%X\n"
>> +                     "\t\t\t\tlocal_port_num..........%u\n"
>>                       "\t\t\t\tlink_width_enabled......0x%X\n"
>>                       "\t\t\t\tlink_width_supported....0x%X\n"
>>                       "\t\t\t\tlink_width_active.......0x%X\n"
>> @@ -879,7 +879,7 @@ osm_dump_portinfo_record(IN osm_log_t * const p_log,
>>                       "\t\t\t\tcapability_mask.........0x%X\n"
>>                       "\t\t\t\tdiag_code...............0x%X\n"
>>                       "\t\t\t\tm_key_lease_period......0x%X\n"
>> -                     "\t\t\t\tlocal_port_num..........0x%X\n"
>> +                     "\t\t\t\tlocal_port_num..........%u\n"
>>                       "\t\t\t\tlink_width_enabled......0x%X\n"
>>                       "\t\t\t\tlink_width_supported....0x%X\n"
>>                       "\t\t\t\tlink_width_active.......0x%X\n"
>> @@ -994,14 +994,14 @@ osm_dump_node_info(IN osm_log_t * const p_log,
>>                       "\t\t\t\tbase_version............0x%X\n"
>>                       "\t\t\t\tclass_version...........0x%X\n"
>>                       "\t\t\t\tnode_type...............%s\n"
>> -                     "\t\t\t\tnum_ports...............0x%X\n"
>> +                     "\t\t\t\tnum_ports...............%u\n"
>>                       "\t\t\t\tsys_guid................0x%016" PRIx64 "\n"
>>                       "\t\t\t\tnode_guid...............0x%016" PRIx64 "\n"
>>                       "\t\t\t\tport_guid...............0x%016" PRIx64 "\n"
>>                       "\t\t\t\tpartition_cap...........0x%X\n"
>>                       "\t\t\t\tdevice_id...............0x%X\n"
>>                       "\t\t\t\trevision................0x%X\n"
>> -                     "\t\t\t\tport_num................0x%X\n"
>> +                     "\t\t\t\tport_num................%u\n"
>>                       "\t\t\t\tvendor_id...............0x%X\n",
>>                       p_ni->base_version,
>>                       p_ni->class_version,
>> @@ -1041,14 +1041,14 @@ osm_dump_node_record(IN osm_log_t * const p_log,
>>                       "\t\t\t\tbase_version............0x%X\n"
>>                       "\t\t\t\tclass_version...........0x%X\n"
>>                       "\t\t\t\tnode_type...............%s\n"
>> -                     "\t\t\t\tnum_ports...............0x%X\n"
>> +                     "\t\t\t\tnum_ports...............%u\n"
>>                       "\t\t\t\tsys_guid................0x%016" PRIx64 "\n"
>>                       "\t\t\t\tnode_guid...............0x%016" PRIx64 "\n"
>>                       "\t\t\t\tport_guid...............0x%016" PRIx64 "\n"
>>                       "\t\t\t\tpartition_cap...........0x%X\n"
>>                       "\t\t\t\tdevice_id...............0x%X\n"
>>                       "\t\t\t\trevision................0x%X\n"
>> -                     "\t\t\t\tport_num................0x%X\n"
>> +                     "\t\t\t\tport_num................%u\n"
>>                       "\t\t\t\tvendor_id...............0x%X\n"
>>                       "\t\t\t\tNodeDescription\n"
>>                       "\t\t\t\t%s\n",
>> @@ -1489,8 +1489,8 @@ osm_dump_link_record(IN osm_log_t * const p_log,
>>               osm_log(p_log, log_level,
>>                       "Link Record dump:\n"
>>                       "\t\t\t\tfrom_lid................%u\n"
>> -                     "\t\t\t\tfrom_port_num...........0x%X\n"
>> -                     "\t\t\t\tto_port_num.............0x%X\n"
>> +                     "\t\t\t\tfrom_port_num...........%u\n"
>> +                     "\t\t\t\tto_port_num.............%u\n"
>>                       "\t\t\t\tto_lid..................%u\n",
>>                       cl_ntoh16(p_lr->from_lid),
>>                       p_lr->from_port_num,
>> @@ -1512,9 +1512,9 @@ osm_dump_switch_info(IN osm_log_t * const p_log,
>>                       "\t\t\t\trand_cap................0x%X\n"
>>                       "\t\t\t\tmcast_cap...............0x%X\n"
>>                       "\t\t\t\tlin_top.................0x%X\n"
>> -                     "\t\t\t\tdef_port................0x%X\n"
>> -                     "\t\t\t\tdef_mcast_pri_port......0x%X\n"
>> -                     "\t\t\t\tdef_mcast_not_port......0x%X\n"
>> +                     "\t\t\t\tdef_port................%u\n"
>> +                     "\t\t\t\tdef_mcast_pri_port......%u\n"
>> +                     "\t\t\t\tdef_mcast_not_port......%u\n"
>>                       "\t\t\t\tlife_state..............0x%X\n"
>>                       "\t\t\t\tlids_per_port...........%u\n"
>>                       "\t\t\t\tpartition_enf_cap.......0x%X\n"
>> @@ -1549,9 +1549,9 @@ osm_dump_switch_info_record(IN osm_log_t * const p_log,
>>                       "\t\t\t\trand_cap................0x%X\n"
>>                       "\t\t\t\tmcast_cap...............0x%X\n"
>>                       "\t\t\t\tlin_top.................0x%X\n"
>> -                     "\t\t\t\tdef_port................0x%X\n"
>> -                     "\t\t\t\tdef_mcast_pri_port......0x%X\n"
>> -                     "\t\t\t\tdef_mcast_not_port......0x%X\n"
>> +                     "\t\t\t\tdef_port................%u\n"
>> +                     "\t\t\t\tdef_mcast_pri_port......%u\n"
>> +                     "\t\t\t\tdef_mcast_not_port......%u\n"
>>                       "\t\t\t\tlife_state..............0x%X\n"
>>                       "\t\t\t\tlids_per_port...........%u\n"
>>                       "\t\t\t\tpartition_enf_cap.......0x%X\n"
>> @@ -1593,7 +1593,7 @@ osm_dump_pkey_block(IN osm_log_t * const p_log,
>>                       "P_Key table dump:\n"
>>                       "\t\t\tport_guid...........0x%016" PRIx64 "\n"
>>                       "\t\t\tblock_num...........0x%X\n"
>> -                     "\t\t\tport_num............0x%X\n\tP_Key Table: %s\n",
>> +                     "\t\t\tport_num............%u\n\tP_Key Table: %s\n",
>>                       cl_ntoh64(port_guid), block_num, port_num, buf_line);
>>       }
>>  }
>> @@ -1621,8 +1621,8 @@ osm_dump_slvl_map_table(IN osm_log_t * const p_log,
>>               osm_log(p_log, log_level,
>>                       "SLtoVL dump:\n"
>>                       "\t\t\tport_guid............0x%016" PRIx64 "\n"
>> -                     "\t\t\tin_port_num..........0x%X\n"
>> -                     "\t\t\tout_port_num.........0x%X\n\tSL: | %s\n\tVL: | %s\n",
>> +                     "\t\t\tin_port_num..........%u\n"
>> +                     "\t\t\tout_port_num.........%u\n\tSL: | %s\n\tVL: | %s\n",
>>                       cl_ntoh64(port_guid),
>>                       in_port_num, out_port_num, buf_line1, buf_line2);
>>       }
>> @@ -1651,7 +1651,7 @@ osm_dump_vl_arb_table(IN osm_log_t * const p_log,
>>               osm_log(p_log, log_level,
>>                       "VLArb dump:\n" "\t\t\tport_guid...........0x%016"
>>                       PRIx64 "\n" "\t\t\tblock_num...........0x%X\n"
>> -                     "\t\t\tport_num............0x%X\n\tVL    : | %s\n\tWEIGHT:| %s\n",
>> +                     "\t\t\tport_num............%u\n\tVL    : | %s\n\tWEIGHT:| %s\n",
>>                       cl_ntoh64(port_guid), block_num, port_num, buf_line1,
>>                       buf_line2);
>>       }
>> diff --git a/opensm/opensm/osm_link_mgr.c b/opensm/opensm/osm_link_mgr.c
>> index 452d151..98870cb 100644
>> --- a/opensm/opensm/osm_link_mgr.c
>> +++ b/opensm/opensm/osm_link_mgr.c
>> @@ -377,7 +377,7 @@ __osm_link_mgr_process_node(osm_sm_t * sm,
>>               if (link_state != IB_LINK_NO_CHANGE &&
>>                   link_state <= current_state)
>>                       OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>> -                             "Physical port 0x%X already %s. Skipping\n",
>> +                             "Physical port %u already %s. Skipping\n",
>>                               p_physp->port_num,
>>                               ib_get_port_state_str(current_state));
>>               else if (__osm_link_mgr_set_physp_pi(sm, p_physp, link_state))
>> diff --git a/opensm/opensm/osm_mcast_mgr.c b/opensm/opensm/osm_mcast_mgr.c
>> index fc8533d..c4cd632 100644
>> --- a/opensm/opensm/osm_mcast_mgr.c
>> +++ b/opensm/opensm/osm_mcast_mgr.c
>> @@ -628,7 +628,7 @@ static osm_mtree_node_t *__osm_mcast_mgr_branch(osm_sm_t * sm,
>>        */
>>       if (depth > 1) {
>>               OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>> -                     "Adding upstream port 0x%X\n", upstream_port);
>> +                     "Adding upstream port %u\n", upstream_port);
>>
>>               CL_ASSERT(upstream_port);
>>               osm_mcast_tbl_set(p_tbl, mlid_ho, upstream_port);
>> @@ -662,7 +662,7 @@ static osm_mtree_node_t *__osm_mcast_mgr_branch(osm_sm_t * sm,
>>                       continue;       /* No routes down this port. */
>>
>>               OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>> -                     "Routing %zu destinations via switch port 0x%X\n",
>> +                     "Routing %zu destinations via switch port %u\n",
>>                       count, i);
>>
>>               /*
>> @@ -716,7 +716,7 @@ static osm_mtree_node_t *__osm_mcast_mgr_branch(osm_sm_t * sm,
>>
>>                       OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>>                               "Found leaf for port 0x%016" PRIx64
>> -                             " on switch port 0x%X\n",
>> +                             " on switch port %u\n",
>>                               cl_ntoh64(osm_port_get_guid (p_wobj->p_port)),
>>                               i);
>>
>> diff --git a/opensm/opensm/osm_node_info_rcv.c b/opensm/opensm/osm_node_info_rcv.c
>> index 86710d1..a37ce0a 100644
>> --- a/opensm/opensm/osm_node_info_rcv.c
>> +++ b/opensm/opensm/osm_node_info_rcv.c
>> @@ -217,7 +217,7 @@ __osm_ni_rcv_set_links(IN osm_sm_t * sm,
>>           port_num != 0 && cl_qmap_count(&sm->p_subn->sw_guid_tbl) == 0) {
>>               OSM_LOG(sm->p_log, OSM_LOG_VERBOSE,
>>                       "Duplicate GUID found by link from a port to itself:"
>> -                     "node 0x%" PRIx64 ", port number 0x%X\n",
>> +                     "node 0x%" PRIx64 ", port number %u\n",
>>                       cl_ntoh64(osm_node_get_node_guid(p_node)), port_num);
>>               p_physp = osm_node_get_physp_ptr(p_node, port_num);
>>               osm_dump_dr_path(sm->p_log,
>> @@ -235,8 +235,8 @@ __osm_ni_rcv_set_links(IN osm_sm_t * sm,
>>
>>       OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>>               "Creating new link between:\n\t\t\t\tnode 0x%" PRIx64
>> -             ", port number 0x%X and\n\t\t\t\tnode 0x%" PRIx64
>> -             ", port number 0x%X\n",
>> +             ", port number %u and\n\t\t\t\tnode 0x%" PRIx64
>> +             ", port number %u\n",
>>               cl_ntoh64(osm_node_get_node_guid(p_node)), port_num,
>>               cl_ntoh64(p_ni_context->node_guid), p_ni_context->port_num);
>>
>> diff --git a/opensm/opensm/osm_perfmgr.c b/opensm/opensm/osm_perfmgr.c
>> index 8d292ab..a168f89 100644
>> --- a/opensm/opensm/osm_perfmgr.c
>> +++ b/opensm/opensm/osm_perfmgr.c
>> @@ -219,7 +219,7 @@ osm_perfmgr_mad_send_err_callback(void *bind_context, osm_madw_t * p_madw)
>>       p_mon_node = (__monitored_node_t *) p_node;
>>
>>       OSM_LOG(pm->log, OSM_LOG_ERROR, "ERR 4C02: %s (0x%" PRIx64
>> -             ") port %d\n", p_mon_node->name, p_mon_node->guid, port);
>> +             ") port %u\n", p_mon_node->name, p_mon_node->guid, port);
>>
>>       if (pm->subn->opt.perfmgr_redir && p_madw->status == IB_TIMEOUT) {
>>               /* First, find the node in the monitored map */
>> @@ -228,8 +228,8 @@ osm_perfmgr_mad_send_err_callback(void *bind_context, osm_madw_t * p_madw)
>>               if (port > p_mon_node->redir_tbl_size) {
>>                       cl_plock_release(pm->lock);
>>                       OSM_LOG(pm->log, OSM_LOG_ERROR, "ERR 4C16: "
>> -                             "Invalid port num %d for %s (GUID 0x%016"
>> -                             PRIx64 ") num ports %d\n", port, p_mon_node->name,
>> +                             "Invalid port num %u for %s (GUID 0x%016"
>> +                             PRIx64 ") num ports %u\n", port, p_mon_node->name,
>>                               p_mon_node->guid, p_mon_node->redir_tbl_size);
>>                       goto Exit;
>>               }
>> diff --git a/opensm/opensm/osm_pkey_rcv.c b/opensm/opensm/osm_pkey_rcv.c
>> index 9e09806..d845c20 100644
>> --- a/opensm/opensm/osm_pkey_rcv.c
>> +++ b/opensm/opensm/osm_pkey_rcv.c
>> @@ -127,7 +127,7 @@ void osm_pkey_rcv_process(IN void *context, IN void *data)
>>        */
>>       if (!p_physp) {
>>               OSM_LOG(sm->p_log, OSM_LOG_ERROR, "ERR 4807: "
>> -                     "Got invalid port number 0x%X\n", port_num);
>> +                     "Got invalid port number %u\n", port_num);
>>               goto Exit;
>>       }
>>
>> diff --git a/opensm/opensm/osm_port.c b/opensm/opensm/osm_port.c
>> index 54a5fd1..48a12f9 100644
>> --- a/opensm/opensm/osm_port.c
>> +++ b/opensm/opensm/osm_port.c
>> @@ -322,7 +322,7 @@ osm_physp_calc_link_mtu(IN osm_log_t * p_log, IN const osm_physp_t * p_physp)
>>                   ib_port_info_get_mtu_cap(&p_remote_physp->port_info);
>>
>>               OSM_LOG(p_log, OSM_LOG_DEBUG,
>> -                     "Remote port 0x%016" PRIx64 " port = 0x%X : "
>> +                     "Remote port 0x%016" PRIx64 " port = %u : "
>>                       "MTU = %u. This Port MTU: %u\n",
>>                       cl_ntoh64(osm_physp_get_port_guid(p_remote_physp)),
>>                       osm_physp_get_port_num(p_remote_physp),
>> @@ -334,8 +334,8 @@ osm_physp_calc_link_mtu(IN osm_log_t * p_log, IN const osm_physp_t * p_physp)
>>
>>                       OSM_LOG(p_log, OSM_LOG_VERBOSE,
>>                               "MTU mismatch between ports."
>> -                             "\n\t\t\t\tPort 0x%016" PRIx64 ", port 0x%X"
>> -                             " and port 0x%016" PRIx64 ", port 0x%X."
>> +                             "\n\t\t\t\tPort 0x%016" PRIx64 ", port %u"
>> +                             " and port 0x%016" PRIx64 ", port %u."
>>                               "\n\t\t\t\tUsing lower MTU of %u\n",
>>                               cl_ntoh64(osm_physp_get_port_guid(p_physp)),
>>                               osm_physp_get_port_num(p_physp),
>> @@ -769,7 +769,7 @@ osm_physp_set_pkey_tbl(IN osm_log_t * p_log,
>>       if (block_num >= max_blocks) {
>>               OSM_LOG(p_log, OSM_LOG_ERROR, "ERR 4108: "
>>                       "Got illegal set for block number:%u "
>> -                     "For GUID: %" PRIx64 " port number:0x%X\n",
>> +                     "For GUID: %" PRIx64 " port number:%u\n",
>>                       block_num,
>>                       cl_ntoh64(p_physp->p_node->node_info.node_guid),
>>                       p_physp->port_num);
>> diff --git a/opensm/opensm/osm_port_info_rcv.c b/opensm/opensm/osm_port_info_rcv.c
>> index a820069..73afd8e 100644
>> --- a/opensm/opensm/osm_port_info_rcv.c
>> +++ b/opensm/opensm/osm_port_info_rcv.c
>> @@ -235,9 +235,9 @@ __osm_pi_rcv_process_switch_port(IN osm_sm_t * sm,
>>
>>                               OSM_LOG(sm->p_log, OSM_LOG_VERBOSE,
>>                                       "Unlinking local node 0x%" PRIx64
>> -                                     ", port 0x%X"
>> +                                     ", port %u"
>>                                       "\n\t\t\t\tand remote node 0x%" PRIx64
>> -                                     ", port 0x%X\n",
>> +                                     ", port %u\n",
>>                                       cl_ntoh64(osm_node_get_node_guid
>>                                                 (p_node)), port_num,
>>                                       cl_ntoh64(osm_node_get_node_guid
>> @@ -292,13 +292,13 @@ __osm_pi_rcv_process_switch_port(IN osm_sm_t * sm,
>>                                               ib_get_err_str(status));
>>                       } else
>>                               OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>> -                                     "Skipping SMP responder port 0x%X\n",
>> +                                     "Skipping SMP responder port %u\n",
>>                                       p_pi->local_port_num);
>>                       break;
>>
>>               default:
>>                       OSM_LOG(sm->p_log, OSM_LOG_ERROR, "ERR 0F03: "
>> -                             "Unknown link state = %u, port = 0x%X\n",
>> +                             "Unknown link state = %u, port = %u\n",
>>                               ib_port_info_get_port_state(p_pi),
>>                               p_pi->local_port_num);
>>                       break;
>> @@ -488,7 +488,7 @@ osm_pi_rcv_process_set(IN osm_sm_t * sm, IN osm_node_t * const p_node,
>>
>>       OSM_LOG(sm->p_log, OSM_LOG_DEBUG,
>>               "Received logical SetResp() for GUID 0x%" PRIx64
>> -             ", port num 0x%X"
>> +             ", port num %u"
>>               "\n\t\t\t\tfor parent node GUID 0x%" PRIx64
>>               " TID 0x%" PRIx64 "\n",
>>               cl_ntoh64(port_guid), port_num,
>> @@ -598,7 +598,7 @@ void osm_pi_rcv_process(IN void *context, IN void *data)
>>                  most likely due to a subnet sweep in progress.
>>                */
>>               OSM_LOG(sm->p_log, OSM_LOG_VERBOSE,
>> -                     "Discovered port num 0x%X with GUID 0x%" PRIx64
>> +                     "Discovered port num %u with GUID 0x%" PRIx64
>>                       " for parent node GUID 0x%" PRIx64
>>                       ", TID 0x%" PRIx64 "\n",
>>                       port_num, cl_ntoh64(port_guid),
>> @@ -613,7 +613,7 @@ void osm_pi_rcv_process(IN void *context, IN void *data)
>>                */
>>               if (!p_physp) {
>>                       OSM_LOG(sm->p_log, OSM_LOG_VERBOSE,
>> -                             "Initializing port number 0x%X\n", port_num);
>> +                             "Initializing port number %u\n", port_num);
>>                       p_physp = &p_node->physp_table[port_num];
>>                       osm_physp_init(p_physp,
>>                                      port_guid,
>> diff --git a/opensm/opensm/osm_sa_link_record.c b/opensm/opensm/osm_sa_link_record.c
>> index 9b1ad90..8e74297 100644
>> --- a/opensm/opensm/osm_sa_link_record.c
>> +++ b/opensm/opensm/osm_sa_link_record.c
>> @@ -194,8 +194,8 @@ __osm_lr_rcv_get_physp_link(IN osm_sa_t * sa,
>>                       goto Exit;
>>
>>       OSM_LOG(sa->p_log, OSM_LOG_DEBUG, "Acquiring link record\n"
>> -             "\t\t\t\tsrc port 0x%" PRIx64 " (port 0x%X)"
>> -             ", dest port 0x%" PRIx64 " (port 0x%X)\n",
>> +             "\t\t\t\tsrc port 0x%" PRIx64 " (port %u)"
>> +             ", dest port 0x%" PRIx64 " (port %u)\n",
>>               cl_ntoh64(osm_physp_get_port_guid(p_src_physp)), src_port_num,
>>               cl_ntoh64(osm_physp_get_port_guid(p_dest_physp)),
>>               dest_port_num);
>> diff --git a/opensm/opensm/osm_sa_pkey_record.c b/opensm/opensm/osm_sa_pkey_record.c
>> index bba01b4..b694c8c 100644
>> --- a/opensm/opensm/osm_sa_pkey_record.c
>> +++ b/opensm/opensm/osm_sa_pkey_record.c
>> @@ -93,7 +93,7 @@ __osm_sa_pkey_create(IN osm_sa_t * sa,
>>
>>       OSM_LOG(sa->p_log, OSM_LOG_DEBUG,
>>               "New P_Key table for: port 0x%016" PRIx64
>> -             ", lid %u, port 0x%X Block:%u\n",
>> +             ", lid %u, port %u Block:%u\n",
>>               cl_ntoh64(osm_physp_get_port_guid(p_physp)),
>>               cl_ntoh16(lid), osm_physp_get_port_num(p_physp), block);
>>
>> diff --git a/opensm/opensm/osm_sa_portinfo_record.c b/opensm/opensm/osm_sa_portinfo_record.c
>> index b263ce8..2ac2611 100644
>> --- a/opensm/opensm/osm_sa_portinfo_record.c
>> +++ b/opensm/opensm/osm_sa_portinfo_record.c
>> @@ -94,7 +94,7 @@ __osm_pir_rcv_new_pir(IN osm_sa_t * sa,
>>
>>       OSM_LOG(sa->p_log, OSM_LOG_DEBUG,
>>               "New PortInfoRecord: port 0x%016" PRIx64
>> -             ", lid %u, port 0x%X\n",
>> +             ", lid %u, port %u\n",
>>               cl_ntoh64(osm_physp_get_port_guid(p_physp)),
>>               cl_ntoh16(lid), osm_physp_get_port_num(p_physp));
>>
>> diff --git a/opensm/opensm/osm_sa_slvl_record.c b/opensm/opensm/osm_sa_slvl_record.c
>> index 46b66a6..886cb9b 100644
>> --- a/opensm/opensm/osm_sa_slvl_record.c
>> +++ b/opensm/opensm/osm_sa_slvl_record.c
>> @@ -100,7 +100,7 @@ __osm_sa_slvl_create(IN osm_sa_t * sa,
>>
>>       OSM_LOG(sa->p_log, OSM_LOG_DEBUG,
>>               "New SLtoVL Map for: OUT port 0x%016" PRIx64
>> -             ", lid 0x%X, port 0x%X to In Port:%u\n",
>> +             ", lid 0x%X, port %u to In Port:%u\n",
>>               cl_ntoh64(osm_physp_get_port_guid(p_physp)),
>>               cl_ntoh16(lid), osm_physp_get_port_num(p_physp), in_port_idx);
>>
>> diff --git a/opensm/opensm/osm_sa_vlarb_record.c b/opensm/opensm/osm_sa_vlarb_record.c
>> index e2aafc1..ae59b13 100644
>> --- a/opensm/opensm/osm_sa_vlarb_record.c
>> +++ b/opensm/opensm/osm_sa_vlarb_record.c
>> @@ -100,7 +100,7 @@ __osm_sa_vl_arb_create(IN osm_sa_t * sa,
>>
>>       OSM_LOG(sa->p_log, OSM_LOG_DEBUG,
>>               "New VLArbitration for: port 0x%016" PRIx64
>> -             ", lid %u, port 0x%X Block:%u\n",
>> +             ", lid %u, port %u Block:%u\n",
>>               cl_ntoh64(osm_physp_get_port_guid(p_physp)),
>>               cl_ntoh16(lid), osm_physp_get_port_num(p_physp), block);
>>
>> diff --git a/opensm/opensm/osm_slvl_map_rcv.c b/opensm/opensm/osm_slvl_map_rcv.c
>> index 000c2a6..e7f1cd2 100644
>> --- a/opensm/opensm/osm_slvl_map_rcv.c
>> +++ b/opensm/opensm/osm_slvl_map_rcv.c
>> @@ -135,7 +135,7 @@ void osm_slvl_rcv_process(IN void *context, IN void *p_data)
>>        */
>>       if (!p_physp) {
>>               OSM_LOG(sm->p_log, OSM_LOG_ERROR,
>> -                     "Got invalid port number 0x%X\n", out_port_num);
>> +                     "Got invalid port number %u\n", out_port_num);
>>               goto Exit;
>>       }
>>
>> diff --git a/opensm/opensm/osm_trap_rcv.c b/opensm/opensm/osm_trap_rcv.c
>> index 0213915..b7c2c77 100644
>> --- a/opensm/opensm/osm_trap_rcv.c
>> +++ b/opensm/opensm/osm_trap_rcv.c
>> @@ -129,7 +129,7 @@ osm_trap_rcv_aging_tracker_callback(IN uint64_t key,
>>       p_physp = get_physp_by_lid_and_num(sm, lid, port_num);
>>       if (!p_physp)
>>               OSM_LOG(sm->p_log, OSM_LOG_VERBOSE,
>> -                     "Cannot find port num:0x%X with lid:%u\n",
>> +                     "Cannot find port num:%u with lid:%u\n",
>>                       port_num, lid);
>>       /* make sure the physp is still valid */
>>       /* If the health port was false - set it to true */
>> diff --git a/opensm/opensm/osm_ucast_mgr.c b/opensm/opensm/osm_ucast_mgr.c
>> index bde0c29..be8e724 100644
>> --- a/opensm/opensm/osm_ucast_mgr.c
>> +++ b/opensm/opensm/osm_ucast_mgr.c
>> @@ -152,7 +152,7 @@ __osm_ucast_mgr_process_neighbor(IN osm_ucast_mgr_t * const p_mgr,
>>
>>       OSM_LOG(p_mgr->p_log, OSM_LOG_DEBUG,
>>               "Node 0x%" PRIx64 ", remote node 0x%" PRIx64
>> -             ", port 0x%X, remote port 0x%X\n",
>> +             ", port %u, remote port %u\n",
>>               cl_ntoh64(osm_node_get_node_guid(p_this_sw->p_node)),
>>               cl_ntoh64(osm_node_get_node_guid(p_remote_sw->p_node)),
>>               port_num, remote_port_num);
>> @@ -273,7 +273,7 @@ __osm_ucast_mgr_process_port(IN osm_ucast_mgr_t * const p_mgr,
>>               osm_physp_t *p = osm_node_get_physp_ptr(p_sw->p_node, port);
>>
>>               OSM_LOG(p_mgr->p_log, OSM_LOG_DEBUG,
>> -                     "Routing LID %u to port 0x%X"
>> +                     "Routing LID %u to port %u"
>>                       " for switch 0x%" PRIx64 "\n",
>>                       lid_ho, port, cl_ntoh64(node_guid));
>>
>> diff --git a/opensm/opensm/osm_vl_arb_rcv.c b/opensm/opensm/osm_vl_arb_rcv.c
>> index 725cc3b..674c2d6 100644
>> --- a/opensm/opensm/osm_vl_arb_rcv.c
>> +++ b/opensm/opensm/osm_vl_arb_rcv.c
>> @@ -132,7 +132,7 @@ void osm_vla_rcv_process(IN void *context, IN void *data)
>>        */
>>       if (!p_physp) {
>>               OSM_LOG(sm->p_log, OSM_LOG_ERROR,
>> -                     "Got invalid port number 0x%X\n", port_num);
>> +                     "Got invalid port number %u\n", port_num);
>>               goto Exit;
>>       }
>>
>
> _______________________________________________
> general mailing list
> general at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
>
> To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
>



More information about the general mailing list