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

kmod-ipsec: enable CONFIG_XFRM_STATISTICS CONFIG_XFRM_MIGRATE #2142

Closed

Conversation

antonyantony
Copy link
Contributor

ipsec enhancements, @lucize

CONFIG_XFRM_STATISTICS helps debug Strongswan or libreswan
CONFIG_XFRM_MIGRATE used by librewan to support mobike

Signed-off-by: Antony Antony <antony@phenome.org>

@ynezz ynezz added needs reviewer core packages pull request/issue for core (in-tree) packages labels Jun 17, 2019
@lucize
Copy link
Contributor

lucize commented Jun 17, 2019

@antonyantony maybe more is needed here because it halts at asking

Transformation sub policy support (XFRM_SUB_POLICY) [N/y/?] n
Transformation migrate database (XFRM_MIGRATE) [N/y/?] (NEW) y
Transformation statistics (XFRM_STATISTICS) [N/y/?] (NEW)

@antonyantony
Copy link
Contributor Author

@antonyantony maybe more is needed here because it halts at asking

Transformation sub policy support (XFRM_SUB_POLICY) [N/y/?] n
Transformation migrate database (XFRM_MIGRATE) [N/y/?] (NEW) y
Transformation statistics (XFRM_STATISTICS) [N/y/?] (NEW)

I am not sure how to fix this patch.
When I try on my existing branch it works. However, on vanilla branch it is missing something.

Any suggestions?

@antonyantony
Copy link
Contributor Author

it worked for me on sunxi aarch64_cortex-a53, where everything was Y no M(modules)

When I try M on mvebu this does not work, I think because when lbreswan is a module these config options trying to be modules.

.config:6493:warning: symbol value 'm' invalid for XFRM_MIGRATE
.config:6494:warning: symbol value 'm' invalid for XFRM_STATISTICS

These two config options are bool. While the ipsec itself is a module.

how I can force the config options to Y when KernelPackage/ipsec is M.

config XFRM_MIGRATE
bool "Transformation migrate database"
config XFRM_STATISTICS
bool "Transformation statistics"

@jow-
Copy link
Contributor

jow- commented Jun 26, 2019

Like this:

KCONFIG:= \
	CONFIG_NET_KEY \
	CONFIG_XFRM_USER \
	CONFIG_XFRM_STATISTICS=y \
	CONFIG_XFRM_MIGRATE=y \
	CONFIG_INET_IPCOMP \
	CONFIG_XFRM_IPCOMP

If symbols are not suffixed with =n, =y or =m they'll inherit the choice state of the parent kmod package.

CONFIG_XFRM_STATISTICS helps debug Strongswan or libreswan
CONFIG_XFRM_MIGRATE used by librewan to support mobike

Signed-off-by: Antony Antony <antony@phenome.org>
@antonyantony
Copy link
Contributor Author

thanks @jow- I think it work. @lucize would you try again?
I pushed an update to the branch.

@lucize
Copy link
Contributor

lucize commented Jul 1, 2019

@antonyantony
sorry, I don't know how to test this, I could only start xfrmi with strongswan but even then I think something I'm doing wrong as I don't see traffic on the interface (on fortigate I see traffic).
oddly enough I can connect only to the xfrm interface, maybe marking is not working as it should
on libreswan I couldn't find a config for use with xfrmi like if_id_in/out

@antonyantony
Copy link
Contributor Author

@antonyantony
sorry, I don't know how to test this, I could only start xfrmi with strongswan but even then I think something I'm doing wrong as I don't see traffic on the interface (on fortigate I see traffic).
oddly enough I can connect only to the xfrm interface, maybe marking is not working as it should
on libreswan I couldn't find a config for use with xfrmi like if_id_in/out

this change is to support mobike, RFC 4555. e.g iPhone moving from 4G to WiFi while keeping the tunnel up when moving networks. I think biggest test is would the kernel compile with this change. Then libreswan, userland will/should do the right thing.

@lucize
Copy link
Contributor

lucize commented Jul 10, 2019

the kernel compiles no problem on my test systems

@adschm
Copy link
Member

adschm commented May 7, 2020

Hi, since this seems to be not a fix, but a feature, and none of the committer seems to be interested in reviewing it for almost a year, I will close it now (i.e. "Won't implement").

I'm sorry, but better crush your remaining hopes now than waiting another year where supposedly nothing different will happen.

@adschm adschm closed this May 7, 2020
@paulwouters
Copy link

paulwouters commented May 6, 2024

This is breaking generic IPsec support. Please reconsider. It is a stable feature of billions of kernels out there. It is needed for proper functioning of libreswan and strongswan packages. If you are not confident on XFRM options, at least follow the linus upstream kernel for defaults. Don't manually disable things.

ping @adschm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core packages pull request/issue for core (in-tree) packages needs reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants