From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hal Rosenstock Subject: Re: [PATCH] infiniband-diags: Add man pages for dump2psl and dump2slvl Date: Thu, 07 Apr 2011 10:56:21 -0400 Message-ID: <4D9DD095.3020800@dev.mellanox.co.il> References: <4D9B6DEB.7060702@dev.mellanox.co.il> <20110406004057.GB4573@obsidianresearch.com> <4D9C4784.6010103@dev.mellanox.co.il> <20110406181220.GA5924@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20110406181220.GA5924-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jason Gunthorpe Cc: Ira Weiny , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Jim Schutt List-Id: linux-rdma@vger.kernel.org On 4/6/2011 2:12 PM, Jason Gunthorpe wrote: > On Wed, Apr 06, 2011 at 06:59:16AM -0400, Hal Rosenstock wrote: >> On 4/5/2011 8:40 PM, Jason Gunthorpe wrote: >>> Since these are only used by ibdmchk it seems more appropriate to >>> bundle it in the ibutils package than in infiniband-diags? >> >> No; the idea is that the files can be prepared for ibdmchk on a machine >> which does not include ibutils and moved to another machine there for >> checking. That's why these are being made part of infiniband-diags and >> not ibutils. > > Your perl scripts don't run any infiniband-diags programs, they are > just text processing filters, so you can copy the input text files > to the machine with ibdmchk just as easily as copying the output > text files. You're right; they should go with ibutils. I'll generate the patch for that. -- 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