From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on gnuweeb.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=5.0 tests=ALL_TRUSTED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NO_DNS_FOR_FROM,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 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) 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 List-Id: 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