[ovs-dev] [PATCH dpdk-latest 0/3] Revert DPDK 20.05/20.08 sparse commits

Eli Britstein elibr at nvidia.com
Wed Oct 21 11:51:29 UTC 2020


On 10/21/2020 11:56 AM, Ilya Maximets wrote:
> External email: Use caution opening links or attachments
>
>
> On 10/21/20 10:34 AM, David Marchand wrote:
>> On Wed, Oct 21, 2020 at 9:48 AM Eli Britstein <elibr at nvidia.com> wrote:
>>> With DPDK 20.11-rc1, travis fails. Reverting the commits that were done
>>> for DPDK 20.05/20.08 fixes it.
>>>
>>> Note: maybe we can avoid merging them to master instead of applying and
>>> reverting.
>>>
>>> Travis:
>>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci.org%2Fgithub%2Felibritstein%2FOVS%2Fbuilds%2F737627018&data=04%7C01%7Celibr%40nvidia.com%7C0267ab57c1bb41e5044308d8759f380f%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637388673986707821%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=SRcJy5GwwbthlqdFWvIUoeU65D%2F7KY7l4j15f5Dt28k%3D&reserved=0
>> This is due to yesterday changes in sparse:
>> https://git.kernel.org/pub/scm/devel/sparse/sparse.git/commit/?id=cf8f104749f5bca36852989297af8cc19ff24d5f
>>
>> I'd rather keep those wrappers in OVS if someone is still using a
>> previous version of sparse.
>> We can simply test if __ATOMIC_* is already defined.
> If that is the only reason then it's better to just wrap the definitions in
> 'ifndef', so stable versions of Sparse will work.  I agree.

Please abandon this series. I've posted 
https://patchwork.ozlabs.org/project/openvswitch/patch/20201021114912.2552-1-elibr@nvidia.com/ 
instead.

>
> Best regards, Ilya Maximets.


More information about the dev mailing list