public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
From: Doug Ledford <doug.ledford@hpe.com>
To: Michael Margolin <mrgolin@amazon.com>, <jgg@nvidia.com>,
	<leon@kernel.org>, <linux-rdma@vger.kernel.org>
Cc: <sleybo@amazon.com>, <matua@amazon.com>, <gal.pressman@linux.dev>
Subject: Re: [PATCH for-next 0/4] Introduce Completion Counters
Date: Tue, 28 Apr 2026 16:31:51 -0600	[thread overview]
Message-ID: <e41b13aa-0cdb-4b07-97d0-cd6a6a0cbf07@hpe.com> (raw)
In-Reply-To: <20260407115424.13359-1-mrgolin@amazon.com>


[-- Attachment #1.1: Type: text/plain, Size: 2973 bytes --]

On 4/7/26 6:54 AM, Michael Margolin wrote:
> Add core infrastructure for Completion Counters, a light-weight
> alternative to polling CQ for tracking operation completions. The
> related rdma-core interface proposal is linked in [1].
> 
> Define the UVERBS_OBJECT_COMP_CNTR ioctl object with create, destroy,
> set, inc and read methods for both success and error counters. Add a
> QP attach method on the QP object to associate a completion counter
> with a queue pair.
> 
> Completion Counters can be backed by user-provided VA or dmabuf or by
> internal device/driver memory. Common command infrastructure allows any
> of the implementations to support the various device capabilities.
> 
> Add EFA Completion Counters support as first implementer.
> 
> [1] https://github.com/linux-rdma/rdma-core/pull/1701
> 
> Michael Margolin (4):
>    RDMA/core: Add Completion Counters support
>    RDMA/core: Add Completion Counters to resource tracking
>    RDMA/efa: Update device interface
>    RDMA/efa: Add Completion Counters support
> 
>   drivers/infiniband/core/Makefile              |   1 +
>   drivers/infiniband/core/device.c              |  10 +
>   drivers/infiniband/core/nldev.c               |   1 +
>   drivers/infiniband/core/rdma_core.h           |   1 +
>   drivers/infiniband/core/restrack.c            |   2 +
>   drivers/infiniband/core/uverbs_cmd.c          |   1 +
>   .../core/uverbs_std_types_comp_cntr.c         | 379 ++++++++++++++++++
>   drivers/infiniband/core/uverbs_std_types_qp.c |  45 ++-
>   drivers/infiniband/core/uverbs_uapi.c         |   1 +
>   drivers/infiniband/hw/efa/efa.h               |  15 +
>   .../infiniband/hw/efa/efa_admin_cmds_defs.h   | 185 ++++++++-
>   drivers/infiniband/hw/efa/efa_com_cmd.c       | 106 +++++
>   drivers/infiniband/hw/efa/efa_com_cmd.h       |  36 ++
>   drivers/infiniband/hw/efa/efa_io_defs.h       |  62 ++-
>   drivers/infiniband/hw/efa/efa_main.c          |   8 +
>   drivers/infiniband/hw/efa/efa_verbs.c         | 177 ++++++++
>   include/rdma/ib_verbs.h                       |  27 ++
>   include/rdma/restrack.h                       |   4 +
>   include/uapi/rdma/efa-abi.h                   |   1 +
>   include/uapi/rdma/ib_user_ioctl_cmds.h        |  65 +++
>   include/uapi/rdma/ib_user_ioctl_verbs.h       |   9 +
>   include/uapi/rdma/ib_user_verbs.h             |   2 +-
>   22 files changed, 1131 insertions(+), 7 deletions(-)
>   create mode 100644 drivers/infiniband/core/uverbs_std_types_comp_cntr.c
> 

Our hardware does have completion counters.  I haven't dug into them in 
detail in the past, but if this isn't already merged, I can dig in at 
least well enough to make sure the API will be usable for us too. 
Probably next week as I'm at the UEC Face to Face this week.

-- 
Doug Ledford <doug.ledford@hpe.com>
     GPG KeyID: B826A3330E572FDD
     Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]

      parent reply	other threads:[~2026-04-28 22:47 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-07 11:54 [PATCH for-next 0/4] Introduce Completion Counters Michael Margolin
2026-04-07 11:54 ` [PATCH for-next 1/4] RDMA/core: Add Completion Counters support Michael Margolin
2026-04-07 14:17   ` Jason Gunthorpe
2026-04-09 16:00     ` Michael Margolin
2026-04-09 16:13       ` Jason Gunthorpe
2026-04-09 17:29         ` Sean Hefty
2026-04-09 18:55           ` Jason Gunthorpe
2026-04-09 19:15             ` Sean Hefty
2026-04-09 19:44               ` Jason Gunthorpe
2026-04-09 22:23                 ` Sean Hefty
2026-04-13 16:14                   ` Michael Margolin
2026-04-09 18:36         ` Michael Margolin
2026-04-07 11:54 ` [PATCH for-next 2/4] RDMA/core: Add Completion Counters to resource tracking Michael Margolin
2026-04-07 11:54 ` [PATCH for-next 3/4] RDMA/efa: Update device interface Michael Margolin
2026-04-07 11:54 ` [PATCH for-next 4/4] RDMA/efa: Add Completion Counters support Michael Margolin
2026-04-17 22:41   ` kernel test robot
2026-04-28 22:31 ` Doug Ledford [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e41b13aa-0cdb-4b07-97d0-cd6a6a0cbf07@hpe.com \
    --to=doug.ledford@hpe.com \
    --cc=gal.pressman@linux.dev \
    --cc=jgg@nvidia.com \
    --cc=leon@kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=matua@amazon.com \
    --cc=mrgolin@amazon.com \
    --cc=sleybo@amazon.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox