From: "Steve Wise" <swise-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
To: 'Nicolas Morey-Chaisemartin'
<NMoreyChaisemartin-l3A5Bk7waGM@public.gmane.org>,
'Jason Gunthorpe'
<jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
Cc: 'Leon Romanovsky' <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
'Doug Ledford' <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: RE: rdma-core build environment enabling out-of-core providers
Date: Fri, 4 Aug 2017 06:30:49 -0500 [thread overview]
Message-ID: <011f01d30d15$1fa92750$5efb75f0$@opengridcomputing.com> (raw)
In-Reply-To: <7cc0ba31-7a05-4db1-5807-4d971fc3d773-l3A5Bk7waGM@public.gmane.org>
>
> Le 03/08/2017 à 19:07, Jason Gunthorpe a écrit :
> > On Thu, Aug 03, 2017 at 05:01:16PM +0200, Nicolas Morey-Chaisemartin
> wrote:
> >
> >> I haven't looked into the build system but isn't this something the
> >> devel rpm could provide ?
> > Conceptually, perhaps, but someone has to figure out how to split out
> > the necessary cmake bits into something reusable, it is no longer
> > entirely trivial :)
> Yes, this is going to take some time.
>
> > Also don't forget that the internal API inside the header files
> > changes, a provider that has been modified for rdma core 15 will not
> > compile on rdma core 13, just like the kernel..
> I guess there are enough VERSION defines in rdma-core to make the
> provider portable.
> But yes, supporting multiple versions is a lot more work.
>
I'm not sure the rdma-core needs to worry about this, other than to provide a compile-time way to detect the version. IE I don't think rdma-core v15 should bother supporting out-of-tree providers that don't support the v15 provider<->libibverbs API. It would be up to the out-of-tree providers to support both v13 and v15, for example.
> > I just wonder if it is worth the all the proposed work, I expect out
> > of tree providers to be very rare and temporary things.
>
> The lazy (smart?) thing to do here is to wait for an out-of-tree provider to
> need it (Steve ?) and give them some support to do it.
> If they can't be bothered, it's probably not worth it. If they want it, they'll
> do most of the work and the feature gets upstreamed.
>
Oh I'm already bothered. 😊 I will sign up for this. If I fall in and sink, hopefully Jason will fish me out of the drink. Cmake is new to me.
> >
> >> If this is done upstream once and for all, and not by every
> >> out-of-tree provider, it is not fragile anymore.
> > Someone would also have to contribute some kind of test infrastructure
> > to keep it working for upstream.. If it doesn't run in cbuild I don't
> > test it :P
>
> It shouldn't be too hard to automatically extract one of the in-tree provider
> and try to compile it as an out-of-tree.
> And this way it's always up-to-date for testing.
>
That's a good idea. We could also have a simple "noop" type provider that at least registers with the core but never claims any devices. This would at least verify the build/install, and provider-load by libibverbs.
Steve.
--
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-08-04 11:30 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-02 13:55 rdma-core build environment enabling out-of-core providers Steve Wise
2017-08-02 16:09 ` Jason Gunthorpe
[not found] ` <20170802160900.GB21208-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2017-08-02 19:16 ` Steve Wise
2017-08-02 19:34 ` Jason Gunthorpe
[not found] ` <20170802193403.GA23777-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2017-08-02 20:05 ` Steve Wise
2017-08-02 20:23 ` Jason Gunthorpe
[not found] ` <20170802202300.GA24244-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2017-08-03 5:26 ` Leon Romanovsky
2017-08-03 7:08 ` Nicolas Morey-Chaisemartin
[not found] ` <44b511c6-45fa-d5aa-4d4c-e47d2edcf604-l3A5Bk7waGM@public.gmane.org>
2017-08-03 13:48 ` Steve Wise
2017-08-03 14:47 ` Jason Gunthorpe
[not found] ` <20170803144733.GA13127-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2017-08-03 15:01 ` Nicolas Morey-Chaisemartin
[not found] ` <890f961c-7aa8-1a73-e98d-f7ce9da333d7-l3A5Bk7waGM@public.gmane.org>
2017-08-03 17:07 ` Jason Gunthorpe
[not found] ` <20170803170713.GD13127-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2017-08-04 6:02 ` Nicolas Morey-Chaisemartin
[not found] ` <7cc0ba31-7a05-4db1-5807-4d971fc3d773-l3A5Bk7waGM@public.gmane.org>
2017-08-04 11:30 ` Steve Wise [this message]
2017-08-04 11:52 ` Nicolas Morey-Chaisemartin
[not found] ` <188beb70-9776-33fc-6f72-cc00e968db73-l3A5Bk7waGM@public.gmane.org>
2017-08-04 13:14 ` Steve Wise
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='011f01d30d15$1fa92750$5efb75f0$@opengridcomputing.com' \
--to=swise-7bpotxp6k4+p2yhjcf5u+vpxobypeauw@public.gmane.org \
--cc=NMoreyChaisemartin-l3A5Bk7waGM@public.gmane.org \
--cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org \
--cc=leon-DgEjT+Ai2ygdnm+yROfE0A@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.