From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from aserp1040.oracle.com (aserp1040.oracle.com [141.146.126.69]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 912F81A0745 for ; Thu, 12 Mar 2015 23:52:38 +1100 (AEDT) Message-ID: <55018BBC.6010903@oracle.com> Date: Thu, 12 Mar 2015 08:51:08 -0400 From: Sasha Levin MIME-Version: 1.0 To: Pekka Enberg , linux-kernel@vger.kernel.org Subject: Re: [PATCH] mm: kill kmemcheck References: <1426074547-21888-1-git-send-email-sasha.levin@oracle.com> <55013B38.6040100@iki.fi> In-Reply-To: <55013B38.6040100@iki.fi> Content-Type: text/plain; charset=windows-1252 Cc: Vladimir Davydov , Geert Uytterhoeven , SUPERH , Catalin Marinas , Will Deacon , Jianyu Zhan , Paul Mackerras , Pavel Machek , Miklos Szeredi , Christoph Lameter , Marek Szyprowski , "open list:GENERIC INCLUDE/A..." , Andi Kleen , Russell King , Jingoo Han , James Morris , Chris Bainbridge , Antti Palosaari , Mel Gorman , Ritesh Harjani , Shaohua Li , Alexander Duyck , Wang Nan , Jiri Pirko , Daniel J Blueman , Rusty Russell , Jens Axboe , Alexander Viro , Thomas Gleixner , Konstantin Khlebnikov , Willem de Bruijn , Greg Kroah-Hartman , Randy Dunlap , "Rafael J. Wysocki" , Rabin Vincent , "Aneesh Kumar K.V" , Jiri Kosina , Tejun Heo , Andrew Morton , Patrick McHardy , Christoffer Dall , Mark Rutland , DOCUMENTATION , Dave Hansen , Eric Dumazet , Jungseung Lee , Masami Hiramatsu , Stephen Rothwell , Qiaowei Ren , Prarit Bhargava , bob picco , Sungjinn Chung , Quentin Lambert , Hugh Dickins , Yasuaki Ishimatsu , Kukjin Kim , Pranith Kumar , Tom Herbert , CRYPTO API , Rik van Riel , Johannes Berg , Mauro Carvalho Chehab , Eyal Perry , Joe Perches , chishanmingshen , Cyrill Gorcunov , Boris Ostrovsky , Yinghai Lu , Steven Honeyman , ARM PORT , Theodore Ts'o , Pekka Enberg , David Vrabel , David Rientjes , Joonsoo Kim , Michal Hocko , Paul Gortmaker , Jan Beulich , "moderated list:OPENRISC ARCHITEC..." , Guan Xuetao , Sam Ravnborg , Jonathan Corbet , Andrey Ryabinin , Dexuan Cui , Vegard Nossum , Alexey Kuznetsov , "Paul E. McKenney" , Fenghua Yu , Kees Cook , Arnd Bergmann , Suresh Siddha , Josh Triplett , Steven Rostedt , Borislav Petkov , Andrey Konovalov , Scott Wood , Bryan O'Donoghue , Dan Streetman , Oleg Nesterov , Andy Lutomirski , Hannes Frederic Sowa , linux-fsdevel@vger.kernel.org, "David S. Miller" , "Kirill A. Shutemov" , "Michael S. Tsirkin" , Peter Zijlstra , MEMORY MANAGEMENT , Davidlohr Bueso , Paul Cassella , "H. Peter Anvin" , "open list:SPARC + UltraSPAR..." , Xishi Qiu , Alexei Starovoitov , Jonas Bonn , Herbert Xu , Richard Weinberger , "maintainer:X86 ARCHITECTURE..." , Ingo Molnar , Aaron Tomlin , Greg Thelen , "Eric W. Biederman" , Konrad Rzeszutek Wilk , Fabian Frederick , Michal Marek , Jungseok Lee , Dave Jones , Andy Shevchenko , Vlastimil Babka , Juergen Gross , Hideaki YOSHIFUJI , "open list:NETWORKING \[GENERAL\]" , Daniel Borkmann , Horia Geanta , Johannes Weiner , "open list:LINUX FOR POWERPC..." , Anton Blanchard List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 03/12/2015 03:07 AM, Pekka Enberg wrote: > Hi Sasha, > > On 3/11/15 1:43 PM, Sasha Levin wrote: >> As discussed on LSF/MM, kill kmemcheck. >> >> KASan is a replacement that is able to work without the limitation of >> kmemcheck (single CPU, slow). KASan is already upstream. >> >> We are also not aware of any users of kmemcheck (or users who don't consider >> KASan as a suitable replacement). >> >> I've build tested it using all[yes,no,mod]config and fuzzed a bit with this >> patch applied, didn't notice any bad behaviour. >> >> Signed-off-by: Sasha Levin > > Can you elaborate on what exactly was discussed at LSF/MM? Preferably in the commit log. ;-) There wasn't a long discussion about removing kmemcheck, it just followed up a KASan topic and the question of whether kmemcheck can be deprecated now that KASan is merged came up. No one at the room didn't use it, knew a user of it, or couldn't describe a usecase where kmemcheck was superior to KASan - so the conclusion was to try and replace it. The only thing I can really put in the changelog is a reference to the KASan docs and to ask folks to complain loudly if we missed a usecase. Thanks, Sasha