Patch http://patches.opendataplane.org/patch/1586/ posted to address this
issue.

On Fri, May 8, 2015 at 1:38 PM, Maxim Uvarov <[email protected]>
wrote:

> 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 <[email protected]
>> <mailto:[email protected]>> 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 <[email protected]
>>     <mailto:[email protected]>>
>>     Date:   Fri Apr 10 10:52:15 2015 -0500
>>
>>         linux-generic: buffer: restructure user mdatadata fields
>>         Signed-off-by: Bill Fischofer <[email protected]
>>     <mailto:[email protected]>>
>>         Reviewed-and-tested-by: Ciprian Barbu
>>     <[email protected] <mailto:[email protected]>>
>>         Signed-off-by: Maxim Uvarov <[email protected]
>>     <mailto:[email protected]>>
>>
>>     :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
>>     [email protected] <mailto:[email protected]>
>>     https://lists.linaro.org/mailman/listinfo/lng-odp
>>
>>
>>
>>
>> _______________________________________________
>> lng-odp mailing list
>> [email protected]
>> https://lists.linaro.org/mailman/listinfo/lng-odp
>>
>
> _______________________________________________
> lng-odp mailing list
> [email protected]
> https://lists.linaro.org/mailman/listinfo/lng-odp
>
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to