lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Sun, 14 Jun 2009 21:51:47 +0800
From:	Harald Welte <HaraldWelte@...tech.com>
To:	Pierre Ossman <pierre@...man.eu>
Cc:	Linux Kernel Mailinglist <linux-kernel@...r.kernel.org>,
	JosephChan@....com.tw, Bruce Chang <BruceChang@....com.tw>
Subject: Re: [PATCH] mmc: Add new via-sdmmc host controller driver

On Sun, Jun 14, 2009 at 12:55:37PM +0200, Pierre Ossman wrote:

> > Also, you stated that mmc_remove_host needs to be the first step.  However,
> > sdhci first kills the ongoing rquest (if "dead"), and then removes the host.
> 
> Right, because the reason you need to end the request is that failure
> to do so will wedge the MMC system and mmc_remove_host() will never
> return.
> 
> Normally the requests will eventually be completed by interrupts coming
> from the hardware, but if it got removed then that might not happen.

Ok, I will copy+paste the snippet from sdhci.c and do some testing, both
removing the card while continuously issuing requests, as well as removing the
driver while doing that.

Will submit the result again in two days or so.

-- 
- Harald Welte <HaraldWelte@...tech.com>	    http://linux.via.com.tw/
============================================================================
VIA Free and Open Source Software Liaison
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ