diff mbox

IPv6: Update documentation for better built-in help with params

Message ID 4B5A0991.9080701@hp.com
State Rejected, archived
Delegated to: David Miller
Headers show

Commit Message

Brian Haley Jan. 22, 2010, 8:24 p.m. UTC
Update IPv6 documentation to describe how to set parameters when it's
built-into the kernel.  Reported by <greg@enjellic.com>.

Signed-off-by: Brian Haley <brian.haley@hp.com>
---

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

David Miller Jan. 23, 2010, 2:32 a.m. UTC | #1
From: Brian Haley <brian.haley@hp.com>
Date: Fri, 22 Jan 2010 15:24:49 -0500

> Update IPv6 documentation to describe how to set parameters when it's
> built-into the kernel.  Reported by <greg@enjellic.com>.
> 
> Signed-off-by: Brian Haley <brian.haley@hp.com>

So are you going to also add a mention of this fact to every single
place in the documentation that talks about some arbitrary module
option?

That doesn't make any sense, and neither does this patch.

It's better to make it more apparent that module options can
be set on the kernel command line when something is compiled
statically into the kernel.

That way it only needs to be explained in one place, not in
several.

I'm not applying this.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Brian Haley Jan. 25, 2010, 5:28 p.m. UTC | #2
David Miller wrote:
> From: Brian Haley <brian.haley@hp.com>
> Date: Fri, 22 Jan 2010 15:24:49 -0500
> 
>> Update IPv6 documentation to describe how to set parameters when it's
>> built-into the kernel.  Reported by <greg@enjellic.com>.
>>
>> Signed-off-by: Brian Haley <brian.haley@hp.com>
> 
> So are you going to also add a mention of this fact to every single
> place in the documentation that talks about some arbitrary module
> option?
> 
> That doesn't make any sense, and neither does this patch.
> 
> It's better to make it more apparent that module options can
> be set on the kernel command line when something is compiled
> statically into the kernel.
> 
> That way it only needs to be explained in one place, not in
> several.

I guess Documentation/kernel-parameters.txt is where such things
live, can I send you a patch for that or does it need to go
through someone else?

-Brian
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Miller Jan. 25, 2010, 9:21 p.m. UTC | #3
From: Brian Haley <brian.haley@hp.com>
Date: Mon, 25 Jan 2010 12:28:15 -0500

> I guess Documentation/kernel-parameters.txt is where such things
> live, can I send you a patch for that or does it need to go
> through someone else?

I think lkml is a suitable place to submit something like
that.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Randy.Dunlap Jan. 25, 2010, 9:31 p.m. UTC | #4
On Mon, 25 Jan 2010 13:21:42 -0800 (PST) David Miller wrote:

> From: Brian Haley <brian.haley@hp.com>
> Date: Mon, 25 Jan 2010 12:28:15 -0500
> 
> > I guess Documentation/kernel-parameters.txt is where such things
> > live, can I send you a patch for that or does it need to go
> > through someone else?
> 
> I think lkml is a suitable place to submit something like
> that.
> --

Sure, but if it's networking parameters, you (David) could still merge it...

---
~Randy
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Miller Jan. 25, 2010, 9:34 p.m. UTC | #5
From: Randy Dunlap <rdunlap@xenotime.net>
Date: Mon, 25 Jan 2010 13:31:33 -0800

> On Mon, 25 Jan 2010 13:21:42 -0800 (PST) David Miller wrote:
> 
>> From: Brian Haley <brian.haley@hp.com>
>> Date: Mon, 25 Jan 2010 12:28:15 -0500
>> 
>> > I guess Documentation/kernel-parameters.txt is where such things
>> > live, can I send you a patch for that or does it need to go
>> > through someone else?
>> 
>> I think lkml is a suitable place to submit something like
>> that.
>> --
> 
> Sure, but if it's networking parameters, you (David) could still merge it...

It's not about networking parameters.

It's about adding a generic mention somewhere central that you can set
"module parameters" on the kernel command line when a module is built
statically into the kernel.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Randy.Dunlap Jan. 25, 2010, 9:43 p.m. UTC | #6
On Mon, 25 Jan 2010 13:34:11 -0800 (PST) David Miller wrote:

> From: Randy Dunlap <rdunlap@xenotime.net>
> Date: Mon, 25 Jan 2010 13:31:33 -0800
> 
> > On Mon, 25 Jan 2010 13:21:42 -0800 (PST) David Miller wrote:
> > 
> >> From: Brian Haley <brian.haley@hp.com>
> >> Date: Mon, 25 Jan 2010 12:28:15 -0500
> >> 
> >> > I guess Documentation/kernel-parameters.txt is where such things
> >> > live, can I send you a patch for that or does it need to go
> >> > through someone else?
> >> 
> >> I think lkml is a suitable place to submit something like
> >> that.
> >> --
> > 
> > Sure, but if it's networking parameters, you (David) could still merge it...
> 
> It's not about networking parameters.

OK, I didn't see the patch itself, sorry.


> It's about adding a generic mention somewhere central that you can set
> "module parameters" on the kernel command line when a module is built
> statically into the kernel.

Maybe like this, from that same file:

Module parameters for modules that are built into the kernel image
are specified on the kernel command line with the module name plus
'.' plus parameter name, with '=' and value if appropriate, such as:

	usbcore.blinkenlights=1



I'll search for the patch...

---
~Randy
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/networking/ipv6.txt b/Documentation/networking/ipv6.txt
index 9fd7e21..d52166b 100644
--- a/Documentation/networking/ipv6.txt
+++ b/Documentation/networking/ipv6.txt
@@ -6,6 +6,10 @@  or modprobe command, but are usually specified in either the
 /etc/modules.conf or /etc/modprobe.conf configuration file, or in a
 distro-specific configuration file.
 
+For cases where IPv6 is built-into the kernel, you can set these
+parameters by specifying them on the kernel command line, for
+example, "ipv6.autoconf=0".
+
 The available ipv6 module parameters are listed below.  If a parameter
 is not specified the default value is used.