Messages in this thread | | | Date | Thu, 13 Feb 2014 16:37:53 -0800 | Subject | Re: [RFC PATCH V5] mm readahead: Fix readahead fail for no local memory and limit readahead pages | From | Linus Torvalds <> |
| |
Is this whole thread still just for the crazy and pointless "max_sane_readahead()"?
Or is there some *real* reason we should care?
Because if it really is just for max_sane_readahead(), then for the love of God, let us just do this
unsigned long max_sane_readahead(unsigned long nr) { return min(nr, 128); }
and bury this whole idiotic thread.
Seriously, if your IO subsystem needs more than 512kB of read-ahead to get full performance, your IO subsystem is just bad, and has latencies that are long enough that you should just replace it. There's no real reason to bend over backwards for that, and the whole complexity and fragility of the insane "let's try to figure out how much memory this node has" is just not worth it. The read-ahead should be small enough that we should never need to care, and large enough that you get reasonable IO throughput. The above does that.
Goddammit, there's a reason the whole "Gordian knot" parable is famous. We're making this all too damn complicated FOR NO GOOD REASON.
Just cut the rope, people. Our aim is not to generate some kind of job security by making things as complicated as possible.
Linus
On Thu, Feb 13, 2014 at 4:14 PM, Nishanth Aravamudan <nacc@linux.vnet.ibm.com> wrote: > > I'm working on this latter bit now. I tried to mirror ia64, but it looks > like they have CONFIG_USER_PERCPU_NUMA_NODE_ID, which powerpc doesn't. > It seems like CONFIG_USER_PERCPU_NUMA_NODE_ID and > CONFIG_HAVE_MEMORYLESS_NODES should be tied together in Kconfig? > > I'll keep working, but would appreciate any further insight. > > -Nish >
| |