On 06/21/2013 10:18 AM, Nathan Zimmer wrote:Oh, The difference is just under 32k.I'm more concerned about bytes of code.Since you made it a compile time option, it would be good to know howThe patchset as a whole is just over 400 lines so it doesn't add alot.
much code it adds, but otherwise I agree with Greg here... this really
shouldn't need to be an option. It *especially* shouldn't need to be a
hand-set runtime option (which looks quite complex, to boot.)
If I were to pull the .config option it would probably remove 30 lines.
The command line option is too complex but some of the data I haven'tI think that is probably key.
found a way to get at runtime yet.
I am not surprised.I suspect the cutoff for this should be a lot lower than 8 TB even, moreEven at lower amounts of ram there is an positive impact.I it knocks
like 128 GB or so. The only concern is to not set the cutoff so low
that we can end up running out of memory or with suboptimal NUMA
placement just because of this.
time off
boot even at as small as a 1TB of ram.
-hpa