public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgg-uk2M96/98Pc@public.gmane.org>
To: Yishai Hadas <yishaih-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
Cc: "Amrani,
	Ram" <Ram.Amrani-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org>,
	"linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"Elior,
	Ariel" <Ariel.Elior-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org>,
	"Kalderon,
	Michal" <Michal.Kalderon-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org>,
	Matan Barak <matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Bodong Wang <bodong-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	Yishai Hadas <yishaih-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Subject: Re: How far back does rdma-core X supports the kernel Y?
Date: Wed, 1 Nov 2017 12:53:19 -0600	[thread overview]
Message-ID: <20171101185319.GK1030@ziepe.ca> (raw)
In-Reply-To: <3424c3a1-22cb-7330-81d6-0e1bf45ab755-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>

On Wed, Nov 01, 2017 at 08:22:37PM +0200, Yishai Hadas wrote:
> On 11/1/2017 8:00 PM, Jason Gunthorpe wrote:
> >>>          if (attr_mask & ~((1ULL << (IBV_QP_DEST_QPN + 1)) - 1))
> >>
> >>The above rdma-core code is correct, look at verbs.h:
> >>IBV_QP_DEST_QPN = 1 << 20,
> >
> >Okay, then it should be
> >
> >  if (attr_mask & ~(IB_QP_RATE_LIMIT - 1))
> >
> 
> Why the library should support values that were never exposed in verbs.h ?

I think Ram is explaining that the kernel rdma_cm provides the
attr_mask to use with modify_qp, and that the 4.3 kernel sets the
three now reserved bits. The expectation is that user space will flow
those bits and values from the kernel rdma_cm interface back to
uverbs.

Userspace never looks at the attr_mask, and it does not matter that
the values were never in verbs.h.

rdma-core is expected to work on 4.3, so if 4.3 is using this kind of
path, we cannot block it in userspace.

At least, I think that is what Ram is saying.

Jason
--
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

  parent reply	other threads:[~2017-11-01 18:53 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-01 12:07 How far back does rdma-core X supports the kernel Y? Amrani, Ram
     [not found] ` <BN3PR07MB2578883C85DD3855C496F367F85F0-EldUQEzkDQfpW3VS/XPqkOFPX92sqiQdvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2017-11-01 16:39   ` Jason Gunthorpe
     [not found]     ` <20171101163915.GB1030-uk2M96/98Pc@public.gmane.org>
2017-11-01 17:27       ` Amrani, Ram
     [not found]         ` <BN3PR07MB257867500E3A15914B2AD075F85F0-EldUQEzkDQfpW3VS/XPqkOFPX92sqiQdvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2017-11-01 17:33           ` Jason Gunthorpe
     [not found]             ` <20171101173324.GF1030-uk2M96/98Pc@public.gmane.org>
2017-11-02  5:54               ` Leon Romanovsky
2017-11-02  7:01               ` Amrani, Ram
     [not found]                 ` <BN3PR07MB257829836974C1D0F1257387F85C0-EldUQEzkDQfpW3VS/XPqkOFPX92sqiQdvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2017-11-02 21:35                   ` Jason Gunthorpe
2017-11-01 17:28       ` Yishai Hadas
     [not found]         ` <287a10c2-15a7-0748-c59e-3f084e7538f4-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
2017-11-01 18:00           ` Jason Gunthorpe
     [not found]             ` <20171101180001.GH1030-uk2M96/98Pc@public.gmane.org>
2017-11-01 18:22               ` Yishai Hadas
     [not found]                 ` <3424c3a1-22cb-7330-81d6-0e1bf45ab755-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
2017-11-01 18:53                   ` Jason Gunthorpe [this message]
     [not found]                     ` <20171101185319.GK1030-uk2M96/98Pc@public.gmane.org>
2017-11-02  6:55                       ` Amrani, Ram

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=20171101185319.GK1030@ziepe.ca \
    --to=jgg-uk2m96/98pc@public.gmane.org \
    --cc=Ariel.Elior-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org \
    --cc=Michal.Kalderon-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org \
    --cc=Ram.Amrani-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org \
    --cc=bodong-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=yishaih-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org \
    --cc=yishaih-VPRAkNaXOzVWk0Htik3J/w@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox