Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 12 Aug 2020 18:11:25 +1000
From:   Nicholas Piggin <npiggin@gmail.com>
Subject: Re: [PATCH v3 0/8] huge vmalloc mappings
To:     Jonathan Cameron <Jonathan.Cameron@huawei.com>,
        Zefan Li <lizefan@huawei.com>
Cc:     Borislav Petkov <bp@alien8.de>,
        =?iso-8859-1?q?Catalin=0A?= Marinas <catalin.marinas@arm.com>,
        "H. Peter Anvin" <hpa@zytor.com>, linux-arch@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-mm@kvack.org, linuxppc-dev@lists.ozlabs.org,
        Ingo Molnar <mingo@redhat.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Will Deacon <will@kernel.org>, x86@kernel.org
References: <20200810022732.1150009-1-npiggin@gmail.com>
        <20200811173217.0000161e@huawei.com>
        <d457aabc-9f58-f47e-f5fa-9539618b2759@huawei.com>
In-Reply-To: <d457aabc-9f58-f47e-f5fa-9539618b2759@huawei.com>
MIME-Version: 1.0
Message-Id: <1597219299.b5noer1k93.astroid@bobo.none>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Excerpts from Zefan Li's message of August 12, 2020 11:07 am:
> On 2020/8/12 0:32, Jonathan Cameron wrote:
>> On Mon, 10 Aug 2020 12:27:24 +1000
>> Nicholas Piggin <npiggin@gmail.com> wrote:
>>=20
>>> Not tested on x86 or arm64, would appreciate a quick test there so I ca=
n
>>> ask Andrew to put it in -mm. Other option is I can disable huge vmalloc=
s
>>> for them for the time being.
>>=20
>> Hi Nicholas,
>>=20
>> For arm64 testing with a Kunpeng920.
>>=20
>> I ran a quick sanity test with this series on top of mainline (yes mid m=
erge window
>> so who knows what state is...).  Could I be missing some dependency?
>>=20
>> Without them it boots, with them it doesn't.  Any immediate guesses?
>>=20
>=20
> I've already reported this bug in v2, and yeah I also tested it on arm64
> (not Kunpeng though), so looks like it still hasn't been fixed.

Huh, I thought I did fix it but seems not. vmap stacks shouldn't be=20
big enough to use huge pages though, so I don't know what's going on
there. I'll dig around a bit more.

>=20
> ...
>>>
>>> Since v2:
>>> - Rebased on vmalloc cleanups, split series into simpler pieces.
>>> - Fixed several compile errors and warnings
>>> - Keep the page array and accounting in small page units because
>>>   struct vm_struct is an interface (this should fix x86 vmap stack debu=
g
>>>   assert). [Thanks Zefan]
>=20
> though the changelog says it's fixed for x86.

Yes, my mistake that was supposed to say arm64.

Thanks,
Nick