[PATCH 13/18] Removed nxt_always_inline from nxt_unit_close().

Alejandro Colomar alx.manpages at gmail.com
Thu Jun 2 18:05:07 UTC 2022


It's used several times in nxt_slow_path, where we shouldn't be
forcing the inline.

See the usage:

$ grep -rnB2 nxt_unit_close
src/nxt_unit.c-192-static int nxt_unit_app_queue_recv(nxt_unit_ctx_t *ctx, nxt_unit_port_t *port,
src/nxt_unit.c-193-    nxt_unit_read_buf_t *rbuf);
src/nxt_unit.c:194:nxt_always_inline static inline int nxt_unit_close(int fd);
--
src/nxt_unit.c-553-    }
src/nxt_unit.c-554-
src/nxt_unit.c:555:    nxt_unit_close(shared_queue_fd);
--
src/nxt_unit.c-569-    }
src/nxt_unit.c-570-
src/nxt_unit.c:571:    nxt_unit_close(ready_port.out_fd);
src/nxt_unit.c:572:    nxt_unit_close(queue_fd);
--
src/nxt_unit.c-581-
src/nxt_unit.c-582-    if (queue_fd != -1) {
src/nxt_unit.c:583:        nxt_unit_close(queue_fd);
--
src/nxt_unit.c-1151-
src/nxt_unit.c-1152-    if (recv_msg.fd[0] != -1) {
src/nxt_unit.c:1153:        nxt_unit_close(recv_msg.fd[0]);
--
src/nxt_unit.c-1155-
src/nxt_unit.c-1156-    if (recv_msg.fd[1] != -1) {
src/nxt_unit.c:1157:        nxt_unit_close(recv_msg.fd[1]);
--
src/nxt_unit.c-1807-
src/nxt_unit.c-1808-    if (req->content_fd != -1) {
src/nxt_unit.c:1809:        nxt_unit_close(req->content_fd);
--
src/nxt_unit.c-3080-
src/nxt_unit.c-3081-        if (res < (ssize_t) size) {
src/nxt_unit.c:3082:            nxt_unit_close(req->content_fd);
--
src/nxt_unit.c-3192-
src/nxt_unit.c-3193-    if (res < (ssize_t) size) {
src/nxt_unit.c:3194:        nxt_unit_close(req->content_fd);
--
src/nxt_unit.c-3767-                       strerror(errno), errno);
src/nxt_unit.c-3768-
src/nxt_unit.c:3769:        nxt_unit_close(fd);
--
src/nxt_unit.c-3805-    }
src/nxt_unit.c-3806-
src/nxt_unit.c:3807:    nxt_unit_close(fd);
--
src/nxt_unit.c-3886-                       strerror(errno), errno);
src/nxt_unit.c-3887-
src/nxt_unit.c:3888:        nxt_unit_close(fd);
--
src/nxt_unit.c-5161-    }
src/nxt_unit.c-5162-
src/nxt_unit.c:5163:    nxt_unit_close(queue_fd);
--
src/nxt_unit.c-5168-
src/nxt_unit.c-5169-    if (queue_fd != -1) {
src/nxt_unit.c:5170:        nxt_unit_close(queue_fd);
--
src/nxt_unit.c-5315-        pthread_mutex_unlock(&lib->mutex);
src/nxt_unit.c-5316-
src/nxt_unit.c:5317:        nxt_unit_close(port_sockets[0]);
src/nxt_unit.c:5318:        nxt_unit_close(port_sockets[1]);
--
src/nxt_unit.c-5333-    port = nxt_unit_add_port(ctx, &new_port, NULL);
src/nxt_unit.c-5334-    if (nxt_slow_path(port == NULL)) {
src/nxt_unit.c:5335:        nxt_unit_close(port_sockets[0]);
src/nxt_unit.c:5336:        nxt_unit_close(port_sockets[1]);
--
src/nxt_unit.c-5408-
src/nxt_unit.c-5409-        if (port->in_fd != -1) {
src/nxt_unit.c:5410:            nxt_unit_close(port->in_fd);
--
src/nxt_unit.c-5414-
src/nxt_unit.c-5415-        if (port->out_fd != -1) {
src/nxt_unit.c:5416:            nxt_unit_close(port->out_fd);
--
src/nxt_unit.c-5463-
src/nxt_unit.c-5464-        if (port->in_fd != -1) {
src/nxt_unit.c:5465:            nxt_unit_close(port->in_fd);
--
src/nxt_unit.c-5473-
src/nxt_unit.c-5474-        if (port->out_fd != -1) {
src/nxt_unit.c:5475:            nxt_unit_close(port->out_fd);
--
src/nxt_unit.c-6326-
src/nxt_unit.c-6327-static inline int
src/nxt_unit.c:6328:nxt_unit_close(int fd)
---
 src/nxt_unit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/nxt_unit.c b/src/nxt_unit.c
index f3f8bb9..cac913d 100644
--- a/src/nxt_unit.c
+++ b/src/nxt_unit.c
@@ -191,7 +191,7 @@ static int nxt_unit_port_queue_recv(nxt_unit_port_t *port,
     nxt_unit_read_buf_t *rbuf);
 static int nxt_unit_app_queue_recv(nxt_unit_ctx_t *ctx, nxt_unit_port_t *port,
     nxt_unit_read_buf_t *rbuf);
-nxt_always_inline static inline int nxt_unit_close(int fd);
+static inline int nxt_unit_close(int fd);
 static int nxt_unit_fd_blocking(int fd);
 
 static int nxt_unit_port_hash_add(nxt_lvlhsh_t *port_hash,
-- 
2.36.1



More information about the unit mailing list