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 74C66C4332F for ; Thu, 24 Nov 2022 08:01:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229379AbiKXIB0 (ORCPT ); Thu, 24 Nov 2022 03:01:26 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49538 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229508AbiKXIBZ (ORCPT ); Thu, 24 Nov 2022 03:01:25 -0500 Received: from gnuweeb.org (gnuweeb.org [51.81.211.47]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 78D5AC6218 for ; Thu, 24 Nov 2022 00:01:24 -0800 (PST) Received: from localhost.localdomain (unknown [182.253.183.240]) by gnuweeb.org (Postfix) with ESMTPSA id AA2078166D; Thu, 24 Nov 2022 08:01:20 +0000 (UTC) X-GW-Data: lPqxHiMPbJw1wb7CM9QUryAGzr0yq5atzVDdxTR0iA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gnuweeb.org; s=default; t=1669276884; bh=ERtkqvk6tbxxYN/xhabf3FdA1VeYaViBkUA0ip8IjMI=; h=From:To:Cc:Subject:Date:From; b=o4+DoaasMmIAp+lP8E6+ImL4DmgOLUpJdLqjTDpoud8BraTA4EhYPZC6xM7hQXnYr H70BqqLRFZrBytKUj9wldTrbcvRVxgdeLhSqBPhCl8tqN+qHD78D8Y3ViBxp9VGjWV tzrfykgHmCjQmg2CG9CSrxV33PPDwxlxThmIxRDIuORKZS+TJsdylvohRJzldFUxRo eaNK4PIuqLiv2Kmpx6+lxEQ2/DcmaFX942SFzCb6mLj3BQEOdNi8x+MwbEHzJrEivH NcPwNUMaSfOi5BFmLayYtNPxlgWRbYKhXVbgOouurxlOdDJ63Mv65yhdtOyVfdNAO9 bJnpz9B9ms2eQ== From: Ammar Faizi To: Jens Axboe Cc: Ammar Faizi , Pavel Begunkov , io-uring Mailing List , GNU/Weeb Mailing List , Gilang Fachrezy , Muhammad Rizki , Alviro Iskandar Setiawan , VNLX Kernel Department Subject: [PATCH liburing v1 0/7] Ensure we mark internal functions and variables as static Date: Thu, 24 Nov 2022 15:00:55 +0700 Message-Id: <20221124075846.3784701-1-ammar.faizi@intel.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: io-uring@vger.kernel.org From: Ammar Faizi Hi Jens, This series is a -Wmissing-prototypes enforcement. -Wmissing-prototypes is a clang C compiler flag that warns us if we have functions or variables that are not used outisde the translation unit, but not marked as static. This enforcement is good because it hints the compiler to do escape analysis and optimization better. There are 7 patches in this series. - Patch 1 is a core library change. Export __io_uring_flush_sq(). - Patch 2 to 6 are cleanups preparation before enforcing -Wmissing-prototypes. - Patch 7 is to add `-Wmissing-prototypes` for GitHub CI bot. This series has been build tested each patch with the GitHub CI robot and no breakage is found. Signed-off-by: Ammar Faizi --- Ammar Faizi (7): liburing.h: Export `__io_uring_flush_sq()` function test/io_uring_setup: Remove unused functions ucontext-cp: Remove an unused function tests: Mark internal functions as static ucontext-cp: Mark internal functions as static test/Makefile: Omit `-Wmissing-prototypes` from the C++ compiler flags github: Add `-Wmissing-prototypes` for GitHub CI bot .github/workflows/build.yml | 7 ++-- examples/ucontext-cp.c | 19 +-------- src/include/liburing.h | 1 + src/liburing.map | 5 +++ test/Makefile | 11 ++++- test/accept-link.c | 2 +- test/accept-reuse.c | 9 ++-- test/ce593a6c480a.c | 4 +- test/defer-taskrun.c | 2 +- test/exit-no-cleanup.c | 2 +- test/hardlink.c | 2 +- test/io_uring_setup.c | 83 ++----------------------------------- test/link_drain.c | 2 +- test/multicqes_drain.c | 27 +++++++----- test/nvme.h | 3 +- test/poll-link.c | 2 +- test/poll-mshot-overflow.c | 2 +- test/read-before-exit.c | 2 +- test/ring-leak2.c | 2 +- test/sq-poll-kthread.c | 2 +- test/sqpoll-cancel-hang.c | 2 +- test/symlink.c | 3 +- test/timeout-new.c | 12 ++++-- 23 files changed, 70 insertions(+), 136 deletions(-) base-commit: b90a28636e5b5efe6dc1383acc90aec61814d9ba -- Ammar Faizi