Re: Setting TCP keepalive for Cyrus daemons

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 30, 2011 at 10:39:29PM +0200, Bron Gondwana wrote:
> On Sat, Feb 13, 2010 at 09:09:05PM +1100, Bron Gondwana wrote:
> > +                if (config_getint(IMAPOPT_TCP_KEEPALIVE_CNT)) {
> > +                    r = setsockopt(be->sock, SOL_TCP, TCP_KEEPCNT, &optval, optlen);
> > +                    if (r < 0) {
> > +                        syslog(LOG_ERR, "unable to setsocketopt(TCP_KEEPCNT): %m");
> > +                    }
> > +                }
> > +                if (config_getint(IMAPOPT_TCP_KEEPALIVE_IDLE)) {
> > +                    r = setsockopt(be->sock, SOL_TCP, TCP_KEEPIDLE, &optval, optlen);
> > +                    if (r < 0) {
> > +                        syslog(LOG_ERR, "unable to setsocketopt(TCP_KEEPIDLE): %m");
> > +                    }
> > +                }
> > +                if (config_getint(IMAPOPT_TCP_KEEPALIVE_INTVL)) {
> > +                    r = setsockopt(be->sock, SOL_TCP, TCP_KEEPINTVL, &optval, optlen);
> > +                    if (r < 0) {
> > +                        syslog(LOG_ERR, "unable to setsocketopt(TCP_KEEPINTVL): %m");
> > +                    }
> > +                }
> 
> Why didn't one of you scumbags code review me and tell me
> how bogus this is??

Because it was so different from the patch that I submitted that I
assumed it addressed a different problem.

> Now I have to fix it.
> 
> "keepalive interval 1 indeed"

-- 
-Gary Mills-        -Unix Group-        -Computer and Network Services-
----
Cyrus Home Page: http://www.cyrusimap.org/
List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/


[Index of Archives]     [Cyrus SASL]     [Squirrel Mail]     [Asterisk PBX]     [Video For Linux]     [Photo]     [Yosemite News]     [gtk]     [KDE]     [Gimp on Windows]     [Steve's Art]

  Powered by Linux