On Sun, 24 Feb 2013 10:22:47 +0200 "Husam Kahalah" <hkaha...@asaltech.com> wrote:
> Thank you for your time, I have the following notes: > I plan to add support for more records other than switchInfoRecord like > smInfo, multipath, nodeInfo That would be great! > I plan also to add component filter to those records attributes in the same > way that pathInfo and MCR were implemented, this is the reason why I started > adding attributes to query_params structure > I'am not talking about RID(record identifier) attributes only, but also > about other attributes that may differ between records. > If it is available to add to query_params structure in that way let me know > , otherwise I will add just RID attributes in the suggested way. > I am not against adding fields to query_params. However, we have a standard way to specify the RID components like LID which should be followed. If you need new components to be specified for new attributes it is fine to add them to query_params. That said, any components which already have options should be overloaded; for example MultiPathRecord.SL should use the --sl option. Does this make sense? Thanks, Ira > > -----Original Message----- > From: Ira Weiny <wei...@llnl.gov> > To: Ira Weiny <wei...@llnl.gov> > Cc: "Husam Kahalah" <hkaha...@asaltech.com>, linux-r...@vger.kernel.org, > linux-kernel@vger.kernel.org > Date: Thu, 21 Feb 2013 10:33:44 -0800 > Subject: Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added > > On Thu, 21 Feb 2013 10:09:00 -0800 > Ira Weiny <wei...@llnl.gov> wrote: > > [snip] > > > > + ib_net16_t swir_lid; > > > > We don't need this field or lid option. Other records simply have an > optional parameter to specify the lid. > > > > The usage should be: > > > > SwitchRecord [lid] > > > > Sorry I meant "SwitchInfoRecord" or SWIR. > > Ira > > [snip] > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
