Description of problem: CC tier.lo tier.c:1922:13: warning: variable 'promote_thread' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized] if (ret) { ^~~ tier.c:1949:23: note: uninitialized use occurs here pthread_join (promote_thread, NULL); ^~~~~~~~~~~~~~ tier.c:1922:9: note: remove the 'if' if its condition is always false if (ret) { ^~~~~~~~~~ tier.c:1897:33: note: initialize the variable 'promote_thread' to silence this warning pthread_t promote_thread; also release-3.8 and release-3.7 Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
see http://download.gluster.org/pub/gluster/glusterfs/static-analysis/master/glusterfs-clang/2016-05-10-3887804d/clang-master.txt correction, warning does not exist in release-3.7
REVIEW: http://review.gluster.org/14297 (Fix uninitialized memory use for clang error for variables pthread_promote and pthread_demote in function tier_start().) posted (#1) for review on master by Dan Lambright (dlambrig)
REVIEW: http://review.gluster.org/14297 (cluster/tier: Fix uninitialized memory use for clang error.) posted (#2) for review on master by Dan Lambright (dlambrig)
REVIEW: http://review.gluster.org/14297 (cluster/tier: Fix uninitialized memory use for clang error.) posted (#3) for review on master by Dan Lambright (dlambrig)
Patch https://review.gluster.org/#/c/glusterfs/+/21331/ removes tier functionality from GlusterFS. https://bugzilla.redhat.com/show_bug.cgi?id=1642807 is used as the tracking bug for this. Recommendation is to convert your tier volume to regular volume (either replicate, ec, or plain distribute) with "tier detach" command before upgrade, and use backend features like dm-cache etc to utilize the caching from backend to provide better performance and functionality.