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

2015-12-28 Thread Andrew Morton
On Wed, 9 Dec 2015 12:19:37 +0900 Taku Izumi wrote: > This patch extends existing "kernelcore" option and > introduces kernelcore=mirror option. By specifying > "mirror" instead of specifying the amount of memory, > non-mirrored (non-reliable) region will be arranged > into ZONE_MOVABLE. > > v1

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

2015-12-28 Thread Andrew Morton
On Wed, 9 Dec 2015 12:19:37 +0900 Taku Izumi wrote: > This patch extends existing "kernelcore" option and > introduces kernelcore=mirror option. By specifying > "mirror" instead of specifying the amount of memory, > non-mirrored (non-reliable) region will be arranged > into ZONE_MOVABLE. > > v1

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

2015-12-17 Thread Luck, Tony
>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 z

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

2015-12-17 Thread Kamezawa Hiroyuki
On 2015/12/18 3:43, Luck, Tony wrote: As Tony requested, we may need a knob to stop a fallback in "movable->normal", later. If the mirrored memory is small and the other is large, I think we can both enable "non-mirrored -> normal" and "normal -> non-mirrored". Size of mirrored memory can

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

2015-12-17 Thread Luck, Tony
>>> As Tony requested, we may need a knob to stop a fallback in >>> "movable->normal", later. >>> >> >> If the mirrored memory is small and the other is large, >> I think we can both enable "non-mirrored -> normal" and "normal -> >> non-mirrored". > > Size of mirrored memory can be configured by

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

2015-12-16 Thread Kamezawa Hiroyuki
..@kvack.org; >>>>> a...@linux-foundation.org; Kamezawa, Hiroyuki/亀澤 寛 >>>>> 之; m...@csn.ul.ie; Hansen, Dave; m...@codeblueprint.co.uk >>>>> Subject: Re: [PATCH v3 2/2] mm: Introduce kernelcore=mirror option >>>>> >>>&

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

2015-12-16 Thread Xishi Qiu
shi Qiu [mailto:qiuxi...@huawei.com] >>>> Sent: Friday, December 11, 2015 6:44 PM >>>> To: Izumi, Taku/泉 拓 >>>> Cc: Luck, Tony; linux-kernel@vger.kernel.org; linux...@kvack.org; >>>> a...@linux-foundation.org; Kamezawa, Hiroyuki/亀澤 寛 >>>

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

2015-12-16 Thread Xishi Qiu
vger.kernel.org; linux...@kvack.org; >> a...@linux-foundation.org; Kamezawa, Hiroyuki/亀澤 寛 >> 之; m...@csn.ul.ie; Hansen, Dave; m...@codeblueprint.co.uk >> Subject: Re: [PATCH v3 2/2] mm: Introduce kernelcore=mirror option >> >> On 2015/12/11 13:53, Izumi, Taku wrote: &g

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

2015-12-16 Thread Kamezawa Hiroyuki
6:44 PM >>> To: Izumi, Taku/泉 拓 >>> Cc: Luck, Tony; linux-kernel@vger.kernel.org; linux...@kvack.org; >>> a...@linux-foundation.org; Kamezawa, Hiroyuki/亀澤 寛 >>> 之; m...@csn.ul.ie; Hansen, Dave; m...@codeblueprint.co.uk >>> Subject: Re: [PATCH v3 2/2] mm:

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

2015-12-16 Thread Izumi, Taku
Hiroyuki/亀澤 寛 > 之; m...@csn.ul.ie; Hansen, Dave; m...@codeblueprint.co.uk > Subject: Re: [PATCH v3 2/2] mm: Introduce kernelcore=mirror option > > On 2015/12/11 13:53, Izumi, Taku wrote: > > > Dear Xishi, > > > >> Hi Taku, > >> > >> Whether it

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

2015-12-11 Thread Xishi Qiu
On 2015/12/11 13:53, Izumi, Taku wrote: > Dear Xishi, > >> Hi Taku, >> >> Whether it is possible that we rewrite the fallback function in buddy system >> when zone_movable and mirrored_kernelcore are both enabled? > > What does "when zone_movable and mirrored_kernelcore are both enabled?" > m

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

2015-12-10 Thread Izumi, Taku
Dear Xishi, > Hi Taku, > > Whether it is possible that we rewrite the fallback function in buddy system > when zone_movable and mirrored_kernelcore are both enabled? What does "when zone_movable and mirrored_kernelcore are both enabled?" mean ? My patchset just provides a new way to creat

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

2015-12-09 Thread Xishi Qiu
On 2015/12/10 13:37, Izumi, Taku wrote: > Dear Tony, Xishi, > How about add some comment, if mirrored memroy is too small, then the normal zone is small, so it may be oom. The mirrored memory is at least 1/64 of whole memory, because struct pages usually take 64 bytes per page

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

2015-12-09 Thread Izumi, Taku
Dear Tony, Xishi, > >> How about add some comment, if mirrored memroy is too small, then the > >> normal zone is small, so it may be oom. > >> The mirrored memory is at least 1/64 of whole memory, because struct > >> pages usually take 64 bytes per page. > > > > 1/64th is the absolute lower bound

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

2015-12-09 Thread Xishi Qiu
On 2015/12/10 5:59, Luck, Tony wrote: >> How about add some comment, if mirrored memroy is too small, then the >> normal zone is small, so it may be oom. >> The mirrored memory is at least 1/64 of whole memory, because struct >> pages usually take 64 bytes per page. > > 1/64th is the absolute low

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

2015-12-09 Thread Luck, Tony
> How about add some comment, if mirrored memroy is too small, then the > normal zone is small, so it may be oom. > The mirrored memory is at least 1/64 of whole memory, because struct > pages usually take 64 bytes per page. 1/64th is the absolute lower bound (for the page structures as you say).

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

2015-12-08 Thread Xishi Qiu
On 2015/12/9 11:19, Taku Izumi wrote: > This patch extends existing "kernelcore" option and > introduces kernelcore=mirror option. By specifying > "mirror" instead of specifying the amount of memory, > non-mirrored (non-reliable) region will be arranged > into ZONE_MOVABLE. > > v1 -> v2: > - Ref

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

2015-12-08 Thread Taku Izumi
This patch extends existing "kernelcore" option and introduces kernelcore=mirror option. By specifying "mirror" instead of specifying the amount of memory, non-mirrored (non-reliable) region will be arranged into ZONE_MOVABLE. v1 -> v2: - Refine so that the following case also can be handled p