Messages in this thread | | | Date | Tue, 06 Aug 2002 12:43:17 +0200 | From | Marcin Dalecki <> | Subject | Re: [PATCH] 2.5.30 IDE 113 |
| |
Uz.ytkownik Jens Axboe napisa?: > On Tue, Aug 06 2002, Marcin Dalecki wrote: > >>device not per channel! If q->request_fn would properly return the >>error count instead of void, we could even get rid ot the >>checking for rq->errors after finishment... But well that's >>entierly different story. > > > That's nonsense! What exactly would you return from a request_fn after > having queued, eg, 20 commands? Error count is per request, anything > else would be stupid.
Returning the error count in the case q->request_fn is called for a self submitted request like for example REQ_SPECIAL would be handy and well defined. For the cumulative case it would of course make sense to return the cumulative error count. Tough not very meaningfull, it would indicate the occurrence of the error very fine.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |