Re: [RFC] aio_down() for i386 and x86_64

[Suparna Bhattacharya]

On Wed, Jun 15, 2005 at 03:18:30PM -0400, Benjamin LaHaise wrote:
> On Wed, Jun 15, 2005 at 10:23:49PM +0530, Suparna Bhattacharya wrote:
> > Interesting approach - using ki_wait.private for this.
> > Could we make aio_down take a wait queue parameter as well instead of
> > the iocb ?
> 
> Hmmm, I guess there might be instances where someone has to wait on 
> multiple wait queues.  Will add that to the next version of the patch.
> 
> > Need to think a little about impact on io cancellation.
> 
> It should be possible to cancel semaphore operations fairly easily -- 
> the aio_down function can set ->ki_cancel to point to a semaphore cancel 
> routine.  I'll give coding that a try.
> 
> > BTW, is the duplication of functions across architectures still needed ? I
> > thought that one of advantages of implementing a separate aio_down
> > routine vs modifiying down to become retryable was to get away from
> > that ... or wasn't it ?
> 
> Good point.  The fast path for down() will probably need to remain a 
> separate function, but we could well unify the code with the 
> down_interruptible() codepath.
> 
> > Meanwhile, I probably need to repost my aio_wait_bit patches - there
> > may be some impact here.
> 
> Sure -- any version of those would be useful to build on.  Cheers!

http://www.kernel.org/pub/linux/kernel/people/suparna/aio/2610-rc2/ has
the patchset. 

I just updated the AIO wait bit ones to 2.6.12-rc6, will post them
in a separate thread.

Regards
Suparna

> 
> 		-ben
> --
> To unsubscribe, send a message with 'unsubscribe linux-aio' in
> the body to majordomo@xxxxxxxxxx  For more info on Linux AIO,
> see: http://www.kvack.org/aio/
> Don't email: <a href=mailto:"aart@xxxxxxxxx";>aart@xxxxxxxxx</a>

-- 
Suparna Bhattacharya (suparna@xxxxxxxxxx)
Linux Technology Center
IBM Software Lab, India

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