glusterd: Optimize glusterd_volinfo_find function

[mohit84]

Optimize glusterd_volinfo_find function, the function is use by multiple places to fetch volinfo object from a volumes list.
In case of brick_mux environment while too many volumes are configured it is not useful to compare everytime volume
based on volname.Instead of comparing again and again based on volname by strcmp save hash of volname in volinfo
object and compare the hash values.

int32_t
glusterd_volinfo_find(const char *volname, glusterd_volinfo_t **volinfo)
{
glusterd_volinfo_t *tmp_volinfo = NULL;
int32_t ret = -1;
xlator_t *this = NULL;
glusterd_conf_t *priv = NULL;

GF_ASSERT(volname);
this = THIS;
GF_ASSERT(this);

priv = this->private;
GF_ASSERT(priv);

cds_list_for_each_entry(tmp_volinfo, &priv->volumes, vol_list)
{
    if (!strcmp(tmp_volinfo->volname, volname)) {
        gf_msg_debug(this->name, 0, "Volume %s found", volname);
        ret = 0;
        *volinfo = tmp_volinfo;
        break;
    }
}

gf_msg_debug(this->name, 0, "Returning %d", ret);
return ret;

}

[stale]

Thank you for your contributions.
Noticed that this issue is not having any activity in last ~6 months! We are marking this issue as stale because it has not had recent activity.
It will be closed in 2 weeks if no one responds with a comment here.

[stale]

Closing this issue as there was no update since my last update on issue. If this is an issue which is still valid, feel free to open it.