Browse Source

Use the one list for proxies in the generator, separating only when needed

master
Con Kolivas 10 years ago
parent
commit
1818d867a0
  1. 17
      src/generator.c

17
src/generator.c

@ -138,8 +138,7 @@ struct proxy_instance {
struct generator_data { struct generator_data {
ckpool_t *ckp; ckpool_t *ckp;
mutex_t lock; /* Lock protecting linked lists */ mutex_t lock; /* Lock protecting linked lists */
proxy_instance_t *proxies; /* Hash list of all global proxies */ proxy_instance_t *proxies; /* Hash list of all proxies */
proxy_instance_t *user_proxies; /* Hash list of all user proxies */
proxy_instance_t *dead_proxies; /* Disabled proxies */ proxy_instance_t *dead_proxies; /* Disabled proxies */
int proxy_notify_id; // Globally increasing notify id int proxy_notify_id; // Globally increasing notify id
@ -1298,8 +1297,6 @@ static proxy_instance_t *proxy_by_id(gdata_t *gdata, const int id)
mutex_lock(&gdata->lock); mutex_lock(&gdata->lock);
HASH_FIND_INT(gdata->proxies, &id, proxi); HASH_FIND_INT(gdata->proxies, &id, proxi);
if (!proxi)
HASH_FIND_INT(gdata->user_proxies, &id, proxi);
mutex_unlock(&gdata->lock); mutex_unlock(&gdata->lock);
return proxi; return proxi;
@ -2018,8 +2015,8 @@ static void *userproxy_recv(void *arg)
int ret; int ret;
mutex_lock(&gdata->lock); mutex_lock(&gdata->lock);
HASH_ITER(hh, gdata->user_proxies, proxy, tmpproxy) { HASH_ITER(hh, gdata->proxies, proxy, tmpproxy) {
if (!proxy->alive) { if (!proxy->global && !proxy->alive) {
proxy->epfd = epfd; proxy->epfd = epfd;
reconnect_proxy(proxy); reconnect_proxy(proxy);
} }
@ -2098,7 +2095,7 @@ static proxy_instance_t *wait_best_proxy(ckpool_t *ckp, gdata_t *gdata)
mutex_lock(&gdata->lock); mutex_lock(&gdata->lock);
HASH_ITER(hh, gdata->proxies, proxi, tmp) { HASH_ITER(hh, gdata->proxies, proxi, tmp) {
if (proxi->disabled) if (proxi->disabled || !proxi->global)
continue; continue;
if (proxi->alive || subproxies_alive(proxi)) { if (proxi->alive || subproxies_alive(proxi)) {
if (!ret || proxi->id < ret->id) if (!ret || proxi->id < ret->id)
@ -2128,8 +2125,8 @@ static void send_list(gdata_t *gdata, const int sockd)
mutex_lock(&gdata->lock); mutex_lock(&gdata->lock);
HASH_ITER(hh, gdata->proxies, proxy, tmp) { HASH_ITER(hh, gdata->proxies, proxy, tmp) {
JSON_CPACK(val, "{si,ss,ss,sf,sb,sb,sb,si}", JSON_CPACK(val, "{si,sb,si,ss,ss,sf,sb,sb,sb,si}",
"id", proxy->id, "id", proxy->id, "global", proxy->global, "userid", proxy->userid,
"auth", proxy->auth, "pass", proxy->pass, "auth", proxy->auth, "pass", proxy->pass,
"diff", proxy->diff, "notified", proxy->notified, "diff", proxy->diff, "notified", proxy->notified,
"disabled", proxy->disabled, "alive", proxy->alive, "disabled", proxy->disabled, "alive", proxy->alive,
@ -2208,7 +2205,7 @@ static proxy_instance_t *__add_userproxy(ckpool_t *ckp, gdata_t *gdata, const in
proxy->ckp = ckp; proxy->ckp = ckp;
mutex_init(&proxy->notify_lock); mutex_init(&proxy->notify_lock);
mutex_init(&proxy->share_lock); mutex_init(&proxy->share_lock);
HASH_ADD_INT(gdata->user_proxies, id, proxy); HASH_ADD_INT(gdata->proxies, id, proxy);
return proxy; return proxy;
} }

Loading…
Cancel
Save