On Sat, 30 Apr 2011 23:28:57 -0700 Eli Dorfman <[email protected]> wrote:
> 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. Thanks applied, Ira > > 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 */ > > > -- Ira Weiny Math Programmer/Computer Scientist Lawrence Livermore National Lab 925-423-8008 [email protected] -- 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
