On 04/28/2011 05:51 PM, Hal Rosenstock wrote: > On 4/28/2011 5:09 AM, Eli Dorfman wrote: >> Fixed general info query >> >> There was no output when query succeeded > > Do you mean failed ?
No I meant succeeded. Since the IBERROR was missing the print of GeneralInfo is skipped. If it fails then the "if (general_info)" code is reached but it will print garbage. Eli > > -- Hal > >> >> Signed-off-by: Eli Dorfman <[email protected]> >> --- >> src/vendstat.c | 1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/src/vendstat.c b/src/vendstat.c >> index 92a90c8..8dbd1ee 100644 >> --- a/src/vendstat.c >> +++ b/src/vendstat.c >> @@ -375,6 +375,7 @@ int main(int argc, char **argv) >> gi = (is3_general_info_t *) & buf; >> if (do_vendor(&portid, srcport, IB_MLX_VENDOR_CLASS, IB_MAD_METHOD_GET, >> IB_MLX_IS3_GENERAL_INFO, 0, gi)) >> + IBERROR("generalinfo query"); >> >> if (general_info) { >> /* dump IS3 or IS4 general info here */ > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
