From 99231d9abe0fd74c7957d25b08c1d1ede4cae5a0 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Fri, 1 Jul 2011 23:02:09 -0400 Subject: nscd cleanups Cleanup and optimize last patch. Add timeout fixes also to the services cache. --- nscd/aicache.c | 5 +++-- nscd/hstcache.c | 5 +++-- nscd/initgrcache.c | 7 ++++--- nscd/pwdcache.c | 5 +++-- nscd/servicescache.c | 11 ++++++++--- 5 files changed, 21 insertions(+), 12 deletions(-) (limited to 'nscd') diff --git a/nscd/aicache.c b/nscd/aicache.c index b1369214ea..8d100f8d31 100644 --- a/nscd/aicache.c +++ b/nscd/aicache.c @@ -512,14 +512,15 @@ next_nip: TEMP_FAILURE_RETRY (send (fd, ¬found, total, MSG_NOSIGNAL)); /* If we cannot permanently store the result, so be it. */ - if (db->negtimeout == 0) + if (__builtin_expect (db->negtimeout == 0, 0)) { /* Mark the old entry as obsolete. */ if (dh != NULL) dh->usable = false; dataset = NULL; } - else if ((dataset = mempool_alloc (db, sizeof (struct dataset) + req->key_len, 1)) != NULL) + else if ((dataset = mempool_alloc (db, (sizeof (struct dataset) + + req->key_len), 1)) != NULL) { dataset->head.allocsize = sizeof (struct dataset) + req->key_len; dataset->head.recsize = total; diff --git a/nscd/hstcache.c b/nscd/hstcache.c index 1f09c0b975..4d68ade98c 100644 --- a/nscd/hstcache.c +++ b/nscd/hstcache.c @@ -142,13 +142,14 @@ cache_addhst (struct database_dyn *db, int fd, request_header *req, all_written = false; /* If we cannot permanently store the result, so be it. */ - if (db->negtimeout == 0) + if (__builtin_expect (db->negtimeout == 0, 0)) { /* Mark the old entry as obsolete. */ if (dh != NULL) dh->usable = false; } - else if ((dataset = mempool_alloc (db, sizeof (struct dataset) + req->key_len, 1)) != NULL) + else if ((dataset = mempool_alloc (db, (sizeof (struct dataset) + + req->key_len), 1)) != NULL) { dataset->head.allocsize = sizeof (struct dataset) + req->key_len; dataset->head.recsize = total; diff --git a/nscd/initgrcache.c b/nscd/initgrcache.c index 7fd564d0bc..4ac9942aeb 100644 --- a/nscd/initgrcache.c +++ b/nscd/initgrcache.c @@ -1,5 +1,5 @@ /* Cache handling for host lookup. - Copyright (C) 2004, 2005, 2006, 2008, 2009, 2011 Free Software Foundation, Inc. + Copyright (C) 2004-2006, 2008, 2009, 2011 Free Software Foundation, Inc. This file is part of the GNU C Library. Contributed by Ulrich Drepper , 2004. @@ -203,13 +203,14 @@ addinitgroupsX (struct database_dyn *db, int fd, request_header *req, MSG_NOSIGNAL)); /* If we cannot permanently store the result, so be it. */ - if (db->negtimeout == 0) + if (__builtin_expect (db->negtimeout == 0, 0)) { /* Mark the old entry as obsolete. */ if (dh != NULL) dh->usable = false; } - else if ((dataset = mempool_alloc (db, sizeof (struct dataset) + req->key_len, 1)) != NULL) + else if ((dataset = mempool_alloc (db, (sizeof (struct dataset) + + req->key_len), 1)) != NULL) { dataset->head.allocsize = sizeof (struct dataset) + req->key_len; dataset->head.recsize = total; diff --git a/nscd/pwdcache.c b/nscd/pwdcache.c index 94e1c52793..49e130c1f0 100644 --- a/nscd/pwdcache.c +++ b/nscd/pwdcache.c @@ -125,13 +125,14 @@ cache_addpw (struct database_dyn *db, int fd, request_header *req, MSG_NOSIGNAL)); /* If we cannot permanently store the result, so be it. */ - if (db->negtimeout == 0) + if (__builtin_expect (db->negtimeout == 0, 0)) { /* Mark the old entry as obsolete. */ if (dh != NULL) dh->usable = false; } - else if ((dataset = mempool_alloc (db, sizeof (struct dataset) + req->key_len, 1)) != NULL) + else if ((dataset = mempool_alloc (db, (sizeof (struct dataset) + + req->key_len), 1)) != NULL) { dataset->head.allocsize = sizeof (struct dataset) + req->key_len; dataset->head.recsize = total; diff --git a/nscd/servicescache.c b/nscd/servicescache.c index b0a2977935..ec5a367db8 100644 --- a/nscd/servicescache.c +++ b/nscd/servicescache.c @@ -108,10 +108,15 @@ cache_addserv (struct database_dyn *db, int fd, request_header *req, written = TEMP_FAILURE_RETRY (send (fd, ¬found, total, MSG_NOSIGNAL)); - dataset = mempool_alloc (db, sizeof (struct dataset) + req->key_len, - 1); /* If we cannot permanently store the result, so be it. */ - if (dataset != NULL) + if (__builtin_expect (db->negtimeout == 0, 0)) + { + /* Mark the old entry as obsolete. */ + if (dh != NULL) + dh->usable = false; + } + else if ((dataset = mempool_alloc (db, (sizeof (struct dataset) + + req->key_len), 1)) != NULL) { dataset->head.allocsize = sizeof (struct dataset) + req->key_len; dataset->head.recsize = total; -- cgit 1.4.1