Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967023AbbLQPJH (ORCPT ); Thu, 17 Dec 2015 10:09:07 -0500 Received: from protonic.xs4all.nl ([83.163.252.89]:23546 "EHLO protonic.xs4all.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755687AbbLQPJF (ORCPT ); Thu, 17 Dec 2015 10:09:05 -0500 Date: Thu, 17 Dec 2015 16:09:03 +0100 From: David Jander To: Ulf Hansson Cc: Lucas Stach , linux-mmc , Pierre Ossman , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" Subject: Re: SDHCI long sleep with interrupts off Message-ID: <20151217160903.556af1af@archvile> In-Reply-To: References: <20151217112814.6250a3b9@archvile> <1450350190.3163.93.camel@pengutronix.de> <20151217122053.036daf37@archvile> <1450351655.3163.97.camel@pengutronix.de> <20151217132229.1c699f69@archvile> Organization: Protonic Holland X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.27; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2708 Lines: 68 Dear Ulf, On Thu, 17 Dec 2015 15:54:54 +0100 Ulf Hansson wrote: > [...] > > >> If/when you decide to fix this issue. Please keep in mind the following > >> things. > >> > >> - Try to convert the SDHCI into a pure library. No more quirks or > >> callbacks. > >> - I assume we can simplify lots of code if we convert SDHCI into using > >> a threaded IRQ in favour of the tasklet. > >> > >> Any patches that moves SDHCI into this direction will be greatly > >> appreciated! > > > > Ok, this sounds like a good way to go. Unfortunately it also sounds like a > > major endeavor, for which good knowledge of the SDHCI standard is > > necessary. This knowledge is based on documentation that is not openly > > available without cost AFAIK. This probably also explains why there hasn't > > been a real fix ever. On top of that, the whole sdhci code is unmaintained > > currently as it seems. I was studying the code a bit more, and I now > > understand that I am not even close to having the experience and > > standards-knowledge it takes to pull this off reliably. I guess the one > > who takes on this task may as well become official maintainer afterwards... > > You are right, a maintainer is needed for sdhci. > > Also, I am a bit surprised that none have stepped up, especially since > it's indeed being *very* widely used. So, you probably understand my surprise as for the state of things. I am only casually walking by because I have a latency problem.... > > OTOH, we pretty much depend on this driver now, since all of our new > > i.MX6/7 boards have eMMC flash. We also use the flexcan peripheral on all > > designs, which is specially sensible to these latency spikes, so we will > > have to do something on the long run.... we cannot live forever with > > disabled PM ;-) > > > > Unfortunate, PM is only one of the problems. I already had that suspicion while looking at the code... > The code is in general fragile. We have have kind of reached the > point, when I apply changes that fixes one issue it may cause another. Oh, that is indeed bad. I wish I was in the position to do this... but this really goes beyond my time and my knowledge. I think most of the effort will be at cleaning up the mess and make sure that each one of the many users works well afterwards, and it definitely takes someone who knows the code (and it's users) very well to pull this off. Best regards, -- David Jander Protonic Holland. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/