mbox series

[v2,0/8] ipconfig: NTP server support, bug fixes, documentation improvements

Message ID 20180424025639.2228-1-chris@chrisn.me.uk
Headers show
Series ipconfig: NTP server support, bug fixes, documentation improvements | expand

Message

Chris Novakovic April 24, 2018, 2:56 a.m. UTC
This series (against net-next) makes various improvements to ipconfig:

 - Patch #1 correctly documents the behaviour of parameter 4 in the
   "ip=" and "nfsaddrs=" command line parameter.
 - Patch #2 tidies up the printk()s for reporting configured name
   servers.
 - Patch #3 fixes a bug in autoconfiguration via BOOTP whereby the IP
   addresses of IEN-116 name servers are requested from the BOOTP
   server, rather than those of DNS name servers.
 - Patch #4 requests the number of DNS servers specified by
   CONF_NAMESERVERS_MAX when autoconfiguring via BOOTP, rather than
   hardcoding it to 2.
 - Patch #5 fully documents the contents and format of /proc/net/pnp in
   Documentation/filesystems/nfs/nfsroot.txt.
 - Patch #6 fixes a bug whereby bogus information is written to
   /proc/net/pnp when ipconfig is not used.
 - Patch #7 creates a new procfs directory for ipconfig-related
   configuration reports at /proc/net/ipconfig.
 - Patch #8 allows for NTP servers to be configured (manually on the
   kernel command line or automatically via DHCP), enabling systems with
   an NFS root filesystem to synchronise their clock before mounting
   their root filesystem. NTP server IP addresses are written to
   /proc/net/ipconfig/ntp_servers.

Changes from v1:

 - David requested that a new directory /proc/net/ipconfig be created to
   contain ipconfig-related configuration reports, which is implemented
   in the new patch #7. NTP server IPs are now written to this directory
   instead of /proc/net/ntp in the new patch #8.
 - Cong and David both requested that the modification to CREDITS be
   dropped. This patch has been removed from the series.

Chris Novakovic (8):
  ipconfig: Document setting of NIS domain name
  ipconfig: Tidy up reporting of name servers
  ipconfig: BOOTP: Don't request IEN-116 name servers
  ipconfig: BOOTP: Request CONF_NAMESERVERS_MAX name servers
  ipconfig: Document /proc/net/pnp
  ipconfig: Correctly initialise ic_nameservers
  ipconfig: Create /proc/net/ipconfig directory
  ipconfig: Write NTP server IPs to /proc/net/ipconfig/ntp_servers

 Documentation/filesystems/nfs/nfsroot.txt |  70 ++++++++++++--
 net/ipv4/ipconfig.c                       | 151 +++++++++++++++++++++++++++---
 2 files changed, 200 insertions(+), 21 deletions(-)

Comments

David Miller April 24, 2018, 5:40 p.m. UTC | #1
From: Chris Novakovic <chris@chrisn.me.uk>
Date: Tue, 24 Apr 2018 03:56:31 +0100

> This series (against net-next) makes various improvements to ipconfig:
> 
>  - Patch #1 correctly documents the behaviour of parameter 4 in the
>    "ip=" and "nfsaddrs=" command line parameter.
>  - Patch #2 tidies up the printk()s for reporting configured name
>    servers.
>  - Patch #3 fixes a bug in autoconfiguration via BOOTP whereby the IP
>    addresses of IEN-116 name servers are requested from the BOOTP
>    server, rather than those of DNS name servers.
>  - Patch #4 requests the number of DNS servers specified by
>    CONF_NAMESERVERS_MAX when autoconfiguring via BOOTP, rather than
>    hardcoding it to 2.
>  - Patch #5 fully documents the contents and format of /proc/net/pnp in
>    Documentation/filesystems/nfs/nfsroot.txt.
>  - Patch #6 fixes a bug whereby bogus information is written to
>    /proc/net/pnp when ipconfig is not used.
>  - Patch #7 creates a new procfs directory for ipconfig-related
>    configuration reports at /proc/net/ipconfig.
>  - Patch #8 allows for NTP servers to be configured (manually on the
>    kernel command line or automatically via DHCP), enabling systems with
>    an NFS root filesystem to synchronise their clock before mounting
>    their root filesystem. NTP server IP addresses are written to
>    /proc/net/ipconfig/ntp_servers.
> 
> Changes from v1:
> 
>  - David requested that a new directory /proc/net/ipconfig be created to
>    contain ipconfig-related configuration reports, which is implemented
>    in the new patch #7. NTP server IPs are now written to this directory
>    instead of /proc/net/ntp in the new patch #8.
>  - Cong and David both requested that the modification to CREDITS be
>    dropped. This patch has been removed from the series.

Series applied, thank you.