Bug #37059 for bind9-public: runtime error

Hi there. I built bind 9.10-0-P2 on solaris 10 sparc, using studio compiler 12.3 I selected use_tuning=large just for fun. Build worked ok, but at runtime I get these messages in the log. Sep 3 09:45:40 sif1.its.uow.edu.au named[5461]: [ID 873579 syslog.error] socket.c:2786: unexpected error: Sep 3 09:45:40 sif1.its.uow.edu.au named[5461]: [ID 873579 syslog.error] setsockopt(560, SO_RCVBUF, 16777216) failed: No buffer space available Sep 3 09:45:41 sif1.its.uow.edu.au named[5461]: [ID 873579 syslog.error] socket.c:2786: unexpected error: Sep 3 09:45:41 sif1.its.uow.edu.au named[5461]: [ID 873579 syslog.error] setsockopt(560, SO_RCVBUF, 16777216) failed: No buffer space available Sep 3 09:45:41 sif1.its.uow.edu.au named[5461]: [ID 873579 syslog.error] socket.c:2786: unexpected error: Sep 3 09:45:41 sif1.its.uow.edu.au named[5461]: [ID 873579 syslog.error] setsockopt(560, SO_RCVBUF, 16777216) failed: No buffer space available It looks like the setting of RCVBUFSIZE in lib/isc/unix/socket.c is a bit excessive. /*% * The size to raise the receive buffer to (from BIND 8). */ #ifdef TUNE_LARGE #define RCVBUFSIZE (16*1024*1024) #else #define RCVBUFSIZE (32*1024) #endif /* TUNE_LARGE */ Increasing it by a factor of 500 seems a tad overkill and seems to cause problems on solaris. Could I suggest a factor of 8-32 might be a better idea. Actually, given the increasing size of DNS answers, maybe increasing the default would be a good idea as well. Anyway, just thought you would like to know. I am rebuilding without use_tuning. Keep up the good work. Regards, pdg Peter Gray Ph (direct): +61 2 4221 3770 Information Technology Services Ph (switch): +61 2 4221 3555 University of Wollongong Fax: +61 2 4229 1958 Wollongong NSW 2522 Email: pdg@uow.edu.au Australia URL: http://pdg.uow.edu.au

---

This change should build diff --git a/lib/isc/unix/socket.c b/lib/isc/unix/socket.c index 17012ed..b4c1c3f 100644 --- a/lib/isc/unix/socket.c +++ b/lib/isc/unix/socket.c @@ -306,7 +306,11 @@ typedef isc_event_t intev_t; * The size to raise the receive buffer to (from BIND 8). */ #ifdef TUNE_LARGE +#ifdef sun +#define RCVBUFSIZE (1*1024*1024) +#else #define RCVBUFSIZE (16*1024*1024) +#endif #else #define RCVBUFSIZE (32*1024) #endif /* TUNE_LARGE */

---

On Thu, Sep 04, 2014 at 02:55:09AM +0000, Mark Andrews via RT wrote: > This change should build > > diff --git a/lib/isc/unix/socket.c b/lib/isc/unix/socket.c > index 17012ed..b4c1c3f 100644 > --- a/lib/isc/unix/socket.c > +++ b/lib/isc/unix/socket.c > @@ -306,7 +306,11 @@ typedef isc_event_t intev_t; > * The size to raise the receive buffer to (from BIND 8). > */ > #ifdef TUNE_LARGE > +#ifdef sun > +#define RCVBUFSIZE (1*1024*1024) > +#else > #define RCVBUFSIZE (16*1024*1024) > +#endif > #else > #define RCVBUFSIZE (32*1024) > #endif /* TUNE_LARGE */ > Yes, works no problems. Have you considered changing all the places where you test for TUNE_LARGE to something like #define RCVBUFSIZE (16*1024*1024*TUNEABLE_FACTOR) and then putting something like #ifdef TUNE_LARGE #define TUNEABLE_FACTOR 8 #else #define TUNEABLE_FACTOR 1 #endif That way you get a consistant increase in buffer sizes across all buffers and you can easily add additional tuneable settings in 1 place for other sizes if you want (say you wanted a TUNE_MEDIUM setting). Alternatively, you could make TUNEABLE_FACTOR a global variable (yes I know) and set that at runtime and it would flow to all locations where you want tuneable buffer sizes. Just my 2c worth. Regards, pdg

---