Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752740Ab2JQXaK (ORCPT ); Wed, 17 Oct 2012 19:30:10 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:58227 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752086Ab2JQXaG (ORCPT ); Wed, 17 Oct 2012 19:30:06 -0400 Date: Wed, 17 Oct 2012 16:30:04 -0700 From: Andrew Morton To: mtk.manpages@gmail.com Cc: "Paton J. Lewis" , Alexander Viro , Jason Baron , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Paul Holland , Davide Libenzi , libc-alpha@sourceware.org, Linux API Subject: Re: [PATCH v2] epoll: Support for disabling items, and a self-test app. Message-Id: <20121017163004.fb9de1b3.akpm@linux-foundation.org> In-Reply-To: References: <1345756535-8372-1-git-send-email-palewis@adobe.com> X-Mailer: Sylpheed 3.0.2 (GTK+ 2.20.1; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1594 Lines: 31 On Tue, 16 Oct 2012 17:12:57 +0200 "Michael Kerrisk (man-pages)" wrote: > On Thu, Aug 23, 2012 at 11:15 PM, Paton J. Lewis wrote: > > From: "Paton J. Lewis" > > > > Enhanced epoll_ctl to support EPOLL_CTL_DISABLE, which disables an epoll item. > > If epoll_ctl doesn't return -EBUSY in this case, it is then safe to delete the > > epoll item in a multi-threaded environment. Also added a new test_epoll self- > > test app to both demonstrate the need for this feature and test it. > > (There's a lot of background missing from this version of the patch > that was included in the previous version > [http://thread.gmane.org/gmane.linux.kernel/1311457]. It helps to > include the full rationale with a revised patch--best not to assume > that someone has the context of past mails when reading a revised > patch.) > > I've taken a look at this patch as it currently stands in 3.7-rc1, and > done a bit of testing. (By the way, the test program > tools/testing/selftests/epoll/test_epoll.c does not compile...) Thanks for this. You raise significant issues. If we can't get these fully resolved over the next month or so, we should revert the patch so this new API doesn't get released in 3.7. I have queued a patch to do this and shall maintain it while I watch developments... -- 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/