[Bugs] [Bug 1191030] Use rcu to protect concurrent access to data structures in GlusterD

bugzilla at redhat.com bugzilla at redhat.com
Wed Mar 4 07:50:28 UTC 2015


https://bugzilla.redhat.com/show_bug.cgi?id=1191030



--- Comment #19 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/9624 committed in master by Krishnan
Parthasarathi (kparthas at redhat.com) 
------
commit 673ba2659cebe22ee30c43f9fb080f330150f55e
Author: Kaushal M <kaushal at redhat.com>
Date:   Tue Jan 6 18:23:41 2015 +0530

    glusterd: Replace libglusterfs lists with liburcu lists

    This patch replaces usage of the libglusterfs lists data structures and
    API in glusterd with the lists data structures and API from liburcu. The
    liburcu data structes and APIs are a drop-in replacement for
    libglusterfs lists.

    All usages have been changed to keep the code consistent, and free from
    confusion.

    NOTE: glusterd_conf_t->xprt_list still uses the libglusterfs data
    structures and API, as it holds rpc_transport_t objects, which is not a
    part of glusterd and is not being changed in this patch.

    This change was developed on the git branch at [1]. This commit is a
    combination of the following commits on the development branch.
      6dac576 Replace libglusterfs lists with liburcu lists
      a51b5ab Fix compilation issues
      d98a06f Fix merge issues
      a5d918e Remove merge remnant
      1cca113 More style cleanup
      1917be3 Address review comments on 9624/1
      8d10f13 Use cds_lists for glusterd_svc_t
      524ad5d Add rculist header in glusterd-conn-helper.c
      646f294 glusterd: add list_add_order API honouring rcu

    [1]: https://github.com/kshlm/glusterfs/tree/urcu

    Change-Id: Ic613c5b6e496a677b9d3de15fc042a0492109fb0
    BUG: 1191030
    Signed-off-by: Kaushal M <kaushal at redhat.com>
    Signed-off-by: Krishnan Parthasarathi <kparthas at redhat.com>
    Reviewed-on: http://review.gluster.org/9624
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Atin Mukherjee <amukherj at redhat.com>
    Reviewed-by: Gaurav Kumar Garg <ggarg at redhat.com>
    Reviewed-by: Anand Nekkunti <anekkunt at redhat.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ftqIn2NS5Y&a=cc_unsubscribe


More information about the Bugs mailing list