Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Tue, 15 Oct 2002 18:37:41 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Tue, 15 Oct 2002 18:36:19 -0400 Received: from to-velocet.redhat.com ([216.138.202.10]:63216 "EHLO touchme.toronto.redhat.com") by vger.kernel.org with ESMTP id ; Tue, 15 Oct 2002 18:35:28 -0400 Date: Tue, 15 Oct 2002 18:41:23 -0400 From: Benjamin LaHaise To: John Gardiner Myers Cc: Davide Libenzi , Dan Kegel , Shailabh Nagar , linux-kernel , linux-aio , Andrew Morton , David Miller , Linus Torvalds , Stephen Tweedie Subject: Re: [PATCH] async poll for 2.5 Message-ID: <20021015184123.J16156@redhat.com> References: <3DAC9859.5060005@netscape.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <3DAC9859.5060005@netscape.com>; from jgmyers@netscape.com on Tue, Oct 15, 2002 at 03:36:09PM -0700 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1066 Lines: 21 On Tue, Oct 15, 2002 at 03:36:09PM -0700, John Gardiner Myers wrote: > Nonetheless, the requirement for user space to test the condition after > the registration, not before, is subtle. A program which does these in > the wrong order is still likely to pass QA and will fail in production > in a way that will be difficult to diagnose. There is no rational > reason for the kernel to not test the condition upon registration. I suppose one way of getting the async poll code up to snuff would be to cache the poll registration in the file descriptor. Alternatively, the iocb could simply persist until it is cancelled or a refire is permitted (so that the event queue does not get overrun). Would you care to try crunching the numbers with polltest on 2.5? -ben -- "Do you seek knowledge in time travel?" - 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/