lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 11 Sep 2007 09:11:11 +0100 From: James Chapman <jchapman@...alix.com> To: Jan Engelhardt <jengelh@...putergmbh.de> CC: "Maciej W. Rozycki" <macro@...ux-mips.org>, netdev@...r.kernel.org, "David S. Miller" <davem@...emloft.net>, linux-kernel@...r.kernel.org Subject: Re: [PATCH] ipconfig.c: De-clutter IP configuration report Jan Engelhardt wrote: > On Sep 10 2007 13:09, Maciej W. Rozycki wrote: >> The new code builds fine; no semantic changes. >> >> Please apply, >> >> Maciej >> >> patch-mips-2.6.23-rc5-20070904-ipconfig-printk-2 >> diff -up --recursive --new-file linux-mips-2.6.23-rc5-20070904.macro/net/ipv4/ipconfig.c linux-mips-2.6.23-rc5-20070904/net/ipv4/ipconfig.c >> --- linux-mips-2.6.23-rc5-20070904.macro/net/ipv4/ipconfig.c 2007-09-04 04:56:22.000000000 +0000 >> +++ linux-mips-2.6.23-rc5-20070904/net/ipv4/ipconfig.c 2007-09-10 11:53:19.000000000 +0000 >> @@ -1364,17 +1364,17 @@ static int __init ip_auto_config(void) >> /* >> * Clue in the operator. >> */ >> - printk("IP-Config: Complete:"); >> - printk("\n device=%s", ic_dev->name); >> - printk(", addr=%u.%u.%u.%u", NIPQUAD(ic_myaddr)); >> - printk(", mask=%u.%u.%u.%u", NIPQUAD(ic_netmask)); >> - printk(", gw=%u.%u.%u.%u", NIPQUAD(ic_gateway)); >> - printk(",\n host=%s, domain=%s, nis-domain=%s", >> - utsname()->nodename, ic_domain, utsname()->domainname); >> - printk(",\n bootserver=%u.%u.%u.%u", NIPQUAD(ic_servaddr)); >> - printk(", rootserver=%u.%u.%u.%u", NIPQUAD(root_server_addr)); >> - printk(", rootpath=%s", root_server_path); >> - printk("\n"); >> + pr_info("IP-Config: Complete:\n"); >> + pr_info(" device=%s, addr=%u.%u.%u.%u, " >> + "mask=%u.%u.%u.%u, gw=%u.%u.%u.%u,\n", >> + ic_dev->name, NIPQUAD(ic_myaddr), >> + NIPQUAD(ic_netmask), NIPQUAD(ic_gateway)); >> + pr_info(" host=%s, domain=%s, nis-domain=%s,\n", >> + utsname()->nodename, ic_domain, utsname()->domainname); >> + pr_info(" bootserver=%u.%u.%u.%u, " >> + "rootserver=%u.%u.%u.%u, rootpath=%s\n", >> + NIPQUAD(ic_servaddr), >> + NIPQUAD(root_server_addr), root_server_path); >> #endif /* !SILENT */ >> >> return 0; > > It should really be done in userspace. And ripped from the kernel. The output is useful when debugging boot problems on systems whose rootfs is on the network. So I think the patch is ok. However, it would be useful to make the parameters available to userspace for use by boot scripts etc. A proc file listing variables and values in /bin/sh syntax would be easy to use. I've been meaning to do this for ages so I'll roll a patch. -- James Chapman Katalix Systems Ltd http://www.katalix.com Catalysts for your Embedded Linux software development - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists