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;
Date:   Tue, 30 May 2023 11:34:06 +0100
From:   Conor Dooley <conor.dooley@microchip.com>
To:     Catalin Marinas <catalin.marinas@arm.com>
CC:     Jisheng Zhang <jszhang@kernel.org>,
        Paul Walmsley <paul.walmsley@sifive.com>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        Albert Ou <aou@eecs.berkeley.edu>,
        <linux-riscv@lists.infradead.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 5/6] riscv: allow kmalloc() caches aligned to the
 smallest value
Message-ID: <20230530-gyration-handheld-ef4e44e89d61@wendy>
References: <20230526165958.908-1-jszhang@kernel.org>
 <20230526165958.908-6-jszhang@kernel.org>
 <20230529-fidelity-booted-0d4055d1f559@wendy>
 <ZHXJDevEVwUEoOq4@arm.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="yLd8GXUEZrWQsGZ1"
Content-Disposition: inline
In-Reply-To: <ZHXJDevEVwUEoOq4@arm.com>
Precedence: bulk

--yLd8GXUEZrWQsGZ1
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Tue, May 30, 2023 at 10:59:41AM +0100, Catalin Marinas wrote:
> On Mon, May 29, 2023 at 12:17:46PM +0100, Conor Dooley wrote:
> > On Sat, May 27, 2023 at 12:59:57AM +0800, Jisheng Zhang wrote:
> > > After this patch, a simple test of booting to a small buildroot rootfs
> > > on qemu shows:
> > >=20
> > > kmalloc-96           5041    5041     96  ...
> > > kmalloc-64           9606    9606     64  ...
> > > kmalloc-32           5128    5128     32  ...
> > > kmalloc-16           7682    7682     16  ...
> > > kmalloc-8           10246   10246      8  ...
> > >=20
> > > So we save about 1268KB memory. The saving will be much larger in nor=
mal
> > > OS env on real HW platforms.
> > >=20
> > > [1] Link: https://lore.kernel.org/linux-arm-kernel/20230524171904.396=
7031-1-catalin.marinas@arm.com/

While I think of it, Link: goes at the start of the line, the [1] should
go at the end (although I don't think you actually reference the link
anywhere in the text & it'll probably not be particularly relevant if a
subsequent revision of that patchset is applied.

> > >=20
> > > Signed-off-by: Jisheng Zhang <jszhang@kernel.org>
> >=20
> > Fails to build chief, with loads of:
> > linux/dma-mapping.h:546:19: error: redefinition of 'dma_get_cache_align=
ment'
> >=20
> > And for 32-bit there's also a rake of:
> > include/linux/slab.h:239:9: warning: 'ARCH_KMALLOC_MINALIGN' macro rede=
fined [-Wmacro-redefined]
> >=20
> > At the very least, reproducable with rv32_defconfig.
>=20
> Have you this it on top of the KMALLOC_MINALIGN preparation series?
>=20
> https://lore.kernel.org/r/20230524171904.3967031-1-catalin.marinas@arm.co=
m/

Oh, no. Thanks for pointing that out.
Our automation stuff only uses what is in riscv/{for-next,master,fixes}.
Unless my reading comprehension is particularly bad of late it was
non-obvious that this depended on something that had not yet been
applied - it sounded like your series had already been merged last year.
Apologies for the noise then on this patch, but please try to be more
clear about what the dependencies actually are Jisheng.

Cheers,
Conor.

--yLd8GXUEZrWQsGZ1
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZHXRHgAKCRB4tDGHoIJi
0m9uAP9AoI+JEAqv+9RR/YOu/MvDnbTegsXBk/n2SwQtBMLLUwEA8FMW1zWGymoE
agKX1i07+HwjlUVKA0zZWmUkNoUpXQU=
=N2cE
-----END PGP SIGNATURE-----

--yLd8GXUEZrWQsGZ1--