cpud: add an option to sleep before serving #248
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When cpud runs as init, and serves mDNS, it can start serving before
mDNS will work. It is still not clear why this fails, but adding a
5 second sleep before serve() resolves it for now. Such a sleep
is obviously undesirable, but the default is 0.
To set the timeout when cpud runs as init, add -- --sleepBeforeServing=5s
to the kernel command line.
This sleep is required to make mDNS work for in-flash or netbooted images.
BUT: if you have it, you don't even need a DHCP server to use it.
rminnich@pop-os:~/oreboot/linux$ sidecore . /bbin/ip a
...
2: eth0: <UP,BROADCAST,MULTICAST> mtu 1500 state UP
link/ether 6c:cf:39:00:48:20
inet6 fe80::6ecf:39ff:fe00:4820 scope link
valid_lft forever preferred_lft forever
rminnich@pop-os:~/oreboot/linux$ sidecore vf2 date
Thu Jan 1 00:11:44 UTC 1970
rminnich@pop-os:~/oreboot/linux$
vf2 is in my ~/.ssh/config as
Host vf2
HostName fe80::6ecf:39ff:fe00:4820
IdentityFile ~/.ssh/cpu_rsa
i.e. link-local only
You can bring up a set of machines, get their link local address via mDNS, and then use link local from then on.
No DHCP server needed.
In fact this on command
sidecore . /bbin/ip l
will get the macs for ALL machines.
Note: this is why it pays to just put all the u-root commands in the image.
One such command to do this:
(p=$PWD && \
cd ~/go/src/github.com/u-root/cpu/cmds/cpud/ && \
GOARCH=riscv64 u-root -tags mDNS \
-uroot-source ~/go/src/github.com/u-root/u-root/ \
-files ~/.ssh/cpu_rsa.pub:key.pub \
-initcmd=/bbin/cpud \
. all)
xz --check=crc32 -9 --lzma2=dict=1MiB \
--stdout /tmp/initramfs.linux_riscv64.cpio \
| dd conv=sync bs=512 of=emb.cpio.xz
Signed-off-by: Ronald G Minnich <[email protected]>
orangecms
approved these changes
Sep 23, 2023
rminnich
force-pushed
the
more-debug-add-sleep-option
branch
from
9ee3ee3 to
0ff3f92
Compare
|
@rminnich I've enabled the auto-merge option in the settings. That label is obsolete and only worked with some 3rd party integration up to some point. This is now an option of the merge button instead. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When cpud runs as init, and serves mDNS, it can start serving before mDNS will work. It is still not clear why this fails, but adding a 5 second sleep before serve() resolves it for now. Such a sleep is obviously undesirable, but the default is 0.
To set the timeout when cpud runs as init, add -- --sleepBeforeServing=5s to the kernel command line.