From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751713AbeCTBn3 (ORCPT ); Mon, 19 Mar 2018 21:43:29 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:45294 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751650AbeCTBnX (ORCPT ); Mon, 19 Mar 2018 21:43:23 -0400 Date: Tue, 20 Mar 2018 09:43:18 +0800 From: Baoquan He To: AKASHI Takahiro , Dave Young Cc: herbert@gondor.apana.org.au, ard.biesheuvel@linaro.org, catalin.marinas@arm.com, julien.thierry@arm.com, will.deacon@arm.com, linux-kernel@vger.kernel.org, kexec@lists.infradead.org, dhowells@redhat.com, arnd@arndb.de, linux-arm-kernel@lists.infradead.org, mpe@ellerman.id.au, bauerman@linux.vnet.ibm.com, akpm@linux-foundation.org, Linus Torvalds , davem@davemloft.net, vgoyal@redhat.com Subject: Re: [PATCH v8 01/13] resource: add walk_system_ram_res_rev() Message-ID: <20180320014318.GA2514@localhost.localdomain> References: <20180222111732.23051-1-takahiro.akashi@linaro.org> <20180222111732.23051-2-takahiro.akashi@linaro.org> <20180223083619.GA4433@dhcp-128-65.nay.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180223083619.GA4433@dhcp-128-65.nay.redhat.com> User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/23/18 at 04:36pm, Dave Young wrote: > Hi AKASHI, > > On 02/22/18 at 08:17pm, AKASHI Takahiro wrote: > > This function, being a variant of walk_system_ram_res() introduced in > > commit 8c86e70acead ("resource: provide new functions to walk through > > resources"), walks through a list of all the resources of System RAM > > in reversed order, i.e., from higher to lower. > > > > It will be used in kexec_file implementation on arm64. > > I remember there was an old discussion about this, it should be added > in patch log why this is needed. It's used to load kernel/initrd at the top of system RAM, and this is consistent with user space kexec behaviour. In x86 64, Vivek didn't do like this since there's no reverse iomem resource iterating function, he just chose a match RAM region bottom up, then put kernel/initrd top down in the found RAM region. This is different than kexec_tools utility. I am considering to change resource sibling as double list, seems AKASHI's way is easier to be accepted by people. So I will use this one to change x86 64 code. Hi AKASHI, About arm64 kexec_file patches, will you post recently? Or any other plan? Thanks Baoquan > > > > > Signed-off-by: AKASHI Takahiro > > Cc: Vivek Goyal > > Cc: Andrew Morton > > Cc: Linus Torvalds > > --- > > include/linux/ioport.h | 3 +++ > > kernel/resource.c | 57 ++++++++++++++++++++++++++++++++++++++++++++++++++ > > 2 files changed, 60 insertions(+) > > > > diff --git a/include/linux/ioport.h b/include/linux/ioport.h > > index da0ebaec25f0..f12d95fe038b 100644 > > --- a/include/linux/ioport.h > > +++ b/include/linux/ioport.h > > @@ -277,6 +277,9 @@ extern int > > walk_system_ram_res(u64 start, u64 end, void *arg, > > int (*func)(struct resource *, void *)); > > extern int > > +walk_system_ram_res_rev(u64 start, u64 end, void *arg, > > + int (*func)(struct resource *, void *)); > > +extern int > > walk_iomem_res_desc(unsigned long desc, unsigned long flags, u64 start, u64 end, > > void *arg, int (*func)(struct resource *, void *)); > > > > diff --git a/kernel/resource.c b/kernel/resource.c > > index e270b5048988..bdaa93407f4c 100644 > > --- a/kernel/resource.c > > +++ b/kernel/resource.c > > @@ -23,6 +23,8 @@ > > #include > > #include > > #include > > +#include > > +#include > > #include > > > > > > @@ -486,6 +488,61 @@ int walk_mem_res(u64 start, u64 end, void *arg, > > arg, func); > > } > > > > +int walk_system_ram_res_rev(u64 start, u64 end, void *arg, > > + int (*func)(struct resource *, void *)) > > +{ > > + struct resource res, *rams; > > + int rams_size = 16, i; > > + int ret = -1; > > + > > + /* create a list */ > > + rams = vmalloc(sizeof(struct resource) * rams_size); > > + if (!rams) > > + return ret; > > + > > + res.start = start; > > + res.end = end; > > + res.flags = IORESOURCE_SYSTEM_RAM | IORESOURCE_BUSY; > > + i = 0; > > + while ((res.start < res.end) && > > + (!find_next_iomem_res(&res, IORES_DESC_NONE, true))) { > > + if (i >= rams_size) { > > + /* re-alloc */ > > + struct resource *rams_new; > > + int rams_new_size; > > + > > + rams_new_size = rams_size + 16; > > + rams_new = vmalloc(sizeof(struct resource) > > + * rams_new_size); > > + if (!rams_new) > > + goto out; > > + > > + memcpy(rams_new, rams, > > + sizeof(struct resource) * rams_size); > > + vfree(rams); > > + rams = rams_new; > > + rams_size = rams_new_size; > > + } > > + > > + rams[i].start = res.start; > > + rams[i++].end = res.end; > > + > > + res.start = res.end + 1; > > + res.end = end; > > + } > > + > > + /* go reverse */ > > + for (i--; i >= 0; i--) { > > + ret = (*func)(&rams[i], arg); > > + if (ret) > > + break; > > + } > > + > > +out: > > + vfree(rams); > > + return ret; > > +} > > + > > #if !defined(CONFIG_ARCH_HAS_WALK_MEMORY) > > > > /* > > -- > > 2.16.2 > > > > Thanks > Dave > > _______________________________________________ > kexec mailing list > kexec@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/kexec