[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