Net boot: allow both tftpfs and nfs

[bapt]

Summary

Add a new "netproto" which can be set to NET_TFTP or NET_NFS
if one set the root-path to ip:path then the efi loader and the pxeboot
loader will consider it booting over nfs (meaning no breakage on the current
default behaviour.

to enable tftpfs the 150 dhcp option "tftp server address" should be passed
the loader will then accept 2 syntaxes for "root-path":
"/path": the ip address passed via the 150 option as the ip of the tftp server
"IP:/path": the ip address passed in the rootpath overwrites the one passed via
the 150 option

Test Plan

Tested on both loader.efi and pxeboot

[bapt]

Remove garbage
Also note that guards where added around libstand's net.h because it ends up
being included multiple time

[imp]

lib/libstand/nfs.c

461

Can nfs_open even be called when netproto != NET_NFS?

1106

same question.

lib/libstand/tftp.c

405

same question....

sys/boot/common/dev_net.c

171

I didn't catch this last time, but how is rootip different than tftpip?

sys/boot/i386/libi386/pxe.c

327–333

this code looks identical to the code in dev_net.c. Why not have a common routine?

[emaste]

sys/boot/common/dev_net.c

171

Perhaps switch() since we'll hopefully have NET_HTTP added soon?

[bapt]

lib/libstand/nfs.c

461

yes, each time you open a file it loops over all the filesystems if they are in error (file not found) which happens all the time:
boot.4th not there, all files are compressed in .gz so it first fail to find the non compressed version etc.
on the loop it keeps the error which is not EINVAL

sys/boot/common/dev_net.c

171

tftpip is passed via the dhcp 150 option. to avoid being intrusive we set rootip based on it. then I describe on the summary how one can overwrite it playing to the format of the root-path

Concerning the switch() I would say we should do once NET_HTTP is there, NET_HTTP is more complicated we would need a libstand mini tcp stack

sys/boot/i386/libi386/pxe.c

327–333

The whole pxe.c and common/dev_net.c are really similar, I think we should merge both later. replacing the pxe.c by comment/dev_net.c which is cleaner.

But I first need to go further in common/dev_net.c: I want to make the NETIF_OPEN_CLOSE_ONCE the default otherwise it makes the loading very slow: issuing a dhcp request at every file open (on tftp at least)

[bapt]

anyone?

[tsoome]

I like the idea as such, so even as I don't really have much experience with freebsd netboot support as such, it does sound very reasonable.