From: John Gardiner Myers <jgmyers@netscape.com>
To: David Stevens <dlstevens@us.ibm.com>
Cc: Shailabh Nagar <nagar@watson.ibm.com>,
linux-kernel <linux-kernel@vger.kernel.org>,
linux-aio <linux-aio@kvack.org>, Andrew Morton <akpm@digeo.com>,
Ben LaHaise <bcrl@redhat.com>, David Miller <davem@redhat.com>,
Linus Torvalds <torvalds@transmeta.com>,
Stephen Tweedie <sct@redhat.com>
Subject: Re: [PATCH] Re: [PATCH] async poll for 2.5
Date: Tue, 15 Oct 2002 11:16:22 -0700 [thread overview]
Message-ID: <3DAC5B76.1050101@netscape.com> (raw)
In-Reply-To: OF3A9B0975.898CCB55-ON88256C53.00196AF8@boulder.ibm.com
[-- Attachment #1: Type: text/plain, Size: 849 bytes --]
This looks better, but there are still some issues:
The patch arrived here with tabs expanded into spaces, so it doesn't apply.
The EAGAIN fix by Mingming Cao is obviously incomplete, as I've
mentioned in a separate message to linux-aio. I think it would be
better if you would leave this out of this patch and let it be handled
separately.
async_poll_complete() is dead code and should be removed (this was an
issue in the original 2.4.19 code).
You should not reintroduce the ki_data member of struct iocb. Insead
you should use the existing private member of struct iocb.
In aio_abi.h, you should adjust the preceeding comment to remove mention
of IOCB_CMD_POLL.
Incidentally, this patch depends on the sys_io_cancel() portion of the
patch I posted yesterday in order for cancellation to return the correct
result to user space.
[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/x-pkcs7-signature, Size: 3537 bytes --]
prev parent reply other threads:[~2002-10-15 18:10 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-10-15 5:15 [PATCH] Re: [PATCH] async poll for 2.5 David Stevens
2002-10-15 18:16 ` John Gardiner Myers [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3DAC5B76.1050101@netscape.com \
--to=jgmyers@netscape.com \
--cc=akpm@digeo.com \
--cc=bcrl@redhat.com \
--cc=davem@redhat.com \
--cc=dlstevens@us.ibm.com \
--cc=linux-aio@kvack.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nagar@watson.ibm.com \
--cc=sct@redhat.com \
--cc=torvalds@transmeta.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.