Re: [PATCH] 2.5.30 IDE 113

From: Marcin Dalecki (dalecki@evision.ag)
Date: Tue Aug 06 2002 - 05:20:40 EST


Uz.ytkownik Petr Vandrovec napisa?:

> Hi Marcin,
> what synchronizes these accesses to make sure that you do not have
> two ide_raw_taskfile requests on the flight, both using same
> drive->srequest? It looks to me like that nothing, so you can overwrite
> request's contents while somebody else already uses this buffer.

I don't think so. The queue lock is synchronizing them.
And then we usually add them just to the front of the queue in question
and wait for finishment until the request is done.
After all ide_raw_taskfile only gets used for REQ_SPECIAL request
types. This does *not* contain normal data request from block IO.
As of master slave issues - well we have the data pre allocated per
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.

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



This archive was generated by hypermail 2b29 : Wed Aug 07 2002 - 22:00:30 EST