linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Point the UNEVICTABLE_LRU config option at the documentation
@ 2009-03-16 10:59 David Howells
  2009-03-16 12:02 ` Christoph Hellwig
  0 siblings, 1 reply; 5+ messages in thread
From: David Howells @ 2009-03-16 10:59 UTC (permalink / raw)
  To: lee.schermerhorn, kosaki.motohiro, minchan.kim
  Cc: dhowells, akpm, torvalds, linux-kernel, linux-mm

Point the UNEVICTABLE_LRU config option at the documentation describing the
option.

Signed-off-by: David Howells <dhowells@redhat.com>
---

 mm/Kconfig |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)


diff --git a/mm/Kconfig b/mm/Kconfig
index b53427a..57971d2 100644
--- a/mm/Kconfig
+++ b/mm/Kconfig
@@ -213,6 +213,8 @@ config UNEVICTABLE_LRU
 	  will use one page flag and increase the code size a little,
 	  say Y unless you know what you are doing.
 
+	  See Documentation/vm/unevictable-lru.txt for more information.
+
 config HAVE_MLOCK
 	bool
 	default y if MMU=y

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Point the UNEVICTABLE_LRU config option at the documentation
  2009-03-16 10:59 [PATCH] Point the UNEVICTABLE_LRU config option at the documentation David Howells
@ 2009-03-16 12:02 ` Christoph Hellwig
  2009-03-16 12:22   ` KOSAKI Motohiro
  0 siblings, 1 reply; 5+ messages in thread
From: Christoph Hellwig @ 2009-03-16 12:02 UTC (permalink / raw)
  To: David Howells
  Cc: lee.schermerhorn, kosaki.motohiro, minchan.kim, akpm, torvalds,
	linux-kernel, linux-mm

On Mon, Mar 16, 2009 at 10:59:45AM +0000, David Howells wrote:
> Point the UNEVICTABLE_LRU config option at the documentation describing the
> option.

Didn't we decide a while ago that the option is pointless and the code
should always be enabled?

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Point the UNEVICTABLE_LRU config option at the documentation
  2009-03-16 12:02 ` Christoph Hellwig
@ 2009-03-16 12:22   ` KOSAKI Motohiro
  2009-03-16 12:55     ` Minchan Kim
  0 siblings, 1 reply; 5+ messages in thread
From: KOSAKI Motohiro @ 2009-03-16 12:22 UTC (permalink / raw)
  To: Christoph Hellwig
  Cc: kosaki.motohiro, David Howells, lee.schermerhorn, minchan.kim,
	akpm, torvalds, linux-kernel, linux-mm

> On Mon, Mar 16, 2009 at 10:59:45AM +0000, David Howells wrote:
> > Point the UNEVICTABLE_LRU config option at the documentation describing the
> > option.
> 
> Didn't we decide a while ago that the option is pointless and the code
> should always be enabled?

Yeah.
CONFIG_UNEVICTABLE_LRU lost existing reason by David's good patch recently.

if nobody of nommu user post bug report in .30 age, I plan to remove
this config option at .31 age.

his patch is really really good job.



--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Point the UNEVICTABLE_LRU config option at the documentation
  2009-03-16 12:22   ` KOSAKI Motohiro
@ 2009-03-16 12:55     ` Minchan Kim
  2009-03-16 13:14       ` KOSAKI Motohiro
  0 siblings, 1 reply; 5+ messages in thread
From: Minchan Kim @ 2009-03-16 12:55 UTC (permalink / raw)
  To: KOSAKI Motohiro
  Cc: Christoph Hellwig, David Howells, lee.schermerhorn, akpm,
	torvalds, linux-kernel, linux-mm

size vmlinux
   text	   data	    bss	    dec	    hex	filename
6232681	 747665	 708608	7688954	 7552fa	vmlinux

size vmlinux.unevictable
   text	   data	    bss	    dec	    hex	filename
6239404	 747985	 708608	7695997	 756e7d	vmlinux.unevictable

It almost increases about 7K.
Many embedded guys always have a concern about size although it is very small.
It's important about embedded but may not be about server.

In addition, CONFIG_UNEVICTABLE_LRU feature don't have a big impact in
embedded machines which have a very small ram.
I guess many embedded guys will not use this feature.

So, I don't want to remove this configurable option.
Lets not add useless size bloat in embedded system.


On Mon, Mar 16, 2009 at 9:22 PM, KOSAKI Motohiro
<kosaki.motohiro@jp.fujitsu.com> wrote:
>> On Mon, Mar 16, 2009 at 10:59:45AM +0000, David Howells wrote:
>> > Point the UNEVICTABLE_LRU config option at the documentation describing the
>> > option.
>>
>> Didn't we decide a while ago that the option is pointless and the code
>> should always be enabled?
>
> Yeah.
> CONFIG_UNEVICTABLE_LRU lost existing reason by David's good patch recently.
>
> if nobody of nommu user post bug report in .30 age, I plan to remove
> this config option at .31 age.
>
> his patch is really really good job.
>
>
>
>



-- 
Kinds regards,
Minchan Kim

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Point the UNEVICTABLE_LRU config option at the documentation
  2009-03-16 12:55     ` Minchan Kim
@ 2009-03-16 13:14       ` KOSAKI Motohiro
  0 siblings, 0 replies; 5+ messages in thread
From: KOSAKI Motohiro @ 2009-03-16 13:14 UTC (permalink / raw)
  To: Minchan Kim
  Cc: Christoph Hellwig, David Howells, lee.schermerhorn, akpm,
	torvalds, linux-kernel, linux-mm

2009/3/16 Minchan Kim <minchan.kim@gmail.com>:
> size vmlinux
>   text    data     bss     dec     hex filename
> 6232681  747665  708608 7688954  7552fa vmlinux
>
> size vmlinux.unevictable
>   text    data     bss     dec     hex filename
> 6239404  747985  708608 7695997  756e7d vmlinux.unevictable
>
> It almost increases about 7K.
> Many embedded guys always have a concern about size although it is very small.
> It's important about embedded but may not be about server.

Thanks good report.
this is unintetional size to me. I'll digg it later.

Thanks!


-- kosaki


>
> In addition, CONFIG_UNEVICTABLE_LRU feature don't have a big impact in
> embedded machines which have a very small ram.
> I guess many embedded guys will not use this feature.
>
> So, I don't want to remove this configurable option.
> Lets not add useless size bloat in embedded system.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2009-03-16 13:14 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-03-16 10:59 [PATCH] Point the UNEVICTABLE_LRU config option at the documentation David Howells
2009-03-16 12:02 ` Christoph Hellwig
2009-03-16 12:22   ` KOSAKI Motohiro
2009-03-16 12:55     ` Minchan Kim
2009-03-16 13:14       ` KOSAKI Motohiro

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox