Bug 1228157 - Provide and use a common way to do reference counting of (internal) structures
Summary: Provide and use a common way to do reference counting of (internal) structures
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: core
Version: mainline
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
Assignee: Niels de Vos
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1226717 1235628 1235939
TreeView+ depends on / blocked
 
Reported: 2015-06-04 10:18 UTC by Niels de Vos
Modified: 2016-06-16 13:08 UTC (History)
3 users (show)

Fixed In Version: glusterfs-3.8rc2
Doc Type: Enhancement
Doc Text:
Clone Of:
: 1235628 1235939 (view as bug list)
Environment:
Last Closed: 2016-06-16 13:08:45 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:

Attachments (Terms of Use)

Description Niels de Vos 2015-06-04 10:18:17 UTC 
Description of problem:
Many structures need reference counting, and each structure implements its own way of doing it. This is confusing and makes things more difficult to understand than it necessary.

Version-Release number of selected component (if applicable):
mainline

I am not a grape farmer:
http://thread.gmane.org/gmane.comp.file-systems.gluster.devel/11202
Comment 1 Anand Avati 2015-06-04 10:48:57 UTC 
REVIEW: http://review.gluster.org/11022 (core: add "struct gf_ref" for common refcounting strutures) posted (#3) for review on master by Niels de Vos (ndevos)
Comment 2 Anand Avati 2015-06-04 10:49:46 UTC 
REVIEW: http://review.gluster.org/11022 (core: add "gf_ref_t" for common refcounting structures) posted (#4) for review on master by Niels de Vos (ndevos)
Comment 3 Anand Avati 2015-06-04 11:19:00 UTC 
REVIEW: http://review.gluster.org/11022 (core: add "gf_ref_t" for common refcounting structures) posted (#5) for review on master by Niels de Vos (ndevos)
Comment 4 Anand Avati 2015-06-04 13:45:13 UTC 
REVIEW: http://review.gluster.org/11022 (core: add "gf_ref_t" for common refcounting structures) posted (#6) for review on master by Niels de Vos (ndevos)
Comment 5 Anand Avati 2015-06-04 23:15:08 UTC 
REVIEW: http://review.gluster.org/11022 (core: add "gf_ref_t" for common refcounting structures) posted (#7) for review on master by Niels de Vos (ndevos)
Comment 6 Anand Avati 2015-06-16 19:58:00 UTC 
REVIEW: http://review.gluster.org/11022 (core: add "gf_ref_t" for common refcounting structures) posted (#8) for review on master by Niels de Vos (ndevos)
Comment 7 Anand Avati 2015-06-16 20:03:23 UTC 
REVIEW: http://review.gluster.org/11022 (core: add "gf_ref_t" for common refcounting structures) posted (#9) for review on master by Niels de Vos (ndevos)
Comment 8 Anand Avati 2015-06-17 09:29:26 UTC 
REVIEW: http://review.gluster.org/11022 (core: add "gf_ref_t" for common refcounting structures) posted (#10) for review on master by Niels de Vos (ndevos)
Comment 9 Niels de Vos 2016-06-16 13:08:45 UTC 
This bug is getting closed because a release has been made available that should address the reported issue. In case the problem is still not fixed with glusterfs-3.8.0, please open a new bug report.

glusterfs-3.8.0 has been announced on the Gluster mailinglists [1], packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist [2] and the update infrastructure for your distribution.

[1] http://blog.gluster.org/2016/06/glusterfs-3-8-released/
[2] http://thread.gmane.org/gmane.comp.file-systems.gluster.user
Note You need to log in before you can comment on or make changes to this bug.