Andrew Morton wrote: > On Wed, 17 Jan 2007 10:55:54 +0100 > Sébastien Dugué <[email protected]> wrote: > >> +struct lio_event *lio_create(struct sigevent __user *user_event, >> + int mode) >> +{ >> + int ret = 0; >> + struct lio_event *lio = NULL; >> + >> + if (unlikely((mode == LIO_NOWAIT) && !user_event)) >> + return lio; >> + >> + lio = kzalloc(sizeof(*lio), GFP_KERNEL); >> + >> + if (!lio) >> + return ERR_PTR(-EAGAIN); >> + > > Why EAGAIN and not ENOMEM? According to the POSIX AIO specifications: "ERRORS: ... [EAGAIN] The resources necessary to queue all the I/O requests were not available. " I think memory is "a resource necessary to queue all the I/O requests"... http://www.opengroup.org/onlinepubs/009695399/functions/lio_listio.html Regards, Laurent -- ------------- [email protected] -------------- "Any sufficiently advanced technology is indistinguishable from magic." - Arthur C. Clarke
Attachment:
signature.asc
Description: OpenPGP digital signature
- Follow-Ups:
- Re: [PATCH -mm 5/5][AIO] - Add listio syscall support
- From: Sébastien Dugué <[email protected]>
- Re: [PATCH -mm 5/5][AIO] - Add listio syscall support
- References:
- [PATCH -mm 5/5][AIO] - Add listio syscall support
- From: Sébastien Dugué <[email protected]>
- Re: [PATCH -mm 5/5][AIO] - Add listio syscall support
- From: Andrew Morton <[email protected]>
- [PATCH -mm 5/5][AIO] - Add listio syscall support
- Prev by Date: IO-APIC + timer doesn't work - 2.6.20-rc5 on abit KN9-Ultra bios 1.6
- Next by Date: Re: [patch 00/46] High resolution timer / dynamic tick update
- Previous by thread: Re: [PATCH -mm 5/5][AIO] - Add listio syscall support
- Next by thread: Re: [PATCH -mm 5/5][AIO] - Add listio syscall support
- Index(es):
 |