[meta-virtualization] [meta-cloud-services][PATCH 1/3] libibverbs: add new recpie

[Bruce Ashfield]

On Tue, Jul 25, 2017 at 8:44 AM, Mark Asselstine <
mark.asselstine at windriver.com> wrote:

> On Tuesday, July 25, 2017 3:38:24 PM EDT Yi Zhao wrote:
> > 在 2017年07月25日 03:29, Mark Asselstine 写道:
> > > On Mon, Jul 24, 2017 at 1:14 AM, Yi Zhao <yi.zhao at windriver.com>
> wrote:
> > >> Libibverbs is a library that allows userspace processes to use
> > >> InfiniBand/RDMA 'verbs' directly.
> > >>
> > >> Signed-off-by: Yi Zhao <yi.zhao at windriver.com>
> > >> ---
> > >>
> > >>   recipes-extended/libibverbs/libibverbs_1.1.7.bb | 27
> > >>   +++++++++++++++++++++++++ 1 file changed, 27 insertions(+)
> > >>   create mode 100644 recipes-extended/libibverbs/libibverbs_1.1.7.bb
> > >>
> > >> diff --git a/recipes-extended/libibverbs/libibverbs_1.1.7.bb
> > >> b/recipes-extended/libibverbs/libibverbs_1.1.7.bb new file mode
> 100644
> > >> index 0000000..e75f90d
> > >> --- /dev/null
> > >> +++ b/recipes-extended/libibverbs/libibverbs_1.1.7.bb
> > >> @@ -0,0 +1,27 @@
> > >> +SUMMARY = "Support InfiniBand verbs"
> > >> +DESCRIPTION = "Libibverbs is a library that allows userspace
> processes
> > >> to use InfiniBand/RDMA 'verbs' directly." +HOMEPAGE =
> > >> "http://www.openfabrics.org/downloads/verbs/"
> > >
> > > I am fine with this series overall. It is nice to see an uprev to
> > > gluster and an effort to get it working again. I only have one
> > > request. The page above basically indicates that software which used
> > > libibverbs should transition to rdma_core and that this software is
> > > basically deprecated. Are we using this instead of rdma_core because
> > > Gluster has yet to make this transition? We need to capture whatever
> > > the reasons are in the commit log.
> >
> > I checked the commit log for glusterfs, there is no explicit message
> > said it only depend on old libibverbs.
> > For rdma-core, it is a collection including libibverbs and librdmacm. I
> > did a quick test that glusterfs can be built when depending on it.
> > So maybe we could using rdma-core instead of libibverbs and librdmacm ?
>
> I am fine with the current approach but please add the above to the commit
> log
> for the libibverbs commit. Something like
>
> "libibverbs is now part of rdma-core and the standalone libibverbs project
> has
> been deprecated. Rough testing shows that we can move to use rdma-core
> however
> we have yet to perform extensive testing with this change. We therefor will
> continue to use the standalone libibverbs at this point in time. This is
> also
> inline with the upstream glusterfs project that still lists libibverbs as a
> dependency."
>

Agreed.

I'm going to drop this series and wait for a v2 with the updated commit log.

Make sure to send all three patches in the series again.

Cheers,

Bruce


>
> MarkA
>
> >
> > Regards,
> > Yi
> >
> > > Regards,
> > > Mark
> > >
> > >> +SECTION = "libs/devel"
> > >> +
> > >> +LICENSE = "BSD"
> > >> +LIC_FILES_CHKSUM = "file://COPYING;md5=7c557f27dd
> 795ba77cc419dddc656b51"
> > >> +
> > >> +# Official repo is at
> > >> git://git.kernel.org/pub/scm/libs/infiniband/libibverbs.git +SRC_URI
> =
> > >> "http://www.openfabrics.org/downloads/verbs/${BPN}-${PV}.tar.gz" +
> > >> +SRC_URI[md5sum] = "5aea59e4766ae4548419010ac84032ff"
> > >> +SRC_URI[sha256sum] =
> > >> "31b4114bf32d41c9fc81a862ec5febed1e982975f21fa177ea9c7516c9064690" +
> > >> +inherit autotools
> > >> +
> > >> +PACKAGES += "${PN}-utils"
> > >> +FILES_${PN} = "${sysconfdir} ${libdir}/*.so.*"
> > >> +FILES_${PN}-utils = "${bindir}"
> > >> +
> > >> +PACKAGECONFIG ??= ""
> > >> +PACKAGECONFIG[valgrind] = "--with-valgrind,--without-val
> grind,valgrind,"
> > >> +
> > >> +do_install_append() {
> > >> +        mkdir -p ${D}${sysconfdir}/libibverbs.d
> > >> +        rm -f ${D}${libdir}/libibverbs.la
> > >> +}
> > >> --
> > >> 2.7.4
> > >>
> > >> --
> > >> _______________________________________________
> > >> meta-virtualization mailing list
> > >> meta-virtualization at yoctoproject.org
> > >> https://lists.yoctoproject.org/listinfo/meta-virtualization
>
>
> --
> _______________________________________________
> meta-virtualization mailing list
> meta-virtualization at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-virtualization
>



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/meta-virtualization/attachments/20170725/2a8d8b27/attachment-0001.html>