Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751522AbdF1OrW (ORCPT ); Wed, 28 Jun 2017 10:47:22 -0400 Received: from resqmta-ch2-02v.sys.comcast.net ([69.252.207.34]:35434 "EHLO resqmta-ch2-02v.sys.comcast.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751469AbdF1OrR (ORCPT ); Wed, 28 Jun 2017 10:47:17 -0400 Date: Wed, 28 Jun 2017 09:47:13 -0500 (CDT) From: Christoph Lameter X-X-Sender: cl@east.gentwo.org To: Tejun Heo cc: Steven Rostedt , Andrew Morton , Pekka Enberg , LKML , Linus Torvalds , Vladimir Davydov , David Rientjes , Joonsoo Kim Subject: Re: [PATCH] slub: make sysfs file removal asynchronous In-Reply-To: <20170620224814.GK21326@htj.duckdns.org> Message-ID: References: <20170616085507.3cc7d4b8@gandalf.local.home> <20170619203538.GN12062@htj.duckdns.org> <20170619172750.6890df32@gandalf.local.home> <20170620204512.GI21326@htj.duckdns.org> <20170620145814.9eb1b74feaf908c39e9c0de2@linux-foundation.org> <20170620220011.GJ21326@htj.duckdns.org> <20170620182205.6e0390d8@grimm.local.home> <20170620224814.GK21326@htj.duckdns.org> Content-Type: text/plain; charset=US-ASCII X-CMAE-Envelope: MS4wfC+GrBy2OsB0RS4zEDeLIXEUkQFS8RNDB2sz2tsP3wXYW3jzeN2oNrslHfRkNK2zL9b+0UjKZaeUdx/lFRpRWgqlqUHgpgCPY6JG9lFnGkYxQ2vChEdx 5XJiT+BMP46uE7BaZVI+uY6It8UV0usdBVswT8zKtMOEa0QOfpUD64GDt7RccKo6FWBR4IrmsFhU7UVBwKv8s3F+ijlWhNhBmUZpl0AdPrF4gAHWjk84hO/g 7Kaf6yHgsFRPXboqMoNJR9abEGx9wEusbhIesJ/oR+q0oENOdewkZVALqJnYnvcScD++SbkdvrgBIhuAuS/5XjyvBJF4ijLB1jXlkvKI6xpnLIyuF7mN8xTy KBnRlCQHLjQ90TLfnKiMoQFHT7garK/WjhClsBXy0w9//047ispreA+JaPsH+YTHAkFck480tStv6bm/DaIOTYQiT0261g== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 727 Lines: 16 On Tue, 20 Jun 2017, Tejun Heo wrote: > And we have to weight that against the possibility of breakage from > the backport, however low it may be, right? I'm not strongly > convinced either way on this one and AFAICS the slub sysfs files there > are mostly for debugging, so we'd be risking breakage in a way more > common path (kmem_cache destruction) to avoid unlikely deadlock with a > debug facility. I think -stable backports should be conservative and > justified as breaking things through -stable undermines the whole > thing. The sysfs files are mainly used for reporting (the "slabinfo" tool accesses these files f.e.) But given the high rate of breakage of sysfs related patches: Lets just skip stable for now.