Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RFE: add the bpf tests #90

Closed
wants to merge 2 commits into from
Closed

RFE: add the bpf tests #90

wants to merge 2 commits into from

Conversation

olsajiri
Copy link
Contributor

Adding test for bpf audit notification.

Kernel changes are on the way to upstream:
https://lore.kernel.org/netdev/20191206214934.11319-1-jolsa@kernel.org/

audit-userspace changes are in here:
linux-audit/audit-userspace#104

@pcmoore pcmoore changed the title tests: add the bpf tests RFE: add the bpf tests Dec 12, 2019
@pcmoore
Copy link
Member

pcmoore commented Dec 12, 2019

Thanks @olsajiri, unfortunately it looks like the Travis build identified some style problems with the PR, can you fix those?

Signed-off-by: Jiri Olsa <jolsa@redhat.com>
@olsajiri
Copy link
Contributor Author

@pcmoore , that check-syntax stuff forced me to 'fix' bpf.c so it looks mighty ugly now ;-)

also I had to change some other source that did not come from me

@olsajiri
Copy link
Contributor Author

@pcmoore ping, thanks

@pcmoore
Copy link
Member

pcmoore commented Dec 29, 2019

Hi @olsajiri, generally I don't merge new tests until the kernel code has reached Linus' tree or the audit/next branch.

@olsajiri
Copy link
Contributor Author

@pcmoore ok, so I assume you're ok with the change..?

@pcmoore
Copy link
Member

pcmoore commented Dec 29, 2019

Sorry, I haven't had a chance to look at it beyond the initial comments yet.

@pcmoore
Copy link
Member

pcmoore commented Mar 9, 2020

Merged via e7a70dc, thanks @olsajiri!

@pcmoore pcmoore closed this Mar 9, 2020
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.

None yet

2 participants