Red Hat Bugzilla – Bug 1293227
Minor improvements and code cleanup for locks translator
Last modified: 2016-06-16 09:51:54 EDT
Description of problem:
Use this bug to file changes that affect source files under xlators/features/locks directory. Real bugs (problem reports, crashes) should have their own bugzilla entry for related patches.
Acceptable changes include:
-Improve code readability
-Remove dead code
-Better error reporting
- . . .
REVIEW: http://review.gluster.org/13033 (features/locks: Remove unneeded function parameter) posted (#1) for review on master by Anoop C S (email@example.com)
REVIEW: http://review.gluster.org/13033 (features/locks: Remove unneeded function parameter) posted (#2) for review on master by Anoop C S (firstname.lastname@example.org)
COMMIT: http://review.gluster.org/13033 committed in master by Jeff Darcy (email@example.com)
Author: Anoop C S <firstname.lastname@example.org>
Date: Sun Dec 20 21:47:37 2015 +0530
features/locks: Remove unneeded function parameter
We use list_del_init() inside __delete_lock()
to remove a lock from inode's lock list where
pl_inode_t is not required at all. This patch
removes pl_inode_t from list of parameters
required for __delete_lock().
Signed-off-by: Anoop C S <email@example.com>
Smoke: Gluster Build System <firstname.lastname@example.org>
CentOS-regression: Gluster Build System <email@example.com>
NetBSD-regression: NetBSD Build System <firstname.lastname@example.org>
Reviewed-by: Jeff Darcy <email@example.com>
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see firstname.lastname@example.org with any questions
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 , packages for several distributions should become available in the near future. Keep an eye on the Gluster Users mailinglist  and the update infrastructure for your distribution.