From: Bryan Tan <bryantan-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
To: Jason Gunthorpe <jgg-uk2M96/98Pc@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH for-rc 6/6] RDMA/vmw_pvrdma: Add UAR SRQ macros in ABI header file
Date: Mon, 11 Dec 2017 10:35:55 -0800 [thread overview]
Message-ID: <20171211183554.GB20684@bryantan-devbox.prom.eng.vmware.com.prom.eng.vmware.com> (raw)
In-Reply-To: <20171208232809.GA23239-uk2M96/98Pc@public.gmane.org>
On Fri, Dec 08, 2017 at 04:28:09PM -0700, Jason Gunthorpe wrote:
> On Fri, Dec 08, 2017 at 11:03:26AM -0800, Bryan Tan wrote:
> > Add the two UAR SRQ macros that are required by the userlevel
> > library.
> >
> > Fixes: 8b10ba783c9d ("RDMA/vmw_pvrdma: Add shared receive queue support")
> > Reviewed-by: Adit Ranadive <aditr-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
> > Reviewed-by: Aditya Sarwade <asarwade-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
> > Reviewed-by: Jorgen Hansen <jhansen-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
> > Signed-off-by: Bryan Tan <bryantan-pghWNbHTmq7QT0dZR+AlfA@public.gmane.org>
> > include/uapi/rdma/vmw_pvrdma-abi.h | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/include/uapi/rdma/vmw_pvrdma-abi.h b/include/uapi/rdma/vmw_pvrdma-abi.h
> > index aaa352f..4007cac 100644
> > +++ b/include/uapi/rdma/vmw_pvrdma-abi.h
> > @@ -58,6 +58,8 @@
> > #define PVRDMA_UAR_CQ_ARM_SOL BIT(29) /* Arm solicited bit. */
> > #define PVRDMA_UAR_CQ_ARM BIT(30) /* Arm bit. */
> > #define PVRDMA_UAR_CQ_POLL BIT(31) /* Poll bit. */
> > +#define PVRDMA_UAR_SRQ_OFFSET 8 /* SRQ doorbell. */
> > +#define PVRDMA_UAR_SRQ_RECV BIT(30) /* Recv bit. */
>
> I didn't think BIT() was allowed in uapi headers? Where does it get
> defined in user space?
>
> Jason
Looks like we had BIT() defined in userspace. Seems like the rest of
the headers just use a bit shift? If that's fine I can send out a V2
with the BIT()s removed.
Bryan
--
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
next prev parent reply other threads:[~2017-12-11 18:35 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-08 18:58 [PATCH for-rc 0/6] vmw_pvrdma fixes for 4.15 Bryan Tan
[not found] ` <20171208185818.GA28514-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-08 19:00 ` [PATCH for-rc 1/6] RDMA/vmw_pvrdma: Clarify QP is_kernel logic Bryan Tan
[not found] ` <20171208190010.GA31023-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-13 9:02 ` Yuval Shaia
2017-12-13 19:08 ` Bryan Tan
[not found] ` <20171213190825.GH20684-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-13 19:20 ` Yuval Shaia
2017-12-13 21:22 ` Bryan Tan
2017-12-08 19:01 ` [PATCH for-rc 2/6] RDMA/vmw_pvrdma: Call ib_umem_release on destroy QP path Bryan Tan
[not found] ` <20171208190106.GA32066-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-13 8:53 ` Yuval Shaia
2017-12-13 16:17 ` Jason Gunthorpe
[not found] ` <20171213161755.GB16920-uk2M96/98Pc@public.gmane.org>
2017-12-13 18:40 ` Yuval Shaia
2017-12-13 18:55 ` Bryan Tan
[not found] ` <20171213185517.GG20684-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-13 19:04 ` Yuval Shaia
2017-12-13 19:16 ` Yuval Shaia
2017-12-13 18:56 ` Jason Gunthorpe
2017-12-08 19:01 ` [PATCH for-rc 3/6] RDMA/vmw_pvrdma: Use more specific sizeof in kcalloc Bryan Tan
2017-12-08 19:02 ` [PATCH for-rc 4/6] RDMA/vmw_pvrdma: Use refcount_dec_and_test to avoid warning Bryan Tan
[not found] ` <20171208190218.GA744-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-08 23:30 ` Jason Gunthorpe
[not found] ` <20171208233049.GB23239-uk2M96/98Pc@public.gmane.org>
2017-12-11 18:33 ` Bryan Tan
[not found] ` <20171211183314.GA20684-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-11 20:14 ` Jason Gunthorpe
[not found] ` <20171211201421.GD27709-uk2M96/98Pc@public.gmane.org>
2017-12-11 21:59 ` Bryan Tan
[not found] ` <20171211215936.GC20684-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-11 22:14 ` Jason Gunthorpe
[not found] ` <20171211221425.GA7551-uk2M96/98Pc@public.gmane.org>
2017-12-12 17:13 ` Bryan Tan
[not found] ` <20171212171300.GD20684-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-12 18:55 ` Jason Gunthorpe
[not found] ` <20171212185530.GB16603-uk2M96/98Pc@public.gmane.org>
2017-12-13 0:08 ` Bryan Tan
[not found] ` <20171213000815.GE20684-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-13 2:20 ` Jason Gunthorpe
[not found] ` <20171213022031.GE16603-uk2M96/98Pc@public.gmane.org>
2017-12-13 9:07 ` Bryan Tan
[not found] ` <20171213090702.GF20684-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-13 16:12 ` Jason Gunthorpe
2017-12-08 19:02 ` [PATCH for-rc 5/6] RDMA/vmw_pvrdma: Use refcount_t instead of atomic_t Bryan Tan
2017-12-08 19:03 ` [PATCH for-rc 6/6] RDMA/vmw_pvrdma: Add UAR SRQ macros in ABI header file Bryan Tan
[not found] ` <20171208190317.GA3636-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-08 23:28 ` Jason Gunthorpe
[not found] ` <20171208232809.GA23239-uk2M96/98Pc@public.gmane.org>
2017-12-11 18:35 ` Bryan Tan [this message]
[not found] ` <20171211183554.GB20684-qXbCdz4EeRo1jLI2hToXVI42T8aCTgcwy4vvyvUx+exJXi8ZT2ovy+oDBWuYMCC/JZORHMmSJCU@public.gmane.org>
2017-12-11 20:12 ` 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=20171211183554.GB20684@bryantan-devbox.prom.eng.vmware.com.prom.eng.vmware.com \
--to=bryantan-pghwnbhtmq7qt0dzr+alfa@public.gmane.org \
--cc=jgg-uk2M96/98Pc@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
/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.