Skip to content

refactor(xdp): rename GTPU[IPv4+TEID]->PDR map - #593

Merged
pirog-spb merged 1 commit into
edgecomllc:mainfrom
osmocom:pespin/pr-name
Feb 25, 2025
Merged

pirog-spb merged 1 commit into
edgecomllc:mainfrom
osmocom:pespin/pr-name

Conversation

@pespin

@pespin pespin commented Feb 4, 2025

Copy link
Copy Markdown
Contributor

This map is also used for downlink traffic during tunmap operation (N3 interface), hence previous name was misleading and led to setting wrong value on it.

@pirog-spb pirog-spb changed the title xdp: Rename GTPU[IPv4+TEID]->PDR map refactor(xdp): rename GTPU[IPv4+TEID]->PDR map Feb 8, 2025
@pespin

pespin commented Feb 10, 2025

Copy link
Copy Markdown
Contributor Author

@pirog-spb Is there something I need to do for this one, or can it be merged already?

@pirog-spb

Copy link
Copy Markdown
Collaborator

I've approved the PR. Please merge it.

@pespin

pespin commented Feb 10, 2025

Copy link
Copy Markdown
Contributor Author

@pirog-spb I don't seem to find any button allowing me to merge it, maybe because "some checks were not successful"?

@pirog-spb

Copy link
Copy Markdown
Collaborator

@pespin Could you please rebase the PR on top of current main? All checks in pipeline were fixed there.

This map is also used for downlink traffic during tunmap operation (N3
interface), hence previous name was misleading and led to setting wrong
value on it.
@pirog-spb
pirog-spb merged commit 3dd23b4 into edgecomllc:main Feb 25, 2025
@pespin
pespin deleted the pespin/pr-name branch March 27, 2025 12:25
pirog-spb pushed a commit that referenced this pull request Oct 31, 2026
Rename PdrMapUplinkIp4 to PdrMapTeidIp4 across Go and eBPF code for
clarity (the map is keyed by TEID, not by uplink). Adapted from
upstream 3dd23b4 to release branch layout (pdr.go in api/rest, stub
file, PDR_MAP_SIZE macro unchanged).

Author: Pau Espin Pedrol <[email protected]>
Signed-off-by: pirog-spb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants