RE: [PATCH v3 2/2] mm: Introduce kernelcore=mirror option

From: Luck, Tony
Date: Fri Dec 18 2015 - 01:59:16 EST


>Hmm...like this ?
> sysctl.vm.fallback_mirror_memory = 0 // never fallback # default.
> sysctl.vm.fallback_mirror_memory = 1 // the user memory may be allocated from mirrored zone.
> sysctl.vm.fallback_mirror_memory = 2 // usually kernel allocates memory from mirrored zone before OOM.
> sysctl.vm.fallback_mirror_memory = 3 // 1+2

Should option 2 say: // allow kernel to allocate from non-mirror zone to avoid OOM

> However I believe my customer's choice is always 0, above implementation can be done in a clean way.
> (adding a flag to zones (mirrored or not) and controlling fallback zonelist walk.)

Modes allow us to make all of the people happy (I hope).

> BTW, we need this Taku's patch to make a progress. I think other devs should be done in another
> development cycle. What does he need to get your Acks ?

The concept is great. It's even "Tested-by: Tony Luck <tony.luck@xxxxxxxxx>".
I need to read the code more carefully before Acked-by.

-Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/