From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 0E2851F5AE; Sat, 27 Jun 2020 22:32:08 +0000 (UTC) Date: Sat, 27 Jun 2020 22:32:07 +0000 From: Eric Wong To: Kyle Meyer Cc: meta@public-inbox.org Subject: Re: [PATCH 11/34] watch: use signalfd for Maildir watching Message-ID: <20200627223207.GA5053@dcvr> References: <20200627100400.9871-1-e@yhbt.net> <20200627100400.9871-12-e@yhbt.net> <87tuyw5nxy.fsf@kyleam.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <87tuyw5nxy.fsf@kyleam.com> List-Id: Kyle Meyer wrote: > One of these days, I'm going to read through a series on this list and > make a valuable technical comment :] Until then... No worries, your contributions are valuable regardless of whether they're technical or not :> > Eric Wong writes: > > > We can get rid of the janky wannabe > > self-using-a-directory-instead-of-pipe thing we needed to > > workaround Filesys::Notify::Simple being blocking. > > > > For existing Maildir users, this should be more robust and > > immune to missed wakeups for to signalfd and kqueue-enabled > > s/for to/for/ ? Thanks, will queue up some fixes along with your other comment. There's deeper issues with this series w.r.t zombies for existing Maildir users...