From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH rdma-next v3 00/14] Verbs flow counters support Date: Fri, 1 Jun 2018 15:11:49 -0600 Message-ID: <20180601211149.GA24028@ziepe.ca> References: <20180531134341.18441-1-leon@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Doug Ledford , Leon Romanovsky , RDMA mailing list , Boris Pismenny , Matan Barak , "Michael J . Ruhl" , Or Gerlitz , Raed Salem , Yishai Hadas , Saeed Mahameed , linux-netdev To: Leon Romanovsky Return-path: Received: from mail-pf0-f193.google.com ([209.85.192.193]:39887 "EHLO mail-pf0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750732AbeFAVLv (ORCPT ); Fri, 1 Jun 2018 17:11:51 -0400 Received: by mail-pf0-f193.google.com with SMTP id r11-v6so5496068pfl.6 for ; Fri, 01 Jun 2018 14:11:51 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20180531134341.18441-1-leon@kernel.org> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, May 31, 2018 at 04:43:27PM +0300, Leon Romanovsky wrote: > From: Leon Romanovsky > > Changelog: > v2->v3: > * Change function mlx5_fc_query signature to hide the details of > internal core driver struct mlx5_fc > * Add commen to data[] field at struct mlx5_ib_flow_counters_data (mlx5-abi.h) > * Use array of struct mlx5_ib_flow_counters_desc to clarify the output > v1->v2: > * Removed conversion from struct mlx5_fc* to void* > * Fixed one place with double space in it > * Balanced release of hardware handler in case of counters allocation failure > * Added Tested-by > * Minimize time spent holding mutex lock > * Fixed deadlock caused by nested lock in error path > * Protect from handler pointer derefence in the error paths Okay, Acked-by: Jason Gunthorpe I've revised some of the commit messages, fixed the two bad check-patch warnings, and fixed the patch ordering.. https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git/log/?h=wip/jgg-counters Please send a PR with the mlx-core bits and above commits. Thanks, Jason