On Thu, 30 Jan 2014, Andrew Morton wrote: > > Yeah, it shouldn't be temporary it should be the one and only allocation. > > We should construct the name in memcg_create_kmem_cache() and be done with > > it. > > Could. That would require converting memcg_create_kmem_cache() to take > a va_list and call kasprintf() on it. > Why? We already construct the name in memcg_create_kmem_cache() appropriately, we just want to avoid the kstrdup() in kmem_cache_create_memcg() since it's pointless like my patch does. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>