From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2AB72C54EE9 for ; Fri, 2 Sep 2022 07:15:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235297AbiIBHP3 (ORCPT ); Fri, 2 Sep 2022 03:15:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35728 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229482AbiIBHP2 (ORCPT ); Fri, 2 Sep 2022 03:15:28 -0400 Received: from gnuweeb.org (gnuweeb.org [51.81.211.47]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6FA402612B for ; Fri, 2 Sep 2022 00:15:27 -0700 (PDT) Received: from localhost.localdomain (unknown [182.2.70.226]) by gnuweeb.org (Postfix) with ESMTPSA id 3CCEF80C16; Fri, 2 Sep 2022 07:15:23 +0000 (UTC) X-GW-Data: lPqxHiMPbJw1wb7CM9QUryAGzr0yq5atzVDdxTR0iA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gnuweeb.org; s=default; t=1662102927; bh=Ht7mLQSEFXb0IpvAmkmJ+J7h+vJSlc7Gw1ey9ZsP9r8=; h=From:To:Cc:Subject:Date:From; b=Au5ZJdz3TqEecbcUf53hfuJS11Gsm05Elr5nLjGKM58XGwGKDadZ1ohLxHxPJ1t5y uQuV17EZlFqTSm3W+XuByhweM19axX6jt3/pDFxh8imPzCofq1IHFsEqiX+anxpGtd VMrxcdF070EBnknQhe7zuRP4HghEVWS2ApqUuTFCrzyfwOIlbEWnjq+qFWlxSbSQfS SKObnneVtDKe5yhJmrKQoU+9tRhX+GR/FJPuLxKUxpiXiDdklTOsxImMNxrkJMNVCV xeDWgk9vfSX/3OOvdg9yO5ggIUAXOVLsXeOy7MmcwF/Of/0bKcOBSvIyvG67cCE6JQ TJVTMGOf3JqTw== From: Ammar Faizi To: Jens Axboe Cc: Ammar Faizi , Dylan Yudaken , Facebook Kernel Team , Pavel Begunkov , io-uring Mailing List , GNU/Weeb Mailing List , Kanna Scarlet , Muhammad Rizki , Alviro Iskandar Setiawan Subject: [PATCH liburing v2 00/12] Introducing t_bind_ephemeral_port() function Date: Fri, 2 Sep 2022 14:14:53 +0700 Message-Id: <20220902071153.3168814-1-ammar.faizi@intel.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: io-uring@vger.kernel.org From: Ammar Faizi Hi, This is revision v2 of "Introducing t_bind_ephemeral_port() function". After discussing an intermittent bind() issue with Dylan, I decided to introduce a new helper function, t_bind_ephemeral_port(). ## Problem: We have many places where we need to bind() a socket to any unused port number. To achieve that, the current approach does one of the following mechanisms: 1) Randomly brute force the port number until the bind() syscall succeeds. 2) Use a static port at compile time (randomly chosen too). This is not reliable and it results in an intermittent issue (test fails when the selected port is in use). ## Solution: Setting @addr->sin_port to zero on a bind() syscall lets the kernel choose a port number that is not in use. The caller then can know the port number to be bound by invoking a getsockname() syscall after bind() succeeds. Wrap this procedure in a new function called t_bind_ephemeral_port(). The selected port will be returned into @addr->sin_port, the caller can use it later to connect() or whatever they need. ## Patchset summary: There are 12 patches in this series, summary: 1) Patch #1 introduces a new helper function t_bind_ephemeral_port(). 2) Patch #2 to #6 get rid of the port number brute force mechanism. 3) Patch #7 to #12 stop using a static port number. ## Changelog v1 -> v2: - Fix variable placement in patch #2. - Append Reviewed-by tags from Alviro. - Append Tested-by tags from Alviro. Link: https://lore.kernel.org/r/918facd1-78ba-2de7-693a-5f8c65ea2fcd@gnuweeb.org Cc: Dylan Yudaken Cc: Facebook Kernel Team Cc: Pavel Begunkov Signed-off-by: Ammar Faizi --- Ammar Faizi (12): test/helpers: Add `t_bind_ephemeral_port()` function t/poll-link: Don't brute force the port number t/socket-rw: Don't brute force the port number t/socket-rw-eagain: Don't brute force the port number t/socket-rw-offset: Don't brute force the port number t/files-exit-hang-poll: Don't brute force the port number t/socket: Don't use a static port number t/connect: Don't use a static port number t/shutdown: Don't use a static port number t/recv-msgall: Don't use a static port number t/232c93d07b74: Don't use a static port number t/recv-msgall-stream: Don't use a static port number test/232c93d07b74.c | 10 ++++------ test/accept.c | 5 +---- test/files-exit-hang-poll.c | 23 +++-------------------- test/helpers.c | 18 ++++++++++++++++++ test/helpers.h | 7 +++++++ test/poll-link.c | 27 +++++++++------------------ test/recv-msgall-stream.c | 22 ++++++++++------------ test/recv-msgall.c | 10 ++++------ test/shutdown.c | 7 +++---- test/socket-rw-eagain.c | 14 ++------------ test/socket-rw-offset.c | 13 ++----------- test/socket-rw.c | 13 ++----------- test/socket.c | 11 ++++++----- 13 files changed, 71 insertions(+), 109 deletions(-) base-commit: b8c37f02662faa4f2b61840b123201ccc5678fb1 -- Ammar Faizi