Re: [PATCH] aio updates

From: Benjamin LaHaise (bcrl@redhat.com)
Date: Tue Oct 15 2002 - 14:57:38 EST


On Mon, Oct 14, 2002 at 06:17:33PM -0700, John Myers wrote:
> Please apply.
> # fix uninitialized variable causing incorrect timeout
> # add support for IO_CMD_NOOP
> # make sys_io_cancel(), not cancel method, initialize most of returned result
> # minor aio_cancel_all() optimization
> # fix a debug printk

I've applied most of this to my tree, except for the NOOP bits. My
concern is that the way you've implemented NOOP does not allow for all
possible return codes to be passed in due to the error checking the
iocb submit code performs on the inputs. It can also spuriously fail
if the filedescriptor field points to an fd that doesn't exist, which
could be somewhat unexpected (especially if it is initialized to 0 by
default, and therefore would not fail during normal operation where a
program is run with stdin). Got any idea for cleaning those problems
up?

                -ben

-
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 : Tue Oct 15 2002 - 22:00:57 EST