From: Sasha Khapyorsky <sashak-smomgflXvOZWk0Htik3J/w@public.gmane.org>
To: "Stan C. Smith" <stan.smith-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: ofw-ZwoEplunGu1OwGhvXhtEPSCwEArCW2h5@public.gmane.org,
linux-rdma <linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Sean Hefty <sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Subject: Re: [OPENSM] match functions to prototypes in header file
Date: Thu, 1 Oct 2009 20:31:27 +0200 [thread overview]
Message-ID: <20091001183127.GX17846@me> (raw)
In-Reply-To: <3DE88517D38A4EB0B49A2E3AAF96D005-Zpru7NauK7drdx17CPfAsdBPR1lH4CV8@public.gmane.org>
On 13:44 Wed 30 Sep , Stan C. Smith wrote:
>
> From the looks of the cut-n-past from git diff output the emailer is not expanding tabs
> as envisioned, hence the attached Linux EOL patch file; grrrrrrrrrrr.
I'm not using Outlook, but guess that there are other people on the list
which are, maybe they could suggest some useful techniques.
> Signed-off-by: stan smith <stan.smith-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
>
> diff --git a/opensm/opensm/osm_helper.c b/opensm/opensm/osm_helper.c
> index 4b4e320..28edb04 100644
> --- a/opensm/opensm/osm_helper.c
> +++ b/opensm/opensm/osm_helper.c
> @@ -780,10 +780,11 @@ static void dbg_get_capabilities_str(IN char *p_buf, IN const uint32_t buf_size,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_port_info(IN osm_log_t * p_log, IN const ib_net64_t node_guid,
> +void osm_dump_port_info(IN osm_log_t * const p_log,
> + IN const ib_net64_t node_guid,
> IN const ib_net64_t port_guid,
> IN const uint8_t port_num,
> - IN const ib_port_info_t * p_pi,
> + IN const ib_port_info_t * const p_pi,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -870,8 +871,8 @@ void osm_dump_port_info(IN osm_log_t * p_log, IN const ib_net64_t node_guid,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_portinfo_record(IN osm_log_t * p_log,
> - IN const ib_portinfo_record_t * p_pir,
> +void osm_dump_portinfo_record(IN osm_log_t * const p_log,
> + IN const ib_portinfo_record_t * const p_pir,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -956,8 +957,8 @@ void osm_dump_portinfo_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_guidinfo_record(IN osm_log_t * p_log,
> - IN const ib_guidinfo_record_t * p_gir,
> +void osm_dump_guidinfo_record(IN osm_log_t * const p_log,
> + IN const ib_guidinfo_record_t * const p_gir,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -995,7 +996,8 @@ void osm_dump_guidinfo_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_node_info(IN osm_log_t * p_log, IN const ib_node_info_t * p_ni,
> +void osm_dump_node_info(IN osm_log_t * const p_log,
> + IN const ib_node_info_t * const p_ni,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1030,8 +1032,8 @@ void osm_dump_node_info(IN osm_log_t * p_log, IN const ib_node_info_t * p_ni,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_node_record(IN osm_log_t * p_log,
> - IN const ib_node_record_t * p_nr,
> +void osm_dump_node_record(IN osm_log_t * const p_log,
> + IN const ib_node_record_t * const p_nr,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1080,7 +1082,8 @@ void osm_dump_node_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_path_record(IN osm_log_t * p_log, IN const ib_path_rec_t * p_pr,
> +void osm_dump_path_record(IN osm_log_t * const p_log,
> + IN const ib_path_rec_t * const p_pr,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1106,9 +1109,9 @@ void osm_dump_path_record(IN osm_log_t * p_log, IN const ib_path_rec_t * p_pr,
> "\t\t\t\tresv2...................0x%X\n"
> "\t\t\t\tresv3...................0x%X\n",
> cl_ntoh64(p_pr->service_id),
> - inet_ntop(AF_INET6, p_pr->dgid.raw, gid_str,
> + inet_ntop(AF_INET6, (void*)p_pr->dgid.raw, gid_str,
And why is such casting(s) needed?
Also wouldn't it be simpler to remove 'const' in "type * const var"
function parameter definitions? This restricts only value of a pointer
(not structure content), and since function parameters are passed by
values such restriction is only related to a function implementation
and actually meanless in sense of API. Thoughts?
Sasha
> sizeof gid_str),
> - inet_ntop(AF_INET6, p_pr->sgid.raw, gid_str2,
> + inet_ntop(AF_INET6, (void*)p_pr->sgid.raw, gid_str2,
> sizeof gid_str2),
> cl_ntoh16(p_pr->dlid),
> cl_ntoh16(p_pr->slid),
> @@ -1129,8 +1132,8 @@ void osm_dump_path_record(IN osm_log_t * p_log, IN const ib_path_rec_t * p_pr,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_multipath_record(IN osm_log_t * p_log,
> - IN const ib_multipath_rec_t * p_mpr,
> +void osm_dump_multipath_record(IN osm_log_t * const p_log,
> + IN const ib_multipath_rec_t * const p_mpr,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1144,7 +1147,7 @@ void osm_dump_multipath_record(IN osm_log_t * p_log,
> n += sprintf(buf_line + n,
> "\t\t\t\tsgid%02d.................."
> "%s\n", i + 1,
> - inet_ntop(AF_INET6, p_gid->raw,
> + inet_ntop(AF_INET6, (void*)p_gid->raw,
> gid_str, sizeof gid_str));
> p_gid++;
> }
> @@ -1154,7 +1157,7 @@ void osm_dump_multipath_record(IN osm_log_t * p_log,
> n += sprintf(buf_line + n,
> "\t\t\t\tdgid%02d.................."
> "%s\n", i + 1,
> - inet_ntop(AF_INET6, p_gid->raw,
> + inet_ntop(AF_INET6, (void*)p_gid->raw,
> gid_str, sizeof gid_str));
> p_gid++;
> }
> @@ -1193,7 +1196,8 @@ void osm_dump_multipath_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_mc_record(IN osm_log_t * p_log, IN const ib_member_rec_t * p_mcmr,
> +void osm_dump_mc_record(IN osm_log_t * const p_log,
> + IN const ib_member_rec_t * const p_mcmr,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1213,9 +1217,9 @@ void osm_dump_mc_record(IN osm_log_t * p_log, IN const ib_member_rec_t * p_mcmr,
> "\t\t\t\tSLFlowLabelHopLimit.....0x%X\n"
> "\t\t\t\tScopeState..............0x%X\n"
> "\t\t\t\tProxyJoin...............0x%X\n",
> - inet_ntop(AF_INET6, p_mcmr->mgid.raw, gid_str,
> + inet_ntop(AF_INET6, (void*)p_mcmr->mgid.raw, gid_str,
> sizeof gid_str),
> - inet_ntop(AF_INET6, p_mcmr->port_gid.raw, gid_str2,
> + inet_ntop(AF_INET6, (void*)p_mcmr->port_gid.raw, gid_str2,
> sizeof gid_str2),
> cl_ntoh32(p_mcmr->qkey),
> cl_ntoh16(p_mcmr->mlid),
> @@ -1231,8 +1235,8 @@ void osm_dump_mc_record(IN osm_log_t * p_log, IN const ib_member_rec_t * p_mcmr,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_service_record(IN osm_log_t * p_log,
> - IN const ib_service_record_t * p_sr,
> +void osm_dump_service_record(IN osm_log_t * const p_log,
> + IN const ib_service_record_t * const p_sr,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1299,7 +1303,7 @@ void osm_dump_service_record(IN osm_log_t * p_log,
> "\t\t\t\tServiceData64.1.........0x%016" PRIx64 "\n"
> "\t\t\t\tServiceData64.2.........0x%016" PRIx64 "\n",
> cl_ntoh64(p_sr->service_id),
> - inet_ntop(AF_INET6, p_sr->service_gid.raw, gid_str,
> + inet_ntop(AF_INET6, (void*)p_sr->service_gid.raw, gid_str,
> sizeof gid_str),
> cl_ntoh16(p_sr->service_pkey),
> cl_ntoh32(p_sr->service_lease),
> @@ -1332,8 +1336,8 @@ void osm_dump_service_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_inform_info(IN osm_log_t * p_log,
> - IN const ib_inform_info_t * p_ii,
> +void osm_dump_inform_info(IN osm_log_t * const p_log,
> + IN const ib_inform_info_t * const p_ii,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1356,7 +1360,7 @@ void osm_dump_inform_info(IN osm_log_t * p_log,
> "\t\t\t\tqpn.....................0x%06X\n"
> "\t\t\t\tresp_time_val...........0x%X\n"
> "\t\t\t\tnode_type...............0x%06X\n" "",
> - inet_ntop(AF_INET6, p_ii->gid.raw, gid_str,
> + inet_ntop(AF_INET6, (void*)p_ii->gid.raw, gid_str,
> sizeof gid_str),
> cl_ntoh16(p_ii->lid_range_begin),
> cl_ntoh16(p_ii->lid_range_end),
> @@ -1378,7 +1382,7 @@ void osm_dump_inform_info(IN osm_log_t * p_log,
> "\t\t\t\tqpn.....................0x%06X\n"
> "\t\t\t\tresp_time_val...........0x%X\n"
> "\t\t\t\tvendor_id...............0x%06X\n" "",
> - inet_ntop(AF_INET6, p_ii->gid.raw, gid_str,
> + inet_ntop(AF_INET6, (void*)p_ii->gid.raw, gid_str,
> sizeof gid_str),
> cl_ntoh16(p_ii->lid_range_begin),
> cl_ntoh16(p_ii->lid_range_end),
> @@ -1393,8 +1397,8 @@ void osm_dump_inform_info(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_inform_info_record(IN osm_log_t * p_log,
> - IN const ib_inform_info_record_t * p_iir,
> +void osm_dump_inform_info_record(IN osm_log_t * const p_log,
> + IN const ib_inform_info_record_t * const p_iir,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1423,10 +1427,10 @@ void osm_dump_inform_info_record(IN osm_log_t * p_log,
> "\t\t\t\tqpn.....................0x%06X\n"
> "\t\t\t\tresp_time_val...........0x%X\n"
> "\t\t\t\tnode_type...............0x%06X\n" "",
> - inet_ntop(AF_INET6, p_iir->subscriber_gid.raw,
> + inet_ntop(AF_INET6, (void*)p_iir->subscriber_gid.raw,
> gid_str, sizeof gid_str),
> cl_ntoh16(p_iir->subscriber_enum),
> - inet_ntop(AF_INET6, p_iir->inform_info.gid.raw,
> + inet_ntop(AF_INET6, (void*)p_iir->inform_info.gid.raw,
> gid_str2, sizeof gid_str2),
> cl_ntoh16(p_iir->inform_info.lid_range_begin),
> cl_ntoh16(p_iir->inform_info.lid_range_end),
> @@ -1455,10 +1459,10 @@ void osm_dump_inform_info_record(IN osm_log_t * p_log,
> "\t\t\t\tqpn.....................0x%06X\n"
> "\t\t\t\tresp_time_val...........0x%X\n"
> "\t\t\t\tvendor_id...............0x%06X\n" "",
> - inet_ntop(AF_INET6, p_iir->subscriber_gid.raw,
> + inet_ntop(AF_INET6, (void*)p_iir->subscriber_gid.raw,
> gid_str, sizeof gid_str),
> cl_ntoh16(p_iir->subscriber_enum),
> - inet_ntop(AF_INET6, p_iir->inform_info.gid.raw,
> + inet_ntop(AF_INET6, (void*)p_iir->inform_info.gid.raw,
> gid_str2, sizeof gid_str2),
> cl_ntoh16(p_iir->inform_info.lid_range_begin),
> cl_ntoh16(p_iir->inform_info.lid_range_end),
> @@ -1476,8 +1480,8 @@ void osm_dump_inform_info_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_link_record(IN osm_log_t * p_log,
> - IN const ib_link_record_t * p_lr,
> +void osm_dump_link_record(IN osm_log_t * const p_log,
> + IN const ib_link_record_t * const p_lr,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1495,8 +1499,8 @@ void osm_dump_link_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_switch_info(IN osm_log_t * p_log,
> - IN const ib_switch_info_t * p_si,
> +void osm_dump_switch_info(IN osm_log_t * const p_log,
> + IN const ib_switch_info_t * const p_si,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1530,8 +1534,8 @@ void osm_dump_switch_info(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_switch_info_record(IN osm_log_t * p_log,
> - IN const ib_switch_info_record_t * p_sir,
> +void osm_dump_switch_info_record(IN osm_log_t * const p_log,
> + IN const ib_switch_info_record_t * const p_sir,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1568,9 +1572,9 @@ void osm_dump_switch_info_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_pkey_block(IN osm_log_t * p_log, IN uint64_t port_guid,
> +void osm_dump_pkey_block(IN osm_log_t * const p_log, IN uint64_t port_guid,
> IN uint16_t block_num, IN uint8_t port_num,
> - IN const ib_pkey_table_t * p_pkey_tbl,
> + IN const ib_pkey_table_t * const p_pkey_tbl,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1592,9 +1596,9 @@ void osm_dump_pkey_block(IN osm_log_t * p_log, IN uint64_t port_guid,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_slvl_map_table(IN osm_log_t * p_log, IN uint64_t port_guid,
> +void osm_dump_slvl_map_table(IN osm_log_t * const p_log, IN uint64_t port_guid,
> IN uint8_t in_port_num, IN uint8_t out_port_num,
> - IN const ib_slvl_table_t * p_slvl_tbl,
> + IN const ib_slvl_table_t * const p_slvl_tbl,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1619,9 +1623,9 @@ void osm_dump_slvl_map_table(IN osm_log_t * p_log, IN uint64_t port_guid,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_vl_arb_table(IN osm_log_t * p_log, IN uint64_t port_guid,
> +void osm_dump_vl_arb_table(IN osm_log_t * const p_log, IN uint64_t port_guid,
> IN uint8_t block_num, IN uint8_t port_num,
> - IN const ib_vl_arb_table_t * p_vla_tbl,
> + IN const ib_vl_arb_table_t * const p_vla_tbl,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1645,7 +1649,8 @@ void osm_dump_vl_arb_table(IN osm_log_t * p_log, IN uint64_t port_guid,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_sm_info(IN osm_log_t * p_log, IN const ib_sm_info_t * p_smi,
> +void osm_dump_sm_info(IN osm_log_t * const p_log,
> + IN const ib_sm_info_t * const p_smi,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1666,8 +1671,8 @@ void osm_dump_sm_info(IN osm_log_t * p_log, IN const ib_sm_info_t * p_smi,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_sm_info_record(IN osm_log_t * p_log,
> - IN const ib_sminfo_record_t * p_smir,
> +void osm_dump_sm_info_record(IN osm_log_t * const p_log,
> + IN const ib_sminfo_record_t * const p_smir,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1694,7 +1699,7 @@ void osm_dump_sm_info_record(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_notice(IN osm_log_t * p_log,
> +void osm_dump_notice(IN osm_log_t * const p_log,
> IN const ib_mad_notice_attr_t * p_ntci,
> IN const osm_log_level_t log_level)
> {
> @@ -1716,7 +1721,7 @@ void osm_dump_notice(IN osm_log_t * p_log,
> case 67:
> sprintf(buff,
> "\t\t\t\tsrc_gid..................%s\n",
> - inet_ntop(AF_INET6, p_ntci->data_details.
> + inet_ntop(AF_INET6, (void*)p_ntci->data_details.
> ntc_64_67.gid.raw, gid_str,
> sizeof gid_str));
> break;
> @@ -1824,10 +1829,10 @@ void osm_dump_notice(IN osm_log_t * p_log,
> qp1) & 0xffffff,
> cl_ntoh32(p_ntci->data_details.ntc_257_258.
> qp2) & 0xffffff,
> - inet_ntop(AF_INET6, p_ntci->data_details.
> + inet_ntop(AF_INET6, (void*)p_ntci->data_details.
> ntc_257_258.gid1.raw, gid_str,
> sizeof gid_str),
> - inet_ntop(AF_INET6, p_ntci->data_details.
> + inet_ntop(AF_INET6, (void*)p_ntci->data_details.
> ntc_257_258.gid2.raw, gid_str2,
> sizeof gid_str2));
> break;
> @@ -1854,10 +1859,10 @@ void osm_dump_notice(IN osm_log_t * p_log,
> cl_ntoh32(p_ntci->data_details.ntc_259.
> sl_qp1) & 0xffffff,
> cl_ntoh32(p_ntci->data_details.ntc_259.qp2),
> - inet_ntop(AF_INET6, p_ntci->data_details.
> + inet_ntop(AF_INET6, (void*)p_ntci->data_details.
> ntc_259.gid1.raw, gid_str,
> sizeof gid_str),
> - inet_ntop(AF_INET6, p_ntci->data_details.
> + inet_ntop(AF_INET6, (void*)p_ntci->data_details.
> ntc_259.gid2.raw, gid_str2,
> sizeof gid_str2),
> cl_ntoh16(p_ntci->data_details.ntc_259.sw_lid),
> @@ -1889,7 +1894,8 @@ void osm_dump_notice(IN osm_log_t * p_log,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_dr_smp(IN osm_log_t * p_log, IN const ib_smp_t * p_smp,
> +void osm_dump_dr_smp(IN osm_log_t * const p_log,
> + IN const ib_smp_t * const p_smp,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -1996,7 +2002,8 @@ void osm_dump_dr_smp(IN osm_log_t * p_log, IN const ib_smp_t * p_smp,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_sa_mad(IN osm_log_t * p_log, IN const ib_sa_mad_t * p_mad,
> +void osm_dump_sa_mad(IN osm_log_t * const p_log,
> + IN const ib_sa_mad_t * const p_mad,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -2059,7 +2066,8 @@ void osm_dump_sa_mad(IN osm_log_t * p_log, IN const ib_sa_mad_t * p_mad,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_dr_path(IN osm_log_t * p_log, IN const osm_dr_path_t * p_path,
> +void osm_dump_dr_path(IN osm_log_t * const p_log,
> + IN const osm_dr_path_t * const p_path,
> IN const osm_log_level_t log_level)
> {
> if (osm_log_is_active(p_log, log_level)) {
> @@ -2077,7 +2085,8 @@ void osm_dump_dr_path(IN osm_log_t * p_log, IN const osm_dr_path_t * p_path,
>
> /**********************************************************************
> **********************************************************************/
> -void osm_dump_smp_dr_path(IN osm_log_t * p_log, IN const ib_smp_t * p_smp,
> +void osm_dump_smp_dr_path(IN osm_log_t * const p_log,
> + IN const ib_smp_t * const p_smp,
> IN const osm_log_level_t log_level)
> {
>
--
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
next parent reply other threads:[~2009-10-01 18:31 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <3DE88517D38A4EB0B49A2E3AAF96D005@amr.corp.intel.com>
[not found] ` <3DE88517D38A4EB0B49A2E3AAF96D005-Zpru7NauK7drdx17CPfAsdBPR1lH4CV8@public.gmane.org>
2009-10-01 18:31 ` Sasha Khapyorsky [this message]
2009-10-01 18:33 ` [OPENSM] match functions to prototypes in header file Bart Van Assche
2009-10-01 19:06 ` Sean Hefty
2009-10-01 18:51 ` Jason Gunthorpe
[not found] ` <20091001185148.GJ19540-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2009-10-01 19:52 ` Sean Hefty
[not found] ` <4D838DBAE5F245EEA72D4466987294E1-Zpru7NauK7drdx17CPfAsdBPR1lH4CV8@public.gmane.org>
2009-10-01 20:13 ` Jason Gunthorpe
2009-10-01 19:30 ` Smith, Stan
[not found] ` <3F6F638B8D880340AB536D29CD4C1E1912C86E88D1-osO9UTpF0USkrb+BlOpmy7fspsVTdybXVpNB7YpNyf8@public.gmane.org>
2009-10-01 20:22 ` [PATCH] opensm/osm_helper: remove meanless 'const' keywords Sasha Khapyorsky
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20091001183127.GX17846@me \
--to=sashak-smomgflxvozwk0htik3j/w@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=ofw-ZwoEplunGu1OwGhvXhtEPSCwEArCW2h5@public.gmane.org \
--cc=sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=stan.smith-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.