Re: drivers/mmc/wbsd.c

[Pierre Ossman]

Andrew Morton wrote:

> static inline void wbsd_kunmap_sg(struct wbsd_host* host)
> {
> 	kunmap_atomic(host->cur_sg->page, KM_BIO_SRC_IRQ);
> }
>
> Guys, kunmap_atomic() takes a kernel virtual address (the value which
> kmap_atomic() returned).
>
> Passing it the address of a pageframe will have unpleasant results.
>  
Thanks. kunmap_atomic() just messes with the preemption stuff (on x86 at 
least) so it probably would have gone unnoticed for a while.
I've fixed it now and it will be included in the next patch set.

Rgds
Pierre

-
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/