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

generalize conditional compile for update_pmtu #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dirkcgrunwald
Copy link

The following change generalizes the conditional compilation mechanism for the update_pmtu conditional compile. The extra argument was introduced in Linux 4.19.93 but common distributions such as Ubuntu 18.04.04 backported that change. This change to the makefile allows checks for specific kernel versions based on uname and also includes the check for numeric kernels >= 4.19.93.

The module compiles and insmods on Ubuntu 18.04.4 but I don't know if it works because no regression testing method (yet).

@PrinzOwO
Copy link
Owner

I consider that it is a good way to use this flag UPDATE_PMTU_BOOL set in Makefile to control update_pmtu. But this file /usr/include/linux/version.h would not be change when user modifies his Linux kernel version.

Ubuntu has provided a way to check Linux kernel version, please refer to the information provided to trace your Linux kernel version.

Many Thanks

@muthuramanecs03g
Copy link
Collaborator

Hi @dirkcgrunwald ,

Thanks for your support.

This repo is part of the free5GC project. Now, this repo forked into free5GC and maintain it there only.

Could you please send a PR to the following repo

https://github.com/free5gc/gtp5g

coolshou pushed a commit to coolshou/gtp5g that referenced this pull request Nov 30, 2023
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.

3 participants