* [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported
@ 2009-10-13 18:10 Hal Rosenstock
[not found] ` <20091013181037.GA8839-Wuw85uim5zDR7s880joybQ@public.gmane.org>
0 siblings, 1 reply; 5+ messages in thread
From: Hal Rosenstock @ 2009-10-13 18:10 UTC (permalink / raw)
To: sashak-smomgflXvOZWk0Htik3J/w; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA
Per published MgtWG errata RefID 4576
Signed-off-by: Hal Rosenstock <hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
Changes since v2:
Changed ib_port_info_set_client_rereg similarly to ib_port_info_set_mcast_pkey_trap_suppress
Changes since v1:
Don't reset reserved bits packed with subnet timeout
diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h
index aa566f7..79e5f53 100644
--- a/opensm/include/iba/ib_types.h
+++ b/opensm/include/iba/ib_types.h
@@ -4431,7 +4431,7 @@ typedef struct _ib_port_info {
ib_net16_t p_key_violations;
ib_net16_t q_key_violations;
uint8_t guid_cap;
- uint8_t subnet_timeout; /* cli_rereg(1b), resrv(2b), timeout(5b) */
+ uint8_t subnet_timeout; /* cli_rereg(1b), mcast_pkey_trap_suppr(1b), resrv(1b), timeout(5b) */
uint8_t resp_time_value;
uint8_t error_threshold; /* local phy errors(4b), overrun errors(4b) */
ib_net16_t max_credit_hint;
@@ -5281,7 +5281,7 @@ ib_port_info_set_timeout(IN ib_port_info_t * const p_pi,
{
CL_ASSERT(timeout <= 0x1F);
p_pi->subnet_timeout =
- (uint8_t) ((p_pi->subnet_timeout & 0x80) | (timeout & 0x1F));
+ (uint8_t) ((p_pi->subnet_timeout & 0xE0) | (timeout & 0x1F));
}
/*
@@ -5315,8 +5315,7 @@ ib_port_info_set_client_rereg(IN ib_port_info_t * const p_pi,
{
CL_ASSERT(client_rereg <= 0x1);
p_pi->subnet_timeout =
- (uint8_t) ((p_pi->
- subnet_timeout & 0x1F) | ((client_rereg << 7) & 0x80));
+ (uint8_t) ((p_pi->subnet_timeout & 0x9F) | (client_rereg << 7));
}
/*
@@ -5335,6 +5334,42 @@ ib_port_info_set_client_rereg(IN ib_port_info_t * const p_pi,
* SEE ALSO
*********/
+/****f* IBA Base: Types/ib_port_info_set_mcast_pkey_trap_suppress
+* NAME
+* ib_port_info_set_mcast_pkey_trap_suppress
+*
+* DESCRIPTION
+* Sets the encoded multicast pkey trap suppresion enabled bit value
+* in the PortInfo attribute.
+*
+* SYNOPSIS
+*/
+static inline void OSM_API
+ib_port_info_set_mcast_pkey_trap_suppress(IN ib_port_info_t * const p_pi,
+ IN const uint8_t trap_suppress)
+{
+ CL_ASSERT(trap_suppress <= 0x1);
+ p_pi->subnet_timeout =
+ (uint8_t) ((p_pi->subnet_timeout & 0xBF) | (trap_suppress << 6));
+}
+
+/*
+* PARAMETERS
+* p_pi
+* [in] Pointer to a PortInfo attribute.
+*
+* trap_suppress
+* [in] Multicast pkey trap suppresion enabled value to set
+* (either 1 or 0).
+*
+* RETURN VALUES
+* None.
+*
+* NOTES
+*
+* SEE ALSO
+*********/
+
/****f* IBA Base: Types/ib_port_info_get_timeout
* NAME
* ib_port_info_get_timeout
@@ -5391,6 +5426,35 @@ ib_port_info_get_client_rereg(IN ib_port_info_t const *p_pi)
* SEE ALSO
*********/
+/****f* IBA Base: Types/ib_port_info_get_mcast_pkey_trap_suppress
+* NAME
+* ib_port_info_get_mcast_pkey_trap_suppress
+*
+* DESCRIPTION
+* Gets the encoded multicast pkey trap suppresion enabled bit value
+* in the PortInfo attribute.
+*
+* SYNOPSIS
+*/
+static inline uint8_t OSM_API
+ib_port_info_get_mcast_pkey_trap_suppress(IN ib_port_info_t const *p_pi)
+{
+ return ((p_pi->subnet_timeout & 0x40) >> 6);
+}
+
+/*
+* PARAMETERS
+* p_pi
+* [in] Pointer to a PortInfo attribute.
+*
+* RETURN VALUES
+* Multicast PKey trap suppression enabled value (either 1 or 0).
+*
+* NOTES
+*
+* SEE ALSO
+*********/
+
/****f* IBA Base: Types/ib_port_info_set_hoq_lifetime
* NAME
* ib_port_info_set_hoq_lifetime
diff --git a/opensm/opensm/osm_helper.c b/opensm/opensm/osm_helper.c
index 2b1e5ef..40dea10 100644
--- a/opensm/opensm/osm_helper.c
+++ b/opensm/opensm/osm_helper.c
@@ -823,6 +823,7 @@ void osm_dump_port_info(IN osm_log_t * p_log, IN ib_net64_t node_guid,
"\t\t\t\tq_key_violations........0x%X\n"
"\t\t\t\tguid_cap................0x%X\n"
"\t\t\t\tclient_reregister.......0x%X\n"
+ "\t\t\t\tmcast_pkey_trap_suppr...0x%X\n"
"\t\t\t\tsubnet_timeout..........0x%X\n"
"\t\t\t\tresp_time_value.........0x%X\n"
"\t\t\t\terror_threshold.........0x%X\n"
@@ -849,6 +850,7 @@ void osm_dump_port_info(IN osm_log_t * p_log, IN ib_net64_t node_guid,
cl_ntoh16(p_pi->p_key_violations),
cl_ntoh16(p_pi->q_key_violations), p_pi->guid_cap,
ib_port_info_get_client_rereg(p_pi),
+ ib_port_info_get_mcast_pkey_trap_suppress(p_pi),
ib_port_info_get_timeout(p_pi), p_pi->resp_time_value,
p_pi->error_threshold, cl_ntoh16(p_pi->max_credit_hint),
cl_ntoh32(p_pi->link_rt_latency));
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 5+ messages in thread[parent not found: <20091013181037.GA8839-Wuw85uim5zDR7s880joybQ@public.gmane.org>]
* Re: [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported [not found] ` <20091013181037.GA8839-Wuw85uim5zDR7s880joybQ@public.gmane.org> @ 2009-10-13 19:17 ` Sasha Khapyorsky 2009-10-13 19:20 ` Hal Rosenstock 0 siblings, 1 reply; 5+ messages in thread From: Sasha Khapyorsky @ 2009-10-13 19:17 UTC (permalink / raw) To: Hal Rosenstock; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA On 14:10 Tue 13 Oct , Hal Rosenstock wrote: > > Per published MgtWG errata RefID 4576 > > Signed-off-by: Hal Rosenstock <hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > --- > Changes since v2: > Changed ib_port_info_set_client_rereg similarly to ib_port_info_set_mcast_pkey_trap_suppress > > Changes since v1: > Don't reset reserved bits packed with subnet timeout > > diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h > index aa566f7..79e5f53 100644 > --- a/opensm/include/iba/ib_types.h > +++ b/opensm/include/iba/ib_types.h > @@ -4431,7 +4431,7 @@ typedef struct _ib_port_info { > ib_net16_t p_key_violations; > ib_net16_t q_key_violations; > uint8_t guid_cap; > - uint8_t subnet_timeout; /* cli_rereg(1b), resrv(2b), timeout(5b) */ > + uint8_t subnet_timeout; /* cli_rereg(1b), mcast_pkey_trap_suppr(1b), resrv(1b), timeout(5b) */ > uint8_t resp_time_value; > uint8_t error_threshold; /* local phy errors(4b), overrun errors(4b) */ > ib_net16_t max_credit_hint; > @@ -5281,7 +5281,7 @@ ib_port_info_set_timeout(IN ib_port_info_t * const p_pi, > { > CL_ASSERT(timeout <= 0x1F); > p_pi->subnet_timeout = > - (uint8_t) ((p_pi->subnet_timeout & 0x80) | (timeout & 0x1F)); > + (uint8_t) ((p_pi->subnet_timeout & 0xE0) | (timeout & 0x1F)); > } > > /* > @@ -5315,8 +5315,7 @@ ib_port_info_set_client_rereg(IN ib_port_info_t * const p_pi, > { > CL_ASSERT(client_rereg <= 0x1); > p_pi->subnet_timeout = > - (uint8_t) ((p_pi-> > - subnet_timeout & 0x1F) | ((client_rereg << 7) & 0x80)); > + (uint8_t) ((p_pi->subnet_timeout & 0x9F) | (client_rereg << 7)); I don't think that you wanted this - 0x9f will reset newly assigned bit. Guess it should be: + (uint8_t) (p_pi->subnet_timeout | (client_rereg << 7)); Reapplying... Sasha -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported 2009-10-13 19:17 ` Sasha Khapyorsky @ 2009-10-13 19:20 ` Hal Rosenstock [not found] ` <f0e08f230910131220g49462559u8070ebb43832e682-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 5+ messages in thread From: Hal Rosenstock @ 2009-10-13 19:20 UTC (permalink / raw) To: Sasha Khapyorsky; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA On Tue, Oct 13, 2009 at 3:17 PM, Sasha Khapyorsky <sashak-smomgflXvOZWk0Htik3J/w@public.gmane.org> wrote: > On 14:10 Tue 13 Oct , Hal Rosenstock wrote: >> >> Per published MgtWG errata RefID 4576 >> >> Signed-off-by: Hal Rosenstock <hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> >> --- >> Changes since v2: >> Changed ib_port_info_set_client_rereg similarly to ib_port_info_set_mcast_pkey_trap_suppress >> >> Changes since v1: >> Don't reset reserved bits packed with subnet timeout >> >> diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h >> index aa566f7..79e5f53 100644 >> --- a/opensm/include/iba/ib_types.h >> +++ b/opensm/include/iba/ib_types.h >> @@ -4431,7 +4431,7 @@ typedef struct _ib_port_info { >> ib_net16_t p_key_violations; >> ib_net16_t q_key_violations; >> uint8_t guid_cap; >> - uint8_t subnet_timeout; /* cli_rereg(1b), resrv(2b), timeout(5b) */ >> + uint8_t subnet_timeout; /* cli_rereg(1b), mcast_pkey_trap_suppr(1b), resrv(1b), timeout(5b) */ >> uint8_t resp_time_value; >> uint8_t error_threshold; /* local phy errors(4b), overrun errors(4b) */ >> ib_net16_t max_credit_hint; >> @@ -5281,7 +5281,7 @@ ib_port_info_set_timeout(IN ib_port_info_t * const p_pi, >> { >> CL_ASSERT(timeout <= 0x1F); >> p_pi->subnet_timeout = >> - (uint8_t) ((p_pi->subnet_timeout & 0x80) | (timeout & 0x1F)); >> + (uint8_t) ((p_pi->subnet_timeout & 0xE0) | (timeout & 0x1F)); >> } >> >> /* >> @@ -5315,8 +5315,7 @@ ib_port_info_set_client_rereg(IN ib_port_info_t * const p_pi, >> { >> CL_ASSERT(client_rereg <= 0x1); >> p_pi->subnet_timeout = >> - (uint8_t) ((p_pi-> >> - subnet_timeout & 0x1F) | ((client_rereg << 7) & 0x80)); >> + (uint8_t) ((p_pi->subnet_timeout & 0x9F) | (client_rereg << 7)); > > I don't think that you wanted this - 0x9f will reset newly assigned bit. You're right... > Guess it should be: > > + (uint8_t) (p_pi->subnet_timeout | (client_rereg << 7)); Shouldn't this be: (uint8_t) ((p_pi->subnet_timeout & 0xBF) | (client_rereg << 7)); -- Hal > > Reapplying... > > Sasha > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <f0e08f230910131220g49462559u8070ebb43832e682-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported [not found] ` <f0e08f230910131220g49462559u8070ebb43832e682-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2009-10-13 19:21 ` Hal Rosenstock [not found] ` <f0e08f230910131221o6107e55cg2a0e934f90bafa89-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 5+ messages in thread From: Hal Rosenstock @ 2009-10-13 19:21 UTC (permalink / raw) To: Sasha Khapyorsky; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA On Tue, Oct 13, 2009 at 3:20 PM, Hal Rosenstock <hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote: > On Tue, Oct 13, 2009 at 3:17 PM, Sasha Khapyorsky <sashak-smomgflXvObQFizaE/u3fw@public.gmane.orgm> wrote: >> On 14:10 Tue 13 Oct , Hal Rosenstock wrote: >>> >>> Per published MgtWG errata RefID 4576 >>> >>> Signed-off-by: Hal Rosenstock <hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> >>> --- >>> Changes since v2: >>> Changed ib_port_info_set_client_rereg similarly to ib_port_info_set_mcast_pkey_trap_suppress >>> >>> Changes since v1: >>> Don't reset reserved bits packed with subnet timeout >>> >>> diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h >>> index aa566f7..79e5f53 100644 >>> --- a/opensm/include/iba/ib_types.h >>> +++ b/opensm/include/iba/ib_types.h >>> @@ -4431,7 +4431,7 @@ typedef struct _ib_port_info { >>> ib_net16_t p_key_violations; >>> ib_net16_t q_key_violations; >>> uint8_t guid_cap; >>> - uint8_t subnet_timeout; /* cli_rereg(1b), resrv(2b), timeout(5b) */ >>> + uint8_t subnet_timeout; /* cli_rereg(1b), mcast_pkey_trap_suppr(1b), resrv(1b), timeout(5b) */ >>> uint8_t resp_time_value; >>> uint8_t error_threshold; /* local phy errors(4b), overrun errors(4b) */ >>> ib_net16_t max_credit_hint; >>> @@ -5281,7 +5281,7 @@ ib_port_info_set_timeout(IN ib_port_info_t * const p_pi, >>> { >>> CL_ASSERT(timeout <= 0x1F); >>> p_pi->subnet_timeout = >>> - (uint8_t) ((p_pi->subnet_timeout & 0x80) | (timeout & 0x1F)); >>> + (uint8_t) ((p_pi->subnet_timeout & 0xE0) | (timeout & 0x1F)); >>> } >>> >>> /* >>> @@ -5315,8 +5315,7 @@ ib_port_info_set_client_rereg(IN ib_port_info_t * const p_pi, >>> { >>> CL_ASSERT(client_rereg <= 0x1); >>> p_pi->subnet_timeout = >>> - (uint8_t) ((p_pi-> >>> - subnet_timeout & 0x1F) | ((client_rereg << 7) & 0x80)); >>> + (uint8_t) ((p_pi->subnet_timeout & 0x9F) | (client_rereg << 7)); >> >> I don't think that you wanted this - 0x9f will reset newly assigned bit. > > You're right... > >> Guess it should be: >> >> + (uint8_t) (p_pi->subnet_timeout | (client_rereg << 7)); > > Shouldn't this be: > (uint8_t) ((p_pi->subnet_timeout & 0xBF) | (client_rereg << 7)); Doh! 0x7F rather than 0xBF > > -- Hal > >> >> Reapplying... >> >> Sasha >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in >> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <f0e08f230910131221o6107e55cg2a0e934f90bafa89-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported [not found] ` <f0e08f230910131221o6107e55cg2a0e934f90bafa89-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2009-10-13 20:41 ` Sasha Khapyorsky 0 siblings, 0 replies; 5+ messages in thread From: Sasha Khapyorsky @ 2009-10-13 20:41 UTC (permalink / raw) To: Hal Rosenstock; +Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA On 15:21 Tue 13 Oct , Hal Rosenstock wrote: > > Doh! 0x7F rather than 0xBF Yes, I forgot that it is used for bit clear too. Sasha -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-10-13 20:41 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-10-13 18:10 [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported Hal Rosenstock
[not found] ` <20091013181037.GA8839-Wuw85uim5zDR7s880joybQ@public.gmane.org>
2009-10-13 19:17 ` Sasha Khapyorsky
2009-10-13 19:20 ` Hal Rosenstock
[not found] ` <f0e08f230910131220g49462559u8070ebb43832e682-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2009-10-13 19:21 ` Hal Rosenstock
[not found] ` <f0e08f230910131221o6107e55cg2a0e934f90bafa89-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2009-10-13 20:41 ` Sasha Khapyorsky
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox