[meta-intel] [PATCH 1/3] dpdk-dev-libibverbs: fix do_fetch failure in case of multilib

[Anuj Mittal]

On 09/04/2018 05:45 PM, ChenQi wrote:
> We've used dpdk, glusterfs with libibverbs (from meta-cloud-services) 
> for a long time and nothing goes wrong.
> 
> Maybe we should just drop this dpdk-dev-libibverbs recipe? After all, 
> its original repo has gone for a long time ...

This was being discussed on the Yocto list last month too. I also think
this recipe should be removed for now and ideally be replaced to point
to rdma-core if someone needs it later.

Can you please send a patch to remove this?

> 
> Best Regards,
> Chen Qi
> 
> On 01/06/2018 04:17 AM, Cal Sullivan wrote:
>> This sounds like the right way to go to me.
>>
>> In the interim, should I take these patches in order to unblock the 
>> conflict and the multilib issue?
>>
>> Thanks,
>> Cal
>>
>> On 01/04/2018 07:24 AM, Mark Asselstine wrote:
>>> On Wednesday, January 3, 2018 2:02:31 PM EST Cal Sullivan wrote:
>>>> In actually testing this patch, I noticed that I was getting a fetcher
>>>> warning. After a little investigation I found that
>>>> https://github.com/Mellanox/dpdk-dev-libibverbs no longer exists and
>>>> we're only getting this thanks to the YP mirroring. Do you know of any
>>>> other location we could point to?
>>> Maybe we should actually take this as a signal to revisit things. I 
>>> am having
>>> to piece things together as some of the history is lost with the 
>>> removal of
>>> the dpdk-dev-libibverbs repo and newer revisions of Mellanox DPDK 
>>> pages but I
>>> think we have enough to go on to move forward.
>>>
>>> Looking at the history it appears as though dpdk-dev-libibverbs was and
>>> "optimized" version of libibverbs for the ConnectX-3 (reference: 
>>> https://
>>> community.mellanox.com/docs/DOC-2197). The fact that Mellanox has 
>>> dropped the
>>> repository and now simply references libibverbs (and not 
>>> dpdk-dev-libibverbs)
>>> in its latest revision of the Mellanox DPDK documentation (reference: 
>>> https://
>>> community.mellanox.com/docs/DOC-1502). Leads me to believe that we 
>>> should move
>>> on to simply use the latest release of libibverbs.
>>>
>>> So I really think the best solution is to drop dpdk-dev-libibverbs 
>>> and simply
>>> use libibverbs. This would drop the need to do any of the 'virtual/' do
>>> jiggery. We can host the libibverbs recipe here still or move to 
>>> oe-core,
>>> Bruce can use PREFERRED_VERSION to continue to use the required 
>>> version in
>>> meta-cloud-services...
>>>
>>> Sorry about taking my time in coming to this conclusion. I was short 
>>> on time
>>> and flush with work before the holiday so didn't dig in enough to 
>>> make the
>>> right call.
>>>
>>> MarkA
>>>
>>>> Thanks,
>>>> Cal
>>>>
>>>> On 01/02/2018 05:27 PM, Chen Qi wrote:
>>>>> Fix to correctly set SRC_URI and S to avoid do_fetch failure in 
>>>>> case of
>>>>> multilib.
>>>>>
>>>>> Signed-off-by: Chen Qi <Qi.Chen at windriver.com>
>>>>> ---
>>>>>
>>>>> .../dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0.0.0.bb | 8
>>>>>    ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
>>>>>
>>>>> diff --git
>>>>> a/recipes-extended/dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0. 
>>>>>
>>>>> 0.0.bb
>>>>> b/recipes-extended/dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0. 
>>>>>
>>>>> 0.0.bb index e40c63b..9118494 100644
>>>>> ---
>>>>> a/recipes-extended/dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0. 
>>>>>
>>>>> 0.0.bb +++
>>>>> b/recipes-extended/dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0. 
>>>>>
>>>>> 0.0.bb @@ -3,7 +3,7 @@ HOMEPAGE =
>>>>> "https://github.com/Mellanox/dpdk-dev-libibverbs">
>>>>>    LICENSE = "GPLv2"
>>>>>    LIC_FILES_CHKSUM =
>>>>>    "file://COPYING;md5=7c557f27dd795ba77cc419dddc656b51"
>>>>>
>>>>> -SRC_URI =
>>>>> "https://github.com/Mellanox/dpdk-dev-libibverbs/archive/libibverbs-${PV} 
>>>>>
>>>>> .tar.gz;name=${PN} \ +SRC_URI =
>>>>> "https://github.com/Mellanox/dpdk-dev-libibverbs/archive/libibverbs-${PV} 
>>>>>
>>>>> .tar.gz \>
>>>>>               file://init_c.patch \
>>>>>               file://0001-Fix-build-with-clang.patch \
>>>>> file://0002-typecast-enum-to-int-before-comparison.patch \
>>>>>
>>>>> @@ -11,8 +11,8 @@ SRC_URI =
>>>>> "https://github.com/Mellanox/dpdk-dev-libibverbs/archive/libibverbs-${> 
>>>>>
>>>>>               file://0004-Fix-clang-warnings.patch \
>>>>>               "
>>>>>
>>>>> -SRC_URI[dpdk-dev-libibverbs.md5sum] = 
>>>>> "65234ee278eb437a7069326f37cd4d86"
>>>>> -SRC_URI[dpdk-dev-libibverbs.sha256sum] =
>>>>> "a6471515556cb8d10ad471bb7efb8cf760b248a28aceb57d4534d50d572f56cd"
>>>>> +SRC_URI[md5sum] = "65234ee278eb437a7069326f37cd4d86"
>>>>> +SRC_URI[sha256sum] =
>>>>> "a6471515556cb8d10ad471bb7efb8cf760b248a28aceb57d4534d50d572f56cd">
>>>>>    # A machine needs to enable this using:
>>>>>    # COMPATIBLE_MACHINE_pn-dpdk-dev-libibverbs = "<machine name>"
>>>>>
>>>>> @@ -20,7 +20,7 @@ SRC_URI[dpdk-dev-libibverbs.sha256sum] =
>>>>> "a6471515556cb8d10ad471bb7efb8cf760b248>
>>>>>    COMPATIBLE_MACHINE = "null"
>>>>>    COMPATIBLE_HOST_libc-musl_class-target = "null"
>>>>>
>>>>> -S = "${WORKDIR}/${PN}-libibverbs-${PV}"
>>>>> +S = "${WORKDIR}/dpdk-dev-libibverbs-libibverbs-${PV}"
>>>>>
>>>>>    COMPATIBLE_HOST = '(i.86|x86_64).*-linux'
>>>>>    DEPENDS = "libnl"
>>>
>>
>>
>