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 6F74A1F4B4 for ; Sun, 3 Jan 2021 09:48:42 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 0/3] lei-related test fixes Date: Sun, 3 Jan 2021 09:48:39 +0000 Message-Id: <20210103094842.22906-1-e@80x24.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: Still chasing down a weird problem which causes t/lei.t and t/lei-oneshot.t to fail on FreeBSD 11.4 with IO::FDPass under high load. No syscall errors are reported, but it's like the FDs aren't passed at all... Maybe it's fixed in 12.x 1/3 is to cut down on noise 2/3 is a no-brainer :x 3/3 was for me to play around with, but also avoids malloc and a potential leak in IO::FDPass (upstream's been notified). However, I'm considering just making our C code pass all 3 FDs with one syscall since it's possible. In any case, the C parts of PublicInbox::Spawn should probably renamed PublicInbox::C... Eric Wong (3): t/lei: use $lei->() callback wrapper testcommon: prepare_redirects: fix error message spawn: support send_fd+recv_fd w/o IO::FDPass lib/PublicInbox/LEI.pm | 6 ++- lib/PublicInbox/Spawn.pm | 78 ++++++++++++++++++++++++++++++-- lib/PublicInbox/TestCommon.pm | 4 +- script/lei | 7 ++- t/lei.t | 84 ++++++++++++++++------------------- t/spawn.t | 18 ++++++++ 6 files changed, 141 insertions(+), 56 deletions(-)