~RE: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done

2015-08-30 Thread Jialing Fu
t; Like add one line as below? > Fixes: 2220eedfd7ae ("mmc: fix async request mechanism for sequential > read scenarios") > That's it, Jialing. From my git blame, seems this bug has been introduced for a long time, but I feel strange that no one had captured it before you

RE: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done

2015-08-28 Thread Jialing Fu
[...] >>> Hi, ulf >>> >>> We find this bug on Intel-C3230RK platform for very small probability. >>> >>> Whereas I can easily reproduce this case if I add a mdelay(1) or >>> longer delay as Jialing did. >>> >>> This patch seems useful to me. Should we push it forward? :) >> >> >> It seems like

[PATCH] mmc: core: fix race condition in mmc_wait_data_done

2015-01-12 Thread Jialing Fu
ke_up_interruptible(&mrq->host->context_info.wait); } At runtime, IRQ or ICache line missing may just happen at the same place of the mdelay(1). This patch gets the mmc_context_info at the beginning of function, it can avoid this race condition. Signed-off-by: Jialing Fu --- driv