Tamir Duberstein
0cd620a9da
Merge pull request #804 from tamird/deflake-xdp
...
integration/xdp: increase timeout to 60 seconds
1 year ago
Tamir Duberstein
81268b7ac1
integration/xdp: increase timeout to 60 seconds
...
There's absolutely no reason to have a short timeout here. This has
resulted in at least one flake:
https://github.com/aya-rs/bpf-linker/actions/runs/6384472390/job/17327196576 .
1 year ago
Tamir Duberstein
062ebfdc68
integration/xdp: use kernel-allocated port
1 year ago
Tamir Duberstein
64d9799efc
integration/xdp: reduce repetition
1 year ago
ajwerner
c547dd7bcc
Merge pull request #800 from ajwerner/deflake-loaded-at-take-2
...
integration-test: deflake test_loaded_at
1 year ago
Andrew Werner
da832e479d
integration-test: deflake test_loaded_at
...
The test could fail due to the lack of clock monotonicity. This PR
deflakes the test by adding retries.
See
https://github.com/aya-rs/aya/actions/runs/6340369670/job/17221591723 .
1 year ago
Tamir Duberstein
4bb52ad34c
Merge pull request #801 from aya-rs/kill-kernel-on-soft-lockup
...
xtask: terminate QEMU on "BUG: soft lockup"
1 year ago
Tamir Duberstein
5392f9821d
xtask: terminate QEMU on "BUG: soft lockup"
...
See
https://github.com/aya-rs/aya/actions/runs/6329828507/job/17190931752 .
Unlike on kernel panic this message prints periodically so it is
permitted to print twice before QEMU is killed.
1 year ago
Tamir Duberstein
373fb7bf06
Merge pull request #797 from aya-rs/rustfmt-group-imports
...
rustfmt: group_imports = "StdExternalCrate"
1 year ago
Tamir Duberstein
d16e607fd4
rustfmt: group_imports = "StdExternalCrate"
...
High time we stop debating this; let the robots do the work.
1 year ago
Tamir Duberstein
67863833ca
Merge pull request #791 from nrxus/fix-kernel-code-on-submode-gt-255
...
aya: Fix program loading on kernels with a patch > 255
1 year ago
Tamir Duberstein
9eb0419baa
Merge pull request #796 from aya-rs/integration-ebpf-bpf-arch
...
integration-test: fix when host != target
1 year ago
Tamir Duberstein
15de14383d
integration-test: build eBPF for the proper arch
...
Prior to this change we neglected to build integration-ebpf for the
correct target architecture, resulting in test failures on arm64.
1 year ago
Tamir Duberstein
ad460879ef
integration-test: replace macro with function
...
There's just no need for a macro here.
1 year ago
Tamir Duberstein
8cee3f8b01
integration-test: distinguish between success and noop
...
These tests previously produced the same result on certain failures as
they did on success.
1 year ago
Andrés Medina
0a6a2674fa
aya: Fix program loading on kernels with a patch > 255
1 year ago
Tamir Duberstein
2f9c67c735
Merge pull request #795 from aya-rs/clippy
...
appease clippy
1 year ago
Tamir Duberstein
fe13b2eb41
xtask: remove useless copy
...
/tmp/initrd.img is unused.
1 year ago
Tamir Duberstein
cbe9cd9133
xtask: appease new clippy warning
...
```
warning: unnecessary hashes around raw string literal
--> xtask/src/docs.rs:70:17
|
70 | indoc! {r#"
| _________________^
71 | | User-Agent:*
72 | | Disallow: /
73 | | "#},
| |______^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_raw_string_hashes
= note: `#[warn(clippy::needless_raw_string_hashes)]` on by default
help: remove all the hashes around the literal
|
70 ~ indoc! {r"
71 | User-Agent:*
72 | Disallow: /
73 ~ "},
|
```
This false negative was fixed in
https://github.com/rust-lang/rust-clippy/pull/11518 (the title
incorrectly says false positive).
1 year ago
Tamir Duberstein
6720e127b6
Merge pull request #794 from aya-rs/fix-ci
...
Remove 6.4.x kernels from CI
1 year ago
Tamir Duberstein
f01845a379
.github: run tests nightly
...
This should help us catch breakages more quickly.
1 year ago
Tamir Duberstein
6190696346
.github: remove 6.4.x kernel tests
...
These images keep getting pulled, better to get off the treadmill.
1 year ago
dependabot[bot]
8ae5b0d967
Merge pull request #793 from aya-rs/dependabot/cargo/dialoguer-0.11
1 year ago
dependabot[bot]
9680f50c08
build(deps): update dialoguer requirement from 0.10 to 0.11
...
Updates the requirements on [dialoguer](https://github.com/console-rs/dialoguer ) to permit the latest version.
- [Changelog](https://github.com/console-rs/dialoguer/blob/master/CHANGELOG.md )
- [Commits](https://github.com/console-rs/dialoguer/compare/v0.10.0...v0.10.4 )
---
updated-dependencies:
- dependency-name: dialoguer
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
1 year ago
Alessandro Decina
7f9ce062f4
Merge pull request #527 from Tuetuopay/xdpmaps
...
Implement XDP map types
1 year ago
Tuetuopay
0edc13b4d4
bpf: add a shared try_redirect_map function for XDP maps
1 year ago
Tuetuopay
579e3cee22
aya, bpf: misc fixes following review comments
1 year ago
Andrew Stoycos
c130500f18
Merge pull request #769 from astoycos/fix-loaded-at
...
fix load time
1 year ago
astoycos
dffff1ce6b
integration-test: fix load time and add test
...
Time since boot is defined as the UNIX_EPOCH plus the duration
since boot. which is realtime - boottime NOT boottime - realtime.
Add a integration test to ensure this doesn't happen again.
Signed-off-by: astoycos <astoycos@redhat.com>
1 year ago
Tuetuopay
46551de3e7
xtask: bless public-api
1 year ago
Tuetuopay
00dc7a5bd4
maps/xdp: make maps work on kernels not supporting ProgIds
...
On startup, the kernel is probed for support of chained program ids for
CpuMap, DevMap and DevMapHash, and will patch maps at load time to have
the proper size. Then, at runtime, the support is checked and will error
out if a program id is passed when the kernel does not support it.
1 year ago
Tuetuopay
63ce2f013a
bpf/devmap: don't expose `bpf_devmap_value`
...
Use our own type that:
- is stable as not from bindgen
- does not have an union inside
1 year ago
Tuetuopay
c6754c614e
maps/xdp: use ProgramFd instead of impl AsRawFd
...
Not having a generic here allows to pass `None` without specifying the
actual type you don't care about.
1 year ago
Tuetuopay
9ed1d3d281
bpf: add documentation for XDP maps
1 year ago
Tuetuopay
db49633073
bpf: make xdp maps functions safe
...
Values in those map are small enough to return copied values instead of
reference to values.
1 year ago
Tuetuopay
f7fbbcd0e5
aya: fix docstring missing trailing period
1 year ago
Tuetuopay
0647927e32
xdp: add support for chained xdp programs in {cpu,dev}map
...
set/insert functions can now take an optional bpf program fd to run once
the packet has been redirected from the main probe
1 year ago
Tuetuopay
139f382638
aya: add support for map-bound XDP programs
...
Such programs are to be bound to cpumap or devmap instead of the usual
network interfaces.
1 year ago
Tuetuopay
4452364c41
macros: add 'map' option to xdp macro
...
This option allows to place the program in the specific sections to
chain programs with devmaps and cpumaps.
1 year ago
Tuetuopay
ad3087d7eb
bpf: Update XDP maps implementation
...
The implementation changed since the original commit was written, and
some mistakes went in:
- missing bpf_redirect_map wrapper
- extra bpf_map_lookup_elem on maps for which it is forbidden
1 year ago
Tuetuopay
ede3e91014
aya: Update XDP maps implementations
...
Map impls changed since this was first written.
Fixes: 2b726c8 ("aya: Implement XDP Map Types")
1 year ago
Dave Tucker
e90d521a21
bpf: Implement XDP maps
...
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
1 year ago
Dave Tucker
ec8293ab86
aya: Implement XDP Map Types
...
This commit adds implementations for:
- xskmap
- devmap
- devmap_hash
- cpumap
Which can all be used to redirect XDP packets to various different
locations
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
1 year ago
Dave Tucker
42fd82e32b
Merge pull request #790 from dave-tucker/no-map-pinned
1 year ago
Dave Tucker
938f979fe7
aya: Make MapData::pin pub
...
This is to solve a use-case where a user (in this case bpfd) may want
to:
- MapData::from_pin to open a pinned map from bpffs
- MapData::pin to pin that object into another bpffs
Both operations should be easily accomplished without needing to cast
a MapData into a concrete Map type - e.g aya::maps::HashMap.
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
1 year ago
Dave Tucker
0f4021ec89
aya: Remove MapData::pinned
...
BPF objects can be pinned multiple times, to multiple different places.
Tracking whether or not a map is pinned in a bool is therefore not sufficient.
We could track this in a HashSet<PathBuf>, but there is really no reason
to track it at all.
Signed-off-by: Dave Tucker <dave@dtucker.co.uk>
1 year ago
Andrew Stoycos
0b6ea313de
Merge pull request #782 from astoycos/prog-info
...
aya: add program_info() api to `Program`
1 year ago
Tamir Duberstein
41d01f638b
Merge pull request #770 from aya-rs/mapfd-is-owned
...
maps: `MapFd` and `SockMapFd` are owned
1 year ago
Tamir Duberstein
0dacb34d44
maps: fix typos, avoid fallible conversions
1 year ago
Tamir Duberstein
b4d5a1e8db
maps: MapData::{obj, fd} are private
1 year ago