[BUG] wbsd.c: after spin_lock_bh uses spin_unlock instead spin_unlock_bh

From: Alexander Strakh
Date: Thu Oct 08 2009 - 07:16:49 EST


KERNEL_VERSION: 2.6.31
DESCRIBE:
Driver ./drivers/mmc/host/wbsd.c calls spin_lock_bh and then spin_unlock
instead of spin_unlock_bh:

753 static void wbsd_request(struct mmc_host *mmc, struct mmc_request *mrq)
...
761 spin_lock_bh(&host->lock);
...
844 done:
845 wbsd_request_end(host, mrq);
846
847 spin_unlock_bh(&host->lock);
848 }

But in wsdb_request calls spin_unlock/spin_lock instead of
spin_unlock_bh/spin_lock_bh;

206 static void wbsd_request_end(struct wbsd_host *host, struct mmc_request
...
230 spin_unlock(&host->lock);
231 mmc_request_done(host->mmc, mrq);
232 spin_lock(&host->lock);
233 }

Found by : Linux Driver Verification
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/