On Thu, 2014-01-09 at 13:23 -0500, Vivek Goyal wrote: > On Thu, Jan 09, 2014 at 10:24:25AM -0700, Toshi Kani wrote: > > [..] > > > I think creating a new command line option is simpler as compared to > > > creating a new flag in bootparam which in turn disables memory hotplug. > > > More users can use that option. For example, if for some reason hotplug > > > code is crashing, one can just disable it on command line as work around > > > and move on. > > > > I do not have a strong opinion about having such option. However, I > > think it is more user friendly to keep the exactmap option works alone > > on any platforms. > > I think we should create internally a variable which will disable memory > hotplug. And set that variable based on memmap=exactmap, mem=X and also > provide a way to disable memory hotplug directly using command line > option. > > Current kexec-tools can use memmap=exactmap and be happy. I am writing > a new kexec syscall and will not be using memmap=exactmap and would need > to use that command line option to disable memory hotplug behavior. Sounds good to me. Thanks, -Toshi
