All of lore.kernel.org
 help / color / mirror / Atom feed
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 --]

      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.