Re: [patch] aio: streamline read events after woken up

2007-01-02 Thread Zach Brown
buffer index there. By then, most of you would probably veto the patch anyway ;-) haha, touche :) I still think it'd be the right thing, though. We can let the patch speak for itself :). - z - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a

RE: [patch] aio: streamline read events after woken up

2007-01-02 Thread Chen, Kenneth W
Zach Brown wrote on Tuesday, January 02, 2007 5:06 PM > To: Chen, Kenneth W > > Given the previous patch "aio: add per task aio wait event condition" > > that we properly wake up event waiting process knowing that we have > > enough events to reap, it's just plain waste of time to insert itself >

Re: [patch] aio: streamline read events after woken up

2007-01-02 Thread Zach Brown
Given the previous patch "aio: add per task aio wait event condition" that we properly wake up event waiting process knowing that we have enough events to reap, it's just plain waste of time to insert itself into a wait queue, and then immediately remove itself from the wait queue for *every*

Re: [patch] aio: streamline read events after woken up

2007-01-02 Thread Zach Brown
Given the previous patch aio: add per task aio wait event condition that we properly wake up event waiting process knowing that we have enough events to reap, it's just plain waste of time to insert itself into a wait queue, and then immediately remove itself from the wait queue for *every* event

RE: [patch] aio: streamline read events after woken up

2007-01-02 Thread Chen, Kenneth W
Zach Brown wrote on Tuesday, January 02, 2007 5:06 PM To: Chen, Kenneth W Given the previous patch aio: add per task aio wait event condition that we properly wake up event waiting process knowing that we have enough events to reap, it's just plain waste of time to insert itself into a

Re: [patch] aio: streamline read events after woken up

2007-01-02 Thread Zach Brown
buffer index there. By then, most of you would probably veto the patch anyway ;-) haha, touche :) I still think it'd be the right thing, though. We can let the patch speak for itself :). - z - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a

[patch] aio: streamline read events after woken up

2006-12-29 Thread Chen, Kenneth W
The read event loop in the blocking path is also inefficient. For every event it reap (if not blocking), it does the following in a loop: while (i < nr) { prepare_to_wait_exclusive aio_read_evt finish_wait ... } Given the previous patch

[patch] aio: streamline read events after woken up

2006-12-29 Thread Chen, Kenneth W
The read event loop in the blocking path is also inefficient. For every event it reap (if not blocking), it does the following in a loop: while (i nr) { prepare_to_wait_exclusive aio_read_evt finish_wait ... } Given the previous patch