Welcome! Log In Create A New Profile

Advanced

Re: upstream keepalive - call for testing

All files from this thread

File Name File Size   Posted by Date  
upstream.patch 2.3 KB open | download Matthieu Tourne 08/12/2011 Read message
ngx_http_upstream_keepalive.patch 542 bytes open | download Matthieu Tourne 08/12/2011 Read message
upstream.patch 2.6 KB open | download Matthieu Tourne 08/12/2011 Read message
Maxim Dounin
August 26, 2011 05:40AM
Hello!

On Fri, Aug 26, 2011 at 03:08:10AM -0400, magicbear wrote:

> Hello, I found a bug, the stub_status module shows connection and
> waiting is only increase without decrease. But in fact, system
> connection haven't such many.
> Here is the trend chart
> http://m-b.cc/tmp/bugs.png

What's in error log?

Graph suggests there are worker processes dying for some reason,
there should be messages like "[alert] ... worker process ...
exited on signal ..." in global error log.

If there are such messages, please follow [1] to obtain core dump
and provide backtrace. Some more details as outlined in [2] may
be also helpful.

[1] http://wiki.nginx.org/Debugging#Core_dump
[2] http://wiki.nginx.org/Debugging#Asking_for_help

Maxim Dounin

>
> Maxim Dounin Wrote:
> -------------------------------------------------------
> > Hello!
> >
> > On Wed, Aug 24, 2011 at 01:11:43PM -0400,
> > magicbear wrote:
> >
> > > Thanks for your hard work, I have found that if
> > using https backend, it
> > > won't work, server will direct close the
> > connection.
> > >
> > > curl --head 'http://localhost/track.js'
> > > curl: (52) Empty reply from server
> >
> > Yes, thank you for report.
> >
> > Keeping https connections will require additional
> > support both in
> > nginx core and upstream keepalive module. You may
> > try the
> > attached patches, also available here:
> >
> > http://mdounin.ru/files/patch-nginx-keepalive-http
> > s.txt
> > http://mdounin.ru/files/patch-nginx-keepalive-http
> > s-module.txt
> >
> > The first one is for nginx itself, should be
> > applied after
> > keepalive patch.
> >
> > The second one (actually, two grouped in one file)
> > for upstream
> > keepalive module.
> >
> > Maxim Dounin
> > # HG changeset patch
> > # User Maxim Dounin <mdounin@mdounin.ru>
> > # Date 1314229425 -14400
> > # Node ID ac0a7fd4de491e64d42f218691b681f7b3fa931b
> > # Parent e865cb2cc06a88c01a439bfdd0d0d7dec54713f0
> > Upstream: create separate pool for peer
> > connections.
> >
> > This is required to support persistant https
> > connections as various ssl
> > structures are allocated from connection's pool.
> >
> > diff --git a/src/http/ngx_http_upstream.c
> > b/src/http/ngx_http_upstream.c
> > --- a/src/http/ngx_http_upstream.c
> > +++ b/src/http/ngx_http_upstream.c
> > @@ -1146,8 +1146,17 @@
> > ngx_http_upstream_connect(ngx_http_reque
> > c->sendfile &= r->connection->sendfile;
> > u->output.sendfile = c->sendfile;
> >
> > - c->pool = r->pool;
> > + if (c->pool == NULL) {
> > + c->pool = ngx_create_pool(128,
> > r->connection->log);
> > + if (c->pool == NULL) {
> > + ngx_http_upstream_finalize_request(r,
> > u,
> > +
> > NGX_HTTP_INTERNAL_SERVER_ERROR);
> > + return;
> > + }
> > + }
> > +
> > c->log = r->connection->log;
> > + c->pool->log = c->log;
> > c->read->log = c->log;
> > c->write->log = c->log;
> >
> > @@ -2912,6 +2921,7 @@
> > ngx_http_upstream_next(ngx_http_request_
> > }
> > #endif
> >
> > +
> > ngx_destroy_pool(u->peer.connection->pool);
> > ngx_close_connection(u->peer.connection);
> > }
> >
> > @@ -3006,6 +3016,7 @@
> > ngx_http_upstream_finalize_request(ngx_h
> > "close http upstream
> > connection: %d",
> > u->peer.connection->fd);
> >
> > +
> > ngx_destroy_pool(u->peer.connection->pool);
> > ngx_close_connection(u->peer.connection);
> > }
> >
> > # HG changeset patch
> > # User Maxim Dounin <mdounin@mdounin.ru>
> > # Date 1314229646 -14400
> > # Node ID 67b12141506c6be2115b6b0aa151068188b97975
> > # Parent f3b50effc1d476b040908700bb772197d31fbd80
> > Keepalive: set_session and save_session callbacks.
> >
> > diff --git a/ngx_http_upstream_keepalive_module.c
> > b/ngx_http_upstream_keepalive_module.c
> > --- a/ngx_http_upstream_keepalive_module.c
> > +++ b/ngx_http_upstream_keepalive_module.c
> > @@ -32,6 +32,11 @@ typedef struct {
> > ngx_event_get_peer_pt
> > original_get_peer;
> > ngx_event_free_peer_pt
> > original_free_peer;
> >
> > +#if (NGX_HTTP_SSL)
> > + ngx_event_set_peer_session_pt
> > original_set_session;
> > + ngx_event_save_peer_session_pt
> > original_save_session;
> > +#endif
> > +
> > ngx_uint_t failed;
> > /* unsigned:1 */
> >
> > } ngx_http_upstream_keepalive_peer_data_t;
> > @@ -59,6 +64,13 @@ static void
> > ngx_http_upstream_free_keepa
> > static void
> > ngx_http_upstream_keepalive_dummy_handler(ngx_even
> > t_t *ev);
> > static void
> > ngx_http_upstream_keepalive_close_handler(ngx_even
> > t_t *ev);
> >
> > +#if (NGX_HTTP_SSL)
> > +static ngx_int_t
> > ngx_http_upstream_keepalive_set_session(
> > + ngx_peer_connection_t *pc, void *data);
> > +static void
> > ngx_http_upstream_keepalive_save_session(ngx_peer_
> > connection_t *pc,
> > + void *data);
> > +#endif
> > +
> > static void
> > *ngx_http_upstream_keepalive_create_conf(ngx_conf_
> > t *cf);
> > static char
> > *ngx_http_upstream_keepalive(ngx_conf_t *cf,
> > ngx_command_t *cmd,
> > void *conf);
> > @@ -182,6 +194,13 @@
> > ngx_http_upstream_init_keepalive_peer(ng
> > r->upstream->peer.get =
> > ngx_http_upstream_get_keepalive_peer;
> > r->upstream->peer.free =
> > ngx_http_upstream_free_keepalive_peer;
> >
> > +#if (NGX_HTTP_SSL)
> > + kp->original_set_session =
> > r->upstream->peer.set_session;
> > + kp->original_save_session =
> > r->upstream->peer.save_session;
> > + r->upstream->peer.set_session =
> > ngx_http_upstream_keepalive_set_session;
> > + r->upstream->peer.save_session =
> > ngx_http_upstream_keepalive_save_session;
> > +#endif
> > +
> > return NGX_OK;
> > }
> >
> > @@ -423,6 +442,29 @@ close:
> > }
> >
> >
> > +#if (NGX_HTTP_SSL)
> > +
> > +static ngx_int_t
> > +ngx_http_upstream_keepalive_set_session(ngx_peer_
> > connection_t *pc, void *data)
> > +{
> > + ngx_http_upstream_keepalive_peer_data_t *kp
> > = data;
> > +
> > + return kp->original_set_session(pc,
> > kp->data);
> > +}
> > +
> > +
> > +static void
> > +ngx_http_upstream_keepalive_save_session(ngx_peer
> > _connection_t *pc, void *data)
> > +{
> > + ngx_http_upstream_keepalive_peer_data_t *kp
> > = data;
> > +
> > + kp->original_save_session(pc, kp->data);
> > + return;
> > +}
> > +
> > +#endif
> > +
> > +
> > static void *
> > ngx_http_upstream_keepalive_create_conf(ngx_conf_
> > t *cf)
> > {
> > # HG changeset patch
> > # User Maxim Dounin <mdounin@mdounin.ru>
> > # Date 1314229663 -14400
> > # Node ID 3affea1af30649ac1934b01ab30d175abd4fb3be
> > # Parent 67b12141506c6be2115b6b0aa151068188b97975
> > Keepalive: destroy connection pool.
> >
> > diff --git a/ngx_http_upstream_keepalive_module.c
> > b/ngx_http_upstream_keepalive_module.c
> > --- a/ngx_http_upstream_keepalive_module.c
> > +++ b/ngx_http_upstream_keepalive_module.c
> > @@ -353,6 +353,7 @@
> > ngx_http_upstream_free_keepalive_peer(ng
> > item = ngx_queue_data(q,
> > ngx_http_upstream_keepalive_cache_t,
> > queue);
> >
> > +
> > ngx_destroy_pool(item->connection->pool);
> >
> > ngx_close_connection(item->connection);
> >
> > } else {
> > @@ -437,6 +438,7 @@ close:
> > conf = item->conf;
> >
> > ngx_queue_remove(&item->queue);
> > + ngx_destroy_pool(item->connection->pool);
> > ngx_close_connection(item->connection);
> > ngx_queue_insert_head(&conf->free,
> > &item->queue);
> > }
> > _______________________________________________
> > nginx mailing list
> > nginx@nginx.org
> > http://mailman.nginx.org/mailman/listinfo/nginx
>
> Posted at Nginx Forum: http://forum.nginx.org/read.php?2,213207,214320#msg-214320
>
> _______________________________________________
> nginx mailing list
> nginx@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx

_______________________________________________
nginx mailing list
nginx@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx
Subject Author Posted

upstream keepalive - call for testing

Maxim Dounin August 01, 2011 12:10PM

Re: upstream keepalive - call for testing

liseen August 02, 2011 09:38AM

Re: upstream keepalive - call for testing

Maxim Dounin August 02, 2011 01:34PM

Re: upstream keepalive - call for testing

António P. P. Almeida August 02, 2011 11:28AM

Re: upstream keepalive - call for testing

Maxim Dounin August 02, 2011 01:38PM

Re: upstream keepalive - call for testing

David Yu August 02, 2011 01:44PM

Re: upstream keepalive - call for testing

Maxim Dounin August 02, 2011 01:52PM

Re: upstream keepalive - call for testing

David Yu August 02, 2011 01:54PM

Re: upstream keepalive - call for testing

Maxim Dounin August 02, 2011 02:48PM

Re: upstream keepalive - call for testing

David Yu August 02, 2011 03:10PM

Re: upstream keepalive - call for testing

liseen August 02, 2011 11:58PM

Re: upstream keepalive - call for testing

splitice August 03, 2011 01:22AM

Re: upstream keepalive - call for testing

Matthieu Tourne August 03, 2011 08:08PM

Re: upstream keepalive - call for testing

Maxim Dounin August 04, 2011 02:54AM

Re: upstream keepalive - call for testing

splitice August 08, 2011 12:46AM

Re: upstream keepalive - call for testing

Maxim Dounin August 08, 2011 05:24AM

Re: upstream keepalive - call for testing

splitice August 08, 2011 05:36AM

Re: upstream keepalive - call for testing

splitice August 08, 2011 05:38AM

Re: upstream keepalive - call for testing Attachments

Matthieu Tourne August 12, 2011 03:34PM

Re: upstream keepalive - call for testing

Maxim Dounin August 12, 2011 04:00PM

Re: upstream keepalive - call for testing

Matthieu Tourne August 12, 2011 05:14PM

Re: upstream keepalive - call for testing

Maxim Dounin August 12, 2011 06:28PM

Re: upstream keepalive - call for testing Attachments

Matthieu Tourne August 12, 2011 06:44PM

Re: upstream keepalive - call for testing

Matthieu Tourne August 16, 2011 07:32PM

Re: upstream keepalive - call for testing

Maxim Dounin August 16, 2011 08:24PM

Re: upstream keepalive - call for testing

magicbear August 24, 2011 01:11PM

Re: upstream keepalive - call for testing

Maxim Dounin August 24, 2011 08:06PM

Re: upstream keepalive - call for testing

sv August 24, 2011 09:18PM

Re: upstream keepalive - call for testing

magicbear August 25, 2011 01:30AM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 03:08AM

Re: upstream keepalive - call for testing

Maxim Dounin August 26, 2011 05:40AM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 07:01AM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 07:04AM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 07:28AM

Re: upstream keepalive - call for testing

Maxim Dounin August 26, 2011 07:38AM

upstream keepalive close connections actively

cfsego August 02, 2011 10:50PM

Re: upstream keepalive close connections actively

Maxim Dounin August 03, 2011 03:40AM

RE: upstream keepalive close connections actively

Charles Chen August 03, 2011 05:54AM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 07:54AM

Re: upstream keepalive - call for testing

Maxim Dounin August 26, 2011 11:56AM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 12:17PM

Re: upstream keepalive - call for testing

Maxim Dounin August 26, 2011 02:08PM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 03:00PM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 12:28PM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 01:00PM

Re: upstream keepalive - call for testing

magicbear August 26, 2011 01:51PM

Re: upstream keepalive - call for testing

magicbear August 28, 2011 01:07PM

Re: upstream keepalive - call for testing

magicbear August 28, 2011 01:10PM

Re: upstream keepalive - call for testing

Maxim Dounin August 28, 2011 09:48PM

Re: upstream keepalive - call for testing

magicbear August 31, 2011 04:04PM

Re: upstream keepalive - call for testing

splitice August 31, 2011 09:58PM

Re: upstream keepalive - call for testing

magicbear September 01, 2011 09:38AM

Re: upstream keepalive - call for testing

magicbear September 04, 2011 01:33PM

Re: upstream keepalive - call for testing

Maxim Dounin September 04, 2011 02:22PM

Re: upstream keepalive - call for testing

magicbear September 04, 2011 02:34PM

Re: upstream keepalive - call for testing

Maxim Dounin September 05, 2011 03:10AM

Re: upstream keepalive - call for testing

ビリビリⅤ September 05, 2011 11:44AM

Re: upstream keepalive - call for testing

Maxim Dounin September 05, 2011 02:04PM

Re: upstream keepalive - call for testing

magicbear September 06, 2011 02:39AM

Re: upstream keepalive - call for testing

Matthieu Tourne September 07, 2011 07:36PM

Re: upstream keepalive - call for testing

Maxim Dounin September 08, 2011 05:28AM

Re: upstream keepalive - call for testing

Maxim Dounin September 08, 2011 11:44AM

Re: upstream keepalive - call for testing

Matthieu Tourne September 08, 2011 06:06PM

Re: upstream keepalive - call for testing

magicbear September 14, 2011 06:54PM

Re: upstream keepalive - call for testing

magicbear September 15, 2011 01:52PM

Re: upstream keepalive - call for testing

splitice September 15, 2011 09:44PM

Re: upstream keepalive - call for testing

philipp December 29, 2011 07:47AM

Re: upstream keepalive - call for testing

Maxim Dounin December 29, 2011 10:06AM

Re: upstream keepalive - call for testing

alexscott March 08, 2012 09:30AM

Re: upstream keepalive - call for testing

Andrew Alexeev March 09, 2012 01:20AM

Re: upstream keepalive - call for testing

alexscott March 12, 2012 10:35AM

Re: upstream keepalive - call for testing

Maxim Dounin March 12, 2012 10:56AM

Re: upstream keepalive - call for testing

alexscott March 12, 2012 01:40PM

Re: upstream keepalive - call for testing

alexscott March 12, 2012 03:55PM

Re: upstream keepalive - call for testing

Maxim Dounin March 12, 2012 02:00PM



Sorry, only registered users may post in this forum.

Click here to login

Online Users

Guests: 100
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 500 on July 15, 2024
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready