Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Vlastimil Babka <vbabka@xxxxxxx>
Subject: Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
From: Hyeonggon Yoo <42.hyeyoo@xxxxxxxxx>
Date: Thu, 13 May 2021 21:10:11 +0900
Cc: Marco Elver <elver@xxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, iamjoonsoo.kim@xxxxxxx, rientjes@xxxxxxxxxx, penberg@xxxxxxxxxx, cl@xxxxxxxxx, linux-mm@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20210513120829.GB772931@hyeyoo>
References: <20210511173448.GA54466@hyeyoo> <20210512195227.245000695c9014242e9a00e5@linux-foundation.org> <20210513031220.GA133011@hyeyoo> <20210512204024.401ff3de38649d7d0f5a45e8@linux-foundation.org> <20210513062809.GA319973@hyeyoo> <a36ab9a1-f07a-42ca-bb11-5bd0c70660bb@suse.cz> <YJ0ACtMpasnoZdUp@elver.google.com> <90c1b2ae-a5c1-07d1-b056-5b3699b6cc77@suse.cz> <20210513120829.GB772931@hyeyoo>

On Thu, May 13, 2021 at 09:08:29PM +0900, Hyeonggon Yoo wrote:
> I completely agree on what Vlastimil said. there should be comment
> saying that generally you should not use kmalloc_index with
> size_is_const == true.

sorry there was human error.
what I mean was: size_is_const == false

References:
- [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Hyeonggon Yoo
- Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Andrew Morton
- Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Hyeonggon Yoo
- Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Andrew Morton
- Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Hyeonggon Yoo
- Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Vlastimil Babka
- Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Marco Elver
- Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Vlastimil Babka
- Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
  - From: Hyeonggon Yoo

Prev by Date: Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
Next by Date: [PATCH] mm/lru: Add folio LRU functions
Previous by thread: Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
Next by thread: Re: [PATCH v3] mm, slub: change run-time assertion in kmalloc_index() to compile-time
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]