From: Matthew Wilcox <willy@infradead.org>
Date: 2020-04-17 11:23:54 To: Bernard Zhao <bernard@vivo.com> Cc: Christoph Lameter <cl@linux.com>,Pekka Enberg <penberg@kernel.org>,David Rientjes <rientjes@google.com>,Joonsoo Kim <iamjoonsoo.kim@lge.com>,Andrew Morton <akpm@linux-foundation.org>,linux-mm@kvack.org,linux-kernel@vger.kernel.org,kernel@vivo.com Subject: Re: [PATCH] kmalloc_index optimization(code size & runtime stable)>On Thu, Apr 16, 2020 at 07:03:30PM -0700, Bernard Zhao wrote: >> kmalloc_index inline function code size optimization and runtime >> performance stability optimization. After optimization, the function >> kmalloc_index is more stable, the size will never affecte the function`s >> execution efficiency. >> And follow test data shows that the performance of new optimization >> exceeds the original algorithm when applying for more than 512 Bytes >> (include 512B).And new optimization runtime is more stable than before. > >That's all very well and good, but the vast majority of allocations >are less than 512 bytes in size! Your numbers show that on average, >this patch makes the kernel slower!>This is indeed the case, the new algorithm is stable at a time level, butthere is a certain performance loss for relatively small memory(little than 512).I will continue to pay attention to this part later. Thanks.>> size time/Per 100 million times.us >> old fun new fun with optimise >> 8 203777 241934 >> 16 245611 409278 >> 32 236384 408419 >> 64 275499 447732 >> 128 354909 416439 >> 256 360472 406598 >> 512 431072 409168 >> 1024 463822 407401 >