From mboxrd@z Thu Jan 1 00:00:00 1970 From: Or Gerlitz Subject: Re: [PATCH 2/5] IB/qib: Added congestion control agent implementation Date: Mon, 23 Jul 2012 19:50:03 +0300 Message-ID: <500D80BB.4090303@mellanox.com> References: <20120719130356.4706.63775.stgit@kop-dev-sles11-04.qlogic.org> <20120719130403.4706.24376.stgit@kop-dev-sles11-04.qlogic.org> <500B9320.7080803@mellanox.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <500B9320.7080803-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Vepa, Ramkrishna" Cc: Mike Marciniszyn , roland-BHEL68pLQRGGvPXPguhicg@public.gmane.org, linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org On 22/07/2012 08:44, Or Gerlitz wrote: > On 7/19/2012 4:04 PM, Mike Marciniszyn wrote: >> Added a congestion control agent in the driver that handles >> gets and sets from the congestion control manager in the >> fabric for the Performance Scale Messagin (PSM) library. > > Hi Mike, > > Nice doing. > > Can you elaborate a little on the role/usage of the sysfs entries this > patch adds? I wasn't sure to follow, basically, I thought IBTA CCA > defined thing such that one can set/get anything CC related through > MADs, isn't that? resending to Ram which is the author of the patch. > > Also, it seems that lots (if not most or all) the CC MADs related > code is general and can (should) easily be moved to be under the IB > core headers and sources, agree? We can do that as little cleanup > later, after the bigger picture is realized here. > > -- 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