[ofa-general] Re: [PATCH] opensm: don't zero base LID when invalid value is received
Sasha Khapyorsky
sashak at voltaire.com
Thu Dec 13 14:37:04 PST 2007
On 13:37 Thu 13 Dec , Hal Rosenstock wrote:
> On Thu, 2007-12-13 at 01:25 +0000, Sasha Khapyorsky wrote:
> > This addresses bug 246 (https://bugs.openfabrics.org/show_bug.cgi?id=246):
> > zero lid received from opensm in set port_info smp.
> >
> > When invalid value of LID (it was 0xffff in this case) is received,
> > OpenSM clears it to zero now. Instead this patch will try to recover
> > using current LID value stored in osm_physp_t.port_info.
>
> This looks like a step in the right direction as that lid could be
> valid. It might also be 0 or 0xffff, right ?
Do you mean stored at OpenSM in port's port_info? Assuming so it can be
0 when not initialized.
Sasha
> In that case, it ends up
> being the same until the SMA is updated to be 1.2.1 conformant.
>
> -- Hal
>
> > Signed-off-by: Sasha Khapyorsky <sashak at voltaire.com>
> > ---
> > opensm/include/opensm/osm_port.h | 39 -----------------------------------
> > opensm/opensm/osm_port_info_rcv.c | 41 ++++++++++++++++++++----------------
> > 2 files changed, 23 insertions(+), 57 deletions(-)
> >
> > diff --git a/opensm/include/opensm/osm_port.h b/opensm/include/opensm/osm_port.h
> > index fcb0a16..bba4e44 100644
> > --- a/opensm/include/opensm/osm_port.h
> > +++ b/opensm/include/opensm/osm_port.h
> > @@ -459,45 +459,6 @@ osm_physp_set_port_info(IN osm_physp_t * const p_physp,
> > * Port, Physical Port
> > *********/
> >
> > -/****f* OpenSM: Physical Port/osm_physp_trim_base_lid_to_valid_range
> > -* NAME
> > -* osm_physp_trim_base_lid_to_valid_range
> > -*
> > -* DESCRIPTION
> > -* Validates the base LID in the Physical Port object
> > -* and resets it if the base LID is invalid.
> > -*
> > -* SYNOPSIS
> > -*/
> > -static inline ib_net16_t
> > -osm_physp_trim_base_lid_to_valid_range(IN osm_physp_t * const p_physp)
> > -{
> > - ib_net16_t orig_lid = 0;
> > -
> > - CL_ASSERT(osm_physp_is_valid(p_physp));
> > - if ((cl_ntoh16(p_physp->port_info.base_lid) > IB_LID_UCAST_END_HO) ||
> > - (cl_ntoh16(p_physp->port_info.base_lid) < IB_LID_UCAST_START_HO)) {
> > - orig_lid = p_physp->port_info.base_lid;
> > - p_physp->port_info.base_lid = 0;
> > - }
> > - return orig_lid;
> > -}
> > -
> > -/*
> > -* PARAMETERS
> > -* p_physp
> > -* [in] Pointer to an osm_physp_t object.
> > -*
> > -* RETURN VALUES
> > -* Returns 0 if the base LID in the Physical port object is valid.
> > -* Returns original invalid LID otherwise.
> > -*
> > -* NOTES
> > -*
> > -* SEE ALSO
> > -* Port, Physical Port
> > -*********/
> > -
> > /****f* OpenSM: Physical Port/osm_physp_set_pkey_tbl
> > * NAME
> > * osm_physp_set_pkey_tbl
> > diff --git a/opensm/opensm/osm_port_info_rcv.c b/opensm/opensm/osm_port_info_rcv.c
> > index 9ea8738..ea0cb21 100644
> > --- a/opensm/opensm/osm_port_info_rcv.c
> > +++ b/opensm/opensm/osm_port_info_rcv.c
> > @@ -98,6 +98,22 @@ __osm_pi_rcv_set_sm(IN const osm_pi_rcv_t * const p_rcv,
> >
> > /**********************************************************************
> > **********************************************************************/
> > +static void pi_rcv_check_and_fix_lid(osm_log_t *log, ib_port_info_t * const pi,
> > + osm_physp_t * p)
> > +{
> > + if ((cl_ntoh16(pi->base_lid) > IB_LID_UCAST_END_HO) ||
> > + (cl_ntoh16(pi->base_lid) < IB_LID_UCAST_START_HO)) {
> > + osm_log(log, OSM_LOG_ERROR,
> > + "pi_rcv_check_and_fix_lid: ERR 0F04: "
> > + "Got invalid base LID 0x%x from the network. "
> > + "Corrected to 0x%x.\n", cl_ntoh16(pi->base_lid),
> > + cl_ntoh16(p->port_info.base_lid));
> > + pi->base_lid = p->port_info.base_lid;
> > + }
> > +}
> > +
> > +/**********************************************************************
> > + **********************************************************************/
> > static void
> > __osm_pi_rcv_process_endport(IN const osm_pi_rcv_t * const p_rcv,
> > IN osm_physp_t * const p_physp,
> > @@ -204,13 +220,12 @@ static void
> > __osm_pi_rcv_process_switch_port(IN const osm_pi_rcv_t * const p_rcv,
> > IN osm_node_t * const p_node,
> > IN osm_physp_t * const p_physp,
> > - IN const ib_port_info_t * const p_pi)
> > + IN ib_port_info_t * const p_pi)
> > {
> > ib_api_status_t status = IB_SUCCESS;
> > osm_madw_context_t context;
> > osm_physp_t *p_remote_physp;
> > osm_node_t *p_remote_node;
> > - ib_net16_t orig_lid;
> > uint8_t port_num;
> > uint8_t remote_port_num;
> > osm_dr_path_t path;
> > @@ -316,19 +331,15 @@ __osm_pi_rcv_process_switch_port(IN const osm_pi_rcv_t * const p_rcv,
> > if (ib_port_info_get_port_state(p_pi) > IB_LINK_INIT && p_node->sw)
> > p_node->sw->need_update = 0;
> >
> > + if (port_num == 0)
> > + pi_rcv_check_and_fix_lid(p_rcv->p_log, p_pi, p_physp);
> > +
> > /*
> > Update the PortInfo attribute.
> > */
> > osm_physp_set_port_info(p_physp, p_pi);
> >
> > if (port_num == 0) {
> > - /* This is switch management port 0 */
> > - if ((orig_lid =
> > - osm_physp_trim_base_lid_to_valid_range(p_physp)))
> > - osm_log(p_rcv->p_log, OSM_LOG_ERROR,
> > - "__osm_pi_rcv_process_switch_port: ERR 0F04: "
> > - "Invalid base LID 0x%x corrected\n",
> > - cl_ntoh16(orig_lid));
> > /* Determine if base switch port 0 */
> > if (p_node->sw &&
> > !ib_switch_info_is_enhanced_port0(&p_node->sw->switch_info))
> > @@ -346,21 +357,15 @@ static void
> > __osm_pi_rcv_process_ca_or_router_port(IN const osm_pi_rcv_t * const p_rcv,
> > IN osm_node_t * const p_node,
> > IN osm_physp_t * const p_physp,
> > - IN const ib_port_info_t * const p_pi)
> > + IN ib_port_info_t * const p_pi)
> > {
> > - ib_net16_t orig_lid;
> > -
> > OSM_LOG_ENTER(p_rcv->p_log, __osm_pi_rcv_process_ca_or_router_port);
> >
> > UNUSED_PARAM(p_node);
> >
> > - osm_physp_set_port_info(p_physp, p_pi);
> > + pi_rcv_check_and_fix_lid(p_rcv->p_log, p_pi, p_physp);
> >
> > - if ((orig_lid = osm_physp_trim_base_lid_to_valid_range(p_physp)))
> > - osm_log(p_rcv->p_log, OSM_LOG_ERROR,
> > - "__osm_pi_rcv_process_ca_or_router_port: ERR 0F08: "
> > - "Invalid base LID 0x%x corrected\n",
> > - cl_ntoh16(orig_lid));
> > + osm_physp_set_port_info(p_physp, p_pi);
> >
> > __osm_pi_rcv_process_endport(p_rcv, p_physp, p_pi);
> >
More information about the general
mailing list