From: Catalin Marinas <catalin.marinas@arm.com> To: Isaac Manjarres <isaacmanjarres@google.com> Cc: Linus Torvalds <torvalds@linux-foundation.org>, Arnd Bergmann <arnd@arndb.de>, Christoph Hellwig <hch@lst.de>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Will Deacon <will@kernel.org>, Marc Zyngier <maz@kernel.org>, Andrew Morton <akpm@linux-foundation.org>, Herbert Xu <herbert@gondor.apana.org.au>, Ard Biesheuvel <ardb@kernel.org>, Saravana Kannan <saravanak@google.com>, Alasdair Kergon <agk@redhat.com>, Daniel Vetter <daniel@ffwll.ch>, Joerg Roedel <joro@8bytes.org>, Mark Brown <broonie@kernel.org>, Mike Snitzer <snitzer@kernel.org>, "Rafael J. Wysocki" <rafael@kernel.org>, Robin Murphy <robin.murphy@arm.com>, linux-mm@kvack.org, iommu@lists.linux.dev, linux-arm-kernel@lists.infradead.org, Petr Tesarik <petrtesarik@huaweicloud.com> Subject: Re: [PATCH v3 00/13] mm, dma, arm64: Reduce ARCH_KMALLOC_MINALIGN to 8 Date: Wed, 19 Apr 2023 17:06:04 +0100 [thread overview] Message-ID: <ZEARbKMfMS8KdkhJ@arm.com> (raw) In-Reply-To: <ZBNiN9CR2F/fvUTh@google.com> On Thu, Mar 16, 2023 at 11:38:47AM -0700, Isaac Manjarres wrote: > On Sun, Nov 06, 2022 at 10:01:30PM +0000, Catalin Marinas wrote: > > Patches 7-12 change some ARCH_KMALLOC_MINALIGN uses to > > ARCH_DMA_MINALIGN. The crypto changes have been rejected by Herbert > > previously but I still included them here until the crypto code is > > refactored. > > Herbert merged the changes to the crypto code that were required to be > able to safely lower the minimum alignment for kmalloc in [1]. Yes, I saw this. > Given this, I don't think there's anything blocking this series from > being merged. The requirement for SWIOTLB to get to the minimum > kmalloc alignment down to 8 bytes shouldn't prevent this series from > being merged, as the amount of memory that is allocated for SWIOTLB > can be configured through the commandline to minimize the impact of > having SWIOTLB memory. Additionally, even if no SWIOTLB is present, > this series still offers memory savings on a lot of ARM64 platforms > by using the cache line size as the minimum alignment for kmalloc. Actually, there's some progress on the swiotlb front to allow dynamic allocation. I haven't reviewed the series yet (I wasn't aware of it until v2) but at a quick look, it limits the dynamic allocation to bouncing buffers of at least a page size. Maybe this can be later improved for buffers below ARCH_DMA_MINALIGN. https://lore.kernel.org/r/cover.1681898595.git.petr.tesarik.ext@huawei.com > Can you please rebase this series so that it can be merged? I rebased it locally but the last stumbling block is sorting out the iommu bouncing. I was hoping Robin Murphy can lend a hand but he's been busy with other bits. I'll repost the series at 6.4-rc1. -- Catalin
WARNING: multiple messages have this Message-ID (diff)
From: Catalin Marinas <catalin.marinas@arm.com> To: Isaac Manjarres <isaacmanjarres@google.com> Cc: Linus Torvalds <torvalds@linux-foundation.org>, Arnd Bergmann <arnd@arndb.de>, Christoph Hellwig <hch@lst.de>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Will Deacon <will@kernel.org>, Marc Zyngier <maz@kernel.org>, Andrew Morton <akpm@linux-foundation.org>, Herbert Xu <herbert@gondor.apana.org.au>, Ard Biesheuvel <ardb@kernel.org>, Saravana Kannan <saravanak@google.com>, Alasdair Kergon <agk@redhat.com>, Daniel Vetter <daniel@ffwll.ch>, Joerg Roedel <joro@8bytes.org>, Mark Brown <broonie@kernel.org>, Mike Snitzer <snitzer@kernel.org>, "Rafael J. Wysocki" <rafael@kernel.org>, Robin Murphy <robin.murphy@arm.com>, linux-mm@kvack.org, iommu@lists.linux.dev, linux-arm-kernel@lists.infradead.org, Petr Tesarik <petrtesarik@huaweicloud.com> Subject: Re: [PATCH v3 00/13] mm, dma, arm64: Reduce ARCH_KMALLOC_MINALIGN to 8 Date: Wed, 19 Apr 2023 17:06:04 +0100 [thread overview] Message-ID: <ZEARbKMfMS8KdkhJ@arm.com> (raw) In-Reply-To: <ZBNiN9CR2F/fvUTh@google.com> On Thu, Mar 16, 2023 at 11:38:47AM -0700, Isaac Manjarres wrote: > On Sun, Nov 06, 2022 at 10:01:30PM +0000, Catalin Marinas wrote: > > Patches 7-12 change some ARCH_KMALLOC_MINALIGN uses to > > ARCH_DMA_MINALIGN. The crypto changes have been rejected by Herbert > > previously but I still included them here until the crypto code is > > refactored. > > Herbert merged the changes to the crypto code that were required to be > able to safely lower the minimum alignment for kmalloc in [1]. Yes, I saw this. > Given this, I don't think there's anything blocking this series from > being merged. The requirement for SWIOTLB to get to the minimum > kmalloc alignment down to 8 bytes shouldn't prevent this series from > being merged, as the amount of memory that is allocated for SWIOTLB > can be configured through the commandline to minimize the impact of > having SWIOTLB memory. Additionally, even if no SWIOTLB is present, > this series still offers memory savings on a lot of ARM64 platforms > by using the cache line size as the minimum alignment for kmalloc. Actually, there's some progress on the swiotlb front to allow dynamic allocation. I haven't reviewed the series yet (I wasn't aware of it until v2) but at a quick look, it limits the dynamic allocation to bouncing buffers of at least a page size. Maybe this can be later improved for buffers below ARCH_DMA_MINALIGN. https://lore.kernel.org/r/cover.1681898595.git.petr.tesarik.ext@huawei.com > Can you please rebase this series so that it can be merged? I rebased it locally but the last stumbling block is sorting out the iommu bouncing. I was hoping Robin Murphy can lend a hand but he's been busy with other bits. I'll repost the series at 6.4-rc1. -- Catalin _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-04-19 16:06 UTC|newest] Thread overview: 90+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-11-06 22:01 [PATCH v3 00/13] mm, dma, arm64: Reduce ARCH_KMALLOC_MINALIGN to 8 Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 01/13] mm/slab: Decouple ARCH_KMALLOC_MINALIGN from ARCH_DMA_MINALIGN Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 02/13] dma-mapping: Force bouncing if the kmalloc() size is not cacheline-aligned Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-07 9:43 ` Christoph Hellwig 2022-11-07 9:43 ` Christoph Hellwig 2022-11-06 22:01 ` [PATCH v3 03/13] iommu/dma: Force bouncing of the " Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-07 9:46 ` Christoph Hellwig 2022-11-07 9:46 ` Christoph Hellwig 2022-11-07 10:54 ` Catalin Marinas 2022-11-07 10:54 ` Catalin Marinas 2022-11-07 13:26 ` Robin Murphy 2022-11-07 13:26 ` Robin Murphy 2022-11-08 10:51 ` Catalin Marinas 2022-11-08 10:51 ` Catalin Marinas 2022-11-08 11:40 ` Robin Murphy 2022-11-08 11:40 ` Robin Murphy 2022-11-08 7:50 ` Christoph Hellwig 2022-11-08 7:50 ` Christoph Hellwig 2022-11-14 23:23 ` Isaac Manjarres 2022-11-14 23:23 ` Isaac Manjarres 2022-11-15 11:48 ` Catalin Marinas 2022-11-15 11:48 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 04/13] mm/slab: Allow kmalloc() minimum alignment fallback to dma_get_cache_alignment() Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-07 0:50 ` kernel test robot 2022-11-07 9:22 ` Catalin Marinas 2022-11-07 9:22 ` Catalin Marinas 2022-11-07 1:51 ` kernel test robot 2022-11-06 22:01 ` [PATCH v3 05/13] mm/slab: Simplify create_kmalloc_cache() args and make it static Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 06/13] dma: Allow the smaller cache_line_size() returned by dma_get_cache_alignment() Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 07/13] drivers/base: Use ARCH_DMA_MINALIGN instead of ARCH_KMALLOC_MINALIGN Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 08/13] drivers/gpu: " Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 09/13] drivers/usb: " Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 10/13] drivers/spi: " Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-07 12:58 ` Mark Brown 2022-11-07 12:58 ` Mark Brown 2022-11-06 22:01 ` [PATCH v3 11/13] crypto: " Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-07 2:22 ` Herbert Xu 2022-11-07 2:22 ` Herbert Xu 2022-11-07 9:05 ` Catalin Marinas 2022-11-07 9:05 ` Catalin Marinas 2022-11-07 9:12 ` Herbert Xu 2022-11-07 9:12 ` Herbert Xu 2022-11-07 9:38 ` Catalin Marinas 2022-11-07 9:38 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 12/13] drivers/md: " Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-06 22:01 ` [PATCH v3 13/13] dma: arm64: Add CONFIG_DMA_BOUNCE_UNALIGNED_KMALLOC and enable it for arm64 Catalin Marinas 2022-11-06 22:01 ` Catalin Marinas 2022-11-07 13:03 ` Robin Murphy 2022-11-07 13:03 ` Robin Murphy 2022-11-07 14:38 ` Christoph Hellwig 2022-11-07 14:38 ` Christoph Hellwig 2022-11-07 15:24 ` Robin Murphy 2022-11-07 15:24 ` Robin Murphy 2022-11-08 9:52 ` Catalin Marinas 2022-11-08 9:52 ` Catalin Marinas 2022-11-08 10:03 ` Christoph Hellwig 2022-11-08 10:03 ` Christoph Hellwig 2022-11-30 18:48 ` Isaac Manjarres 2022-11-30 18:48 ` Isaac Manjarres 2022-11-30 23:32 ` Alexander Graf 2022-11-30 23:32 ` Alexander Graf 2023-04-20 11:51 ` Petr Tesařík 2023-04-20 11:51 ` Petr Tesařík 2023-03-16 18:38 ` [PATCH v3 00/13] mm, dma, arm64: Reduce ARCH_KMALLOC_MINALIGN to 8 Isaac Manjarres 2023-03-16 18:38 ` Isaac Manjarres 2023-04-19 16:06 ` Catalin Marinas [this message] 2023-04-19 16:06 ` Catalin Marinas 2023-04-20 9:52 ` Petr Tesarik 2023-04-20 9:52 ` Petr Tesarik 2023-04-20 17:43 ` Catalin Marinas 2023-04-20 17:43 ` Catalin Marinas 2023-05-15 19:09 ` Isaac Manjarres 2023-05-15 19:09 ` Isaac Manjarres 2023-05-16 17:19 ` Catalin Marinas 2023-05-16 17:19 ` Catalin Marinas 2023-05-16 18:19 ` Isaac Manjarres 2023-05-16 18:19 ` Isaac Manjarres
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=ZEARbKMfMS8KdkhJ@arm.com \ --to=catalin.marinas@arm.com \ --cc=agk@redhat.com \ --cc=akpm@linux-foundation.org \ --cc=ardb@kernel.org \ --cc=arnd@arndb.de \ --cc=broonie@kernel.org \ --cc=daniel@ffwll.ch \ --cc=gregkh@linuxfoundation.org \ --cc=hch@lst.de \ --cc=herbert@gondor.apana.org.au \ --cc=iommu@lists.linux.dev \ --cc=isaacmanjarres@google.com \ --cc=joro@8bytes.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-mm@kvack.org \ --cc=maz@kernel.org \ --cc=petrtesarik@huaweicloud.com \ --cc=rafael@kernel.org \ --cc=robin.murphy@arm.com \ --cc=saravanak@google.com \ --cc=snitzer@kernel.org \ --cc=torvalds@linux-foundation.org \ --cc=will@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.