On 1/19/24 13:05, Simon Horman wrote:
> On Thu, Jan 18, 2024 at 04:30:06PM +0000, Simon Horman wrote:
>> On Thu, Jan 18, 2024 at 03:59:05PM +0100, Ilya Maximets wrote:
>>> GCC 14 started to advertise c_atomic extension, older versions didn't
>>> do that. Add check for __clang__, so GCC doesn't include headers
>>> designed for Clang.
>>>
>>> Another option would be to prefer stdatomic implementation instead,
>>> but some older versions of Clang are not able to use stdatomic.h
>>> supplied by GCC as described in commit:
>>> 07ece367fb5f ("ovs-atomic: Prefer Clang intrinsics over <stdatomic.h>.")
>>>
>>> This change fixes OVS build with GCC on Fedora Rawhide (40).
>>>
>>> Reported-by: Jakob Meng <[email protected]>
>>> Signed-off-by: Ilya Maximets <[email protected]>
>>
>> Recheck-request: github-robot
>
> That run did not work either [1],
> but I was able to get a successful run in my own
> GitHub repository [2].
>
> [1] https://github.com/ovsrobot/ovs/actions/runs/7572217223/job/20625604785
> [2] https://github.com/horms/ovs/actions/runs/7581260421
>
> Acked-by: Simon Horman <[email protected]>
>
Thanks, Simon, Jakob and Eelco!
Applied and backported down to 2.17.
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
