All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgg@nvidia.com>
To: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
Cc: <dledford@redhat.com>, <leon@kernel.org>, <linux-rdma@vger.kernel.org>
Subject: Re: [PATCH rdma-core 3/5] rdma-core/irdma: Add user/kernel shared libraries
Date: Thu, 28 Jan 2021 09:32:51 -0400	[thread overview]
Message-ID: <20210128133251.GB4247@nvidia.com> (raw)
In-Reply-To: <20210128035704.1781-4-tatyana.e.nikolova@intel.com>

On Wed, Jan 27, 2021 at 09:57:02PM -0600, Tatyana Nikolova wrote:

> +/**
> + * irdma_qp_post_wr - ring doorbell
> + * @qp: hw qp ptr
> + */
> +void irdma_qp_post_wr(struct irdma_qp_uk *qp)
> +{
> +	__u64 temp;
> +	__u32 hw_sq_tail;
> +	__u32 sw_sq_head;
> +
> +	/* valid bit is written and loads completed before reading shadow */
> +	atomic_thread_fence(memory_order_seq_cst);

atomics are only used for synchronization with CPU threads, if this is
synchronizing DMA it needs to be one of the udma operations

Jason

  reply	other threads:[~2021-01-28 13:34 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-28  3:56 [PATCH rdma-core 0/5] Add irdma user space provider for Intel Ethernet RDMA Tatyana Nikolova
2021-01-28  3:57 ` [PATCH rdma-core 1/5] rdma-core/i40iw: Remove provider i40iw Tatyana Nikolova
2021-01-28 13:31   ` Jason Gunthorpe
2021-01-28  3:57 ` [PATCH rdma-core 2/5] rdma-core/irdma: Add Makefile and ABI definitions Tatyana Nikolova
2021-01-28  3:57 ` [PATCH rdma-core 3/5] rdma-core/irdma: Add user/kernel shared libraries Tatyana Nikolova
2021-01-28 13:32   ` Jason Gunthorpe [this message]
2021-01-28  3:57 ` [PATCH rdma-core 4/5] rdma-core/irdma: Add library setup and utility definitions Tatyana Nikolova
2021-01-28 13:34   ` Jason Gunthorpe
2021-01-28  3:57 ` [PATCH rdma-core 5/5] rdma-core/irdma: Implement device supported verb APIs Tatyana Nikolova
2021-01-28 13:36   ` Jason Gunthorpe
2021-01-28 13:26 ` [PATCH rdma-core 0/5] Add irdma user space provider for Intel Ethernet RDMA Leon Romanovsky
2021-01-28 13:37 ` Jason Gunthorpe

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=20210128133251.GB4247@nvidia.com \
    --to=jgg@nvidia.com \
    --cc=dledford@redhat.com \
    --cc=leon@kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=tatyana.e.nikolova@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.