On 05/08/2015 21:29, Bill Fischofer wrote:
For linux-generic all of those size asserts are vestigial and serve no purpose because all buffer/packet headers are cache-aligned elsewhere. I can submit a patch to remove the offending ASSERT.

+1 to remove it. warning is not needed.

Maxim.

On Fri, May 8, 2015 at 1:24 PM, Mike Holmes <mike.hol...@linaro.org <mailto:mike.hol...@linaro.org>> wrote:

    
https://ci.linaro.org/view/odp-ci/job/odp-tool-check/build_type=m32_on_64,label=docker-utopic/59/console

    In file included from ./include/odp/debug.h:28:0,
                     from odp_classification.c:10:
    ../../include/odp/api/debug.h:48:40: error: static assertion
    failed: "ODP_PACKET_HDR_T__SIZE_ERR2"
     #define _ODP_STATIC_ASSERT(cond, msg)  _Static_assert(cond, msg)
                                            ^
    ./include/odp_packet_internal.h:142:1: note: in expansion of macro
    '_ODP_STATIC_ASSERT'
     _ODP_STATIC_ASSERT(sizeof(odp_packet_hdr_t) % sizeof(uint64_t) == 0,
     ^
    In file included from ./include/odp/debug.h:28:0,
                     from odp_crypto.c:12:
    ../../include/odp/api/debug.h:48:40: error: static assertion
    failed: "ODP_PACKET_HDR_T__SIZE_ERR2"
     #define _ODP_STATIC_ASSERT(cond, msg)  _Static_assert(cond, msg)
                                            ^
    ./include/odp_packet_internal.h:142:1: note: in expansion of macro
    '_ODP_STATIC_ASSERT'
     _ODP_STATIC_ASSERT(sizeof(odp_packet_hdr_t) % sizeof(uint64_t) == 0,
     ^

    BISECT looks like it is this commit

    1a042da102ee03f1ba05821af0664906e3409d09 is the first bad commit
    commit 1a042da102ee03f1ba05821af0664906e3409d09
    Author: Bill Fischofer <bill.fischo...@linaro.org
    <mailto:bill.fischo...@linaro.org>>
    Date:   Fri Apr 10 10:52:15 2015 -0500

        linux-generic: buffer: restructure user mdatadata fields
        Signed-off-by: Bill Fischofer <bill.fischo...@linaro.org
    <mailto:bill.fischo...@linaro.org>>
        Reviewed-and-tested-by: Ciprian Barbu
    <ciprian.ba...@linaro.org <mailto:ciprian.ba...@linaro.org>>
        Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org
    <mailto:maxim.uva...@linaro.org>>

    :040000 040000 eb6bd6ceb600e8328a798b26cb34678231e53a1f
    5bdcd46831800064aac606163a03e79cf2665306 Mplatform

    Mike

-- Mike Holmes
    Technical Manager - Linaro Networking Group
    Linaro.org <http://www.linaro.org/>***│ *Open source software for
    ARM SoCs


    _______________________________________________
    lng-odp mailing list
    lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
    https://lists.linaro.org/mailman/listinfo/lng-odp




_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to