Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
Date:   Wed, 09 Nov 2022 11:48:58 +0000
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
From:   "Yajun Deng" <yajun.deng@linux.dev>
Message-ID: <fccbe6b1bd533a84f50cd675b57ecb17@linux.dev>
Subject: Re: [6.1.0-rc3-next-20221104] Boot failure - kernel BUG at
 mm/memblock.c:519
To:     "Mike Rapoport" <rppt@linux.ibm.com>
Cc:     "Sachin Sant" <sachinp@linux.ibm.com>,
        "linuxppc-dev" <linuxppc-dev@lists.ozlabs.org>,
        "open list" <linux-kernel@vger.kernel.org>, linux-mm@kvack.org
In-Reply-To: <Y2uSH8jm1MzY9Rgf@linux.ibm.com>
References: <Y2uSH8jm1MzY9Rgf@linux.ibm.com>
 <74979021-D386-4402-BD70-167531C7405B@linux.ibm.com>
 <e00989c4a69943cb4f60fc7ffaa06f8c@linux.dev>
 <Y2oLYB7Tu7J91tVm@linux.ibm.com>
 <E2499567-0D0F-44DA-AC68-1E279009A6DE@linux.ibm.com>
 <58779468e28e026a1aa30a42ca7e8aec@linux.dev>
 <4fee7f2b0e99e256465ef6e7656c6349@linux.dev>
 <6e32f14ae25909138e40b2cfd7448ed8@linux.dev>
Precedence: bulk

November 9, 2022 7:42 PM, "Mike Rapoport" <rppt@linux.ibm.com> wrote:=0A=
=0A> Hi Yajun,=0A> =0A> On Wed, Nov 09, 2022 at 11:32:27AM +0000, Yajun D=
eng wrote:=0A> =0A>> November 9, 2022 6:55 PM, "Sachin Sant" <sachinp@lin=
ux.ibm.com> wrote:=0A>> =0A>> On 09-Nov-2022, at 3:55 PM, Yajun Deng <yaj=
un.deng@linux.dev> wrote:=0A>> =0A>> November 9, 2022 6:03 PM, "Yajun Den=
g" <yajun.deng@linux.dev> wrote:=0A>> =0A>> Hey Mike,=0A>> =0A>> Sorry, t=
his email should be sent to Sachin but not Mike.=0A>> Please forgive my c=
onfusion. So:=0A>> =0A>> Hey Sachin,=0A>> Can you help me test the attach=
ed file?=0A>> Please use this new patch instead of the one in memblock tr=
ee.=0A>> =0A>> Thanks for the fix. With the updated patch kernel boots co=
rrectly.=0A>> =0A>> Thanks for your test results.=0A>> =0A>> Hi Mike,=0A>=
> Do you have any other suggestions for this patch? If not, I'll send a v=
3 patch.=0A> =0A> Unfortunately I don't think the new version has much va=
lue as it does not=0A> really eliminate the second loop in case memory al=
location is required.=0A> I'd say the improvement is not worth the churn.=
=0A> =0AOK, I got it.=0A=0A>> Tested-by: Sachin Sant <sachinp@linux.ibm.c=
om <sachinp@linux.ibm.com>>=0A>> =0A>> - Sachin=0A> =0A> --=0A> Sincerely=
 yours,=0A> Mike.