Date: Fri, 20 Feb 2004 20:08:14 -0500 (EST) From: Daniel Eischen <eischen@vigrid.com> To: "Brian F. Feldman" <green@FreeBSD.org> Cc: current@FreeBSD.org Subject: Re: Testers wanted: reentrant resolver Message-ID: <Pine.GSO.4.10.10402201952560.19828-100000@pcnet5.pcnet.com> In-Reply-To: <200402210048.i1L0mVGW014390@green.homeunix.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 20 Feb 2004, Brian F. Feldman wrote: > Daniel Eischen <eischen@vigrid.com> wrote: > > On Fri, 20 Feb 2004, Brian F. Feldman wrote: > > > > > Daniel Eischen <eischen@vigrid.com> wrote: > > > > Ugh, can you put h_errno inside the per-thread res stuff. > > > > We shouldn't need to have to add special hooks in the > > > > threads libraries for this. > > > > > > Please explain what you're saying further. On correctly-threaded operating > > > systems, h_errno is just like errno -- and I made it act EXACTLY as errno > > > acts, and is per-thread storage for everything but the first thread. It's > > > absolutely necessary if we want to return the correct errors; even if > > > everything else in the world is totally reentrant, if h_errno isn't, the > > > wrong errors can be returned! What "special hooks" do you mean? There's no > > > way to not change probably hundreds of lines of code without actually doing > > > the work to make h_errno thread-safe. It's the only proper thing to do. > > > > The implementation of __h_errno() need not depend on something > > special stuffed in struct pthread. Use thread-local storage > > (pthread_getspecific()) like you did for the res_send_private > > stuff. Especially since these interfaces should be deprecated > > in favor of what looks to be BIND 8.2.2 interfaces (according > > to the Solaris man pages). > > Other APIs have the option of failing. __h_errno() does not have the option > of failing, so what do I do if pthread_key_create() fails? Also, if > malloc() fails each time pthread_getspecific() returns NULL for the thread? The API isn't thread-safe by design, so if malloc() fails, just use the global errno. A better design would be to add the thread-safe interfaces I mention above, and have the non-thread-safe interfaces first do the pthread_once(), pthread_[gs]etspecific() thing and then call the thread-safe interfaces. Since the malloc() will be the first thing in the entry point, you can fail right away: /* MT-safe */ int res_nsend(res_state statp, const u_char *msg, int msglen, u_char *answer, int anslen) { ... } /* Not MT-safe */ int res_send(const u_char *msg, int msglen, u_char *answer, int anslen) { res_state statp; if (pthread_once(...)) { ... } statp = pthread_getspecific(res_key); if (statp == NULL) { /* Allocate statp. */ pthread_setspecific(res_key, statp); } return (res_nsend(statp, msg, msglen, answer, anslen)); } -- Dan Eischen
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.GSO.4.10.10402201952560.19828-100000>