Closed schaefi closed 1 year ago
I think there might be even a simpler solution... testing it
Looks good apart from my comment, definitely more robust than random ports and also way faster then looping through filenames.
Looks good apart from my comment, definitely more robust than random ports and also way faster then looping through filenames.
Thanks, so if you are fine with it I'll wait for approval prior merge :)
I would also be interested what happens behind the scene, as you mentioned a kernel module for getting a system wide free port through a semaphore I was curious to see if there is some code, poc or something available. You mentioned @m-kat is working on that but it seems there is not much sharing on this activity on the opensource side of things. So if there is anything you can share I would be interested. Thanks
I would be voting against this solution. Current random solution is bad enough to be considered improved, but it still works by far anyhow. There is no need to replace one bad solution with another bad solution, creating more work and ripples around. 😉
The way it would be proper is to allocate systen-wide port from a range, check it of course, and then store its number in either a file somewhere in /run/...
or (for RO systems) take it already from a /proc/...
which is exposed and allocated already by own module.
I still don't get why not make a first half of it: just allocate a port system-wide, check if it is free and store in a file. Then next time increment it with a next process. Since that file must be stored in a volatile storage (always), the next reboot will remove it.
When using the firecracker-pilot in a resume type flake, the pilot creates a UDS socket for listening on data. Each of this socket connections requires a free port. The former implementation selects the socket by a random number within a given range. However, this is not a stable solution and the risk to select an already occupied port is high. This commit refactors the port allocation to a simply calculate a new port from FIRECRACKER_VSOCK_PORT_START + the PID of the pilot process.
Further conversations on this topic brought up some ideas how to select a system wide free port by e.g a kernel module but as a first improvement to the former implementation this commit should serve the purpose.
How to reproduce
This Fixes #114