Skip to content

update_pid_tid_from_sock: add always_inline attr#1091

Merged
kkourt merged 3 commits into
mainfrom
pr/kkourt/always_inline
Jun 19, 2023
Merged

update_pid_tid_from_sock: add always_inline attr#1091
kkourt merged 3 commits into
mainfrom
pr/kkourt/always_inline

Conversation

@kkourt

@kkourt kkourt commented Jun 19, 2023

Copy link
Copy Markdown
Contributor

Testing fixes. Please see patches.

Tests on 5.4 have been failing with:
 tail_calls are not allowed in programs with bpf-to-bpf calls

It seems that recent changes lead the compiler to decide to not inline
the update_pid_tid_from_sock() function. This patch adds an
always_inline attribute (and removes the unnecessary unused attribute)
to the function.

Signed-off-by: Kornilios Kourtis <kornilios@isovalent.com>
@kkourt
kkourt requested a review from a team as a code owner June 19, 2023 07:44
@kkourt
kkourt requested a review from mtardy June 19, 2023 07:44
kkourt added 2 commits June 19, 2023 10:31
TestKprobeLseek does not actually check any events. Add a simple tester.

Signed-off-by: Kornilios Kourtis <kornilios@isovalent.com>
We have been seeing timeouts in go tests. Increase the timeout from 20'
to 40'.

Signed-off-by: Kornilios Kourtis <kornilios@isovalent.com>
@kkourt
kkourt requested a review from willfindlay as a code owner June 19, 2023 08:35
@kkourt
kkourt merged commit e60ba99 into main Jun 19, 2023
@kkourt
kkourt deleted the pr/kkourt/always_inline branch June 19, 2023 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants