Skip to content

chore(ebpf): check for CONFIG_X86_FRED#4713

Merged
geyslan merged 2 commits intoaquasecurity:mainfrom
geyslan:support-fred
Aug 31, 2025
Merged

chore(ebpf): check for CONFIG_X86_FRED#4713
geyslan merged 2 commits intoaquasecurity:mainfrom
geyslan:support-fred

Conversation

@geyslan
Copy link
Member

@geyslan geyslan commented Apr 17, 2025

1. Explain what the PR does

5ff4388 chore(ebpf): update pt_regs struct

This reflects pt_regs struct in kernel 6.15.

a568a5e chore(ebpf): check for CONFIG_X86_FRED

If in a kernel lacking bpf_task_pt_regs() but somehow with CONFIG_X86_FRED
suppport, calculate pt_regs considering FRED extra padding.

2. Explain how to test it

3. Other comments

https://lore.kernel.org/all/[email protected]/

@geyslan geyslan self-assigned this Apr 17, 2025
Copilot AI review requested due to automatic review settings April 17, 2025 12:34

This comment was marked as outdated.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (1)

pkg/ebpf/c/vmlinux_flavors.h:111

  • [nitpick] The structure name 'fred_info___check' uses triple underscores, which can be confusing. Consider renaming it to 'fred_info_check' for improved clarity.
struct fred_info___check {

@geyslan
Copy link
Member Author

geyslan commented Jul 29, 2025

@yanivagman some thoughts?

@yanivagman
Copy link
Collaborator

Does this PR solve some concrete bug we know about?

@geyslan
Copy link
Member Author

geyslan commented Jul 29, 2025

None found so far, it's just a proactive change.

If in a kernel lacking bpf_task_pt_regs() but somehow with CONFIG_X86_FRED
suppport, calculate pt_regs considering FRED extra padding.
This reflects pt_regs struct in kernel 6.15.
Copy link
Collaborator

@yanivagman yanivagman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@geyslan geyslan merged commit 6284394 into aquasecurity:main Aug 31, 2025
41 checks passed
@geyslan geyslan deleted the support-fred branch August 31, 2025 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants