RE: [RESEND PATCH v4] mmc: fix async request mechanism for sequential read scenarios

2012-12-19 Thread Seungwon Jeon
On Wednesday, December 19, 2012, Konstantin Dorfman wrote: On 12/17/2012 02:26 PM, Seungwon Jeon wrote: Hi, Konstantin. I added comments more below. I've answered below. On Wednesday, December 12, 2012, Seungwon Jeon wrote: On Monday, December 10, 2012, Konstantin Dorfman wrote:

Re: [RESEND PATCH v4] mmc: fix async request mechanism for sequential read scenarios

2012-12-18 Thread Konstantin Dorfman
On 12/17/2012 02:26 PM, Seungwon Jeon wrote: Hi, Konstantin. I added comments more below. I've answered below. On Wednesday, December 12, 2012, Seungwon Jeon wrote: On Monday, December 10, 2012, Konstantin Dorfman wrote: When current request is running on the bus and if next request

Re: [RESEND PATCH v4] mmc: fix async request mechanism for sequential read scenarios

2012-12-18 Thread Chris Ball
Hi, On Tue, Dec 18 2012, Konstantin Dorfman wrote: @@ -1374,6 +1367,8 @@ static int mmc_blk_issue_rw_rq(struct mmc_queue *mq, struct request *rqc) mmc_queue_bounce_post(mq_rq); switch (status) { + case MMC_BLK_NEW_REQUEST: + BUG(); /* should

RE: [RESEND PATCH v4] mmc: fix async request mechanism for sequential read scenarios

2012-12-17 Thread Seungwon Jeon
Hi, Konstantin. I added comments more below. On Wednesday, December 12, 2012, Seungwon Jeon wrote: On Monday, December 10, 2012, Konstantin Dorfman wrote: When current request is running on the bus and if next request fetched by mmcqd is NULL, mmc context (mmcqd thread) gets blocked until

RE: [RESEND PATCH v4] mmc: fix async request mechanism for sequential read scenarios

2012-12-12 Thread Seungwon Jeon
On Monday, December 10, 2012, Konstantin Dorfman wrote: When current request is running on the bus and if next request fetched by mmcqd is NULL, mmc context (mmcqd thread) gets blocked until the current request completes. This means if new request comes in while the mmcqd thread is blocked,