diff mbox series

iommu/iova: Simplify IOVA cache allocation with KMEM_CACHE()

Message ID 20240227090807.82318-1-liucong2@kylinos.cn
State New
Headers
Series iommu/iova: Simplify IOVA cache allocation with KMEM_CACHE() |

Commit Message

Cong Liu Feb. 27, 2024, 9:08 a.m. UTC 
  Use the new KMEM_CACHE() macro instead of direct kmem_cache_create
to simplify the creation of SLAB caches.

Signed-off-by: Cong Liu <liucong2@kylinos.cn>
---
 drivers/iommu/iova.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

Comments

Robin Murphy Feb. 27, 2024, 12:28 p.m. UTC | #1 
On 27/02/2024 9:08 am, Cong Liu wrote:
> Use the new KMEM_CACHE() macro instead of direct kmem_cache_create
> to simplify the creation of SLAB caches.

Heh, this "new" macro has existed for more than half the lifetime of 
Linux itself ;) ...and given that users are still outnumbered at least 
5:1 by users of kmem_cache_alloc(), I think it's fair to say that it 
hasn't really caught on all that well.

Most critically, though, as I mentioned on the previous thread, this 
would change the userspace-visible names - where I think the "iommu_" 
namespace is helpful in itself - and so impact anyone who's already 
using meminfo to track IOVA memory consumption, so I don't think such 
disruption is worthwhile just to save a mere 3 lines of code.

Thanks,
Robin.

> Signed-off-by: Cong Liu <liucong2@kylinos.cn>
> ---
>   drivers/iommu/iova.c | 7 ++-----
>   1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/iommu/iova.c b/drivers/iommu/iova.c
> index d59d0ea2fd21..9134acae76f5 100644
> --- a/drivers/iommu/iova.c
> +++ b/drivers/iommu/iova.c
> @@ -950,14 +950,11 @@ int iova_cache_get(void)
>   
>   	mutex_lock(&iova_cache_mutex);
>   	if (!iova_cache_users) {
> -		iova_cache = kmem_cache_create("iommu_iova", sizeof(struct iova), 0,
> -					       SLAB_HWCACHE_ALIGN, NULL);
> +		iova_cache = KMEM_CACHE(iova, SLAB_HWCACHE_ALIGN);
>   		if (!iova_cache)
>   			goto out_err;
>   
> -		iova_magazine_cache = kmem_cache_create("iommu_iova_magazine",
> -							sizeof(struct iova_magazine),
> -							0, SLAB_HWCACHE_ALIGN, NULL);
> +		iova_magazine_cache = KMEM_CACHE(iova_magazine, SLAB_HWCACHE_ALIGN);
>   		if (!iova_magazine_cache)
>   			goto out_err;
>
diff mbox series

Patch

diff --git a/drivers/iommu/iova.c b/drivers/iommu/iova.c
index d59d0ea2fd21..9134acae76f5 100644
--- a/drivers/iommu/iova.c
+++ b/drivers/iommu/iova.c
@@ -950,14 +950,11 @@  int iova_cache_get(void)
 
 	mutex_lock(&iova_cache_mutex);
 	if (!iova_cache_users) {
-		iova_cache = kmem_cache_create("iommu_iova", sizeof(struct iova), 0,
-					       SLAB_HWCACHE_ALIGN, NULL);
+		iova_cache = KMEM_CACHE(iova, SLAB_HWCACHE_ALIGN);
 		if (!iova_cache)
 			goto out_err;
 
-		iova_magazine_cache = kmem_cache_create("iommu_iova_magazine",
-							sizeof(struct iova_magazine),
-							0, SLAB_HWCACHE_ALIGN, NULL);
+		iova_magazine_cache = KMEM_CACHE(iova_magazine, SLAB_HWCACHE_ALIGN);
 		if (!iova_magazine_cache)
 			goto out_err;