From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754874AbbCSP4A (ORCPT ); Thu, 19 Mar 2015 11:56:00 -0400 Received: from mail-vc0-f173.google.com ([209.85.220.173]:50976 "EHLO mail-vc0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752261AbbCSPz5 (ORCPT ); Thu, 19 Mar 2015 11:55:57 -0400 Date: Thu, 19 Mar 2015 11:55:53 -0400 From: Tejun Heo To: Borislav Petkov Cc: Dmitry Torokhov , Doug Thompson , linux-kernel@vger.kernel.org, linux-edac@vger.kernel.org, Mauro Carvalho Chehab , Tetsuo Handa , Olof Johansson , Arjan van de Ven , Greg Kroah-Hartman , "Luis R . Rodriguez" Subject: Re: [PATCH 3/3] EDAC: amd64_edac: decide if driver can load successfully early. Message-ID: <20150319155553.GK25365@htj.duckdns.org> References: <1426726150-983-1-git-send-email-dmitry.torokhov@gmail.com> <1426726150-983-4-git-send-email-dmitry.torokhov@gmail.com> <20150319094054.GC10860@pd.tnic> <20150319152957.GI25365@htj.duckdns.org> <20150319153505.GD11715@pd.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150319153505.GD11715@pd.tnic> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 19, 2015 at 04:35:06PM +0100, Borislav Petkov wrote: > And I don't want to leave the driver loaded when there's nothing to > be loaded for. One instance in this driver's specific case is one > northbridge and there are numascale boxes with hundreds of northbridges. > > If you have a better idea about how to unload the driver, asynchronously > or not, after all probe() calls have failed, I'm all ears. We don't go around and implement random hacks ignoring layering and conventions even if that one off case seems to benefit whatever corner case minutely, because those kind of hacks accumulate and hinder with improvements at much larger scale and the benefit here is minute. The driver model has been moving onto separating module load and probing because that makes far more sense to most drivers that we use nowadays and also gives userland a cleaner way to manage modules by separating the two operations - loading and probing. If you think this is a big enough problem, please go ahead and build proper infrastructure for it. Be it a module attribute or even just a flag telling userland to indicate that the module can be auto-unloaded. Sure, it might look like a smart thing from a very confined viewpoint but you're seeing the problem unfolding right now. This code is standing in the way of a much more impactful generic driver layer improvement. Thanks. -- tejun