Bug 846240
| Summary: | [FEAT] quick-read should use anonymous fd framework | ||
|---|---|---|---|
| Product: | [Community] GlusterFS | Reporter: | Vidya Sakar <vinaraya> |
| Component: | quick-read | Assignee: | Raghavendra G <rgowdapp> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |
| Severity: | unspecified | Docs Contact: | |
| Priority: | medium | ||
| Version: | mainline | CC: | aavati, amarts, gluster-bugs, jahernan, jdarcy, rfortier, vbellur |
| Target Milestone: | --- | Keywords: | FutureFeature |
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | glusterfs-3.4.0 | Doc Type: | Enhancement |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2013-07-24 17:23:59 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | |||
| Bug Blocks: | 850503, 852576, 858485, 895528 | ||
|
Description
Vidya Sakar
2012-08-07 08:54:43 UTC
is more work than thought for finishing before 3.4.0 timeframe, and hence would like to take it off the dependency list for 3.4.0 http://review.gluster.org/4406 and dependent patchsets posted for review. CHANGE: http://review.gluster.org/4403 (quick-read: refactor) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4404 (fd: filter anonymous FD from fd_lookup() result) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4405 (fd: expose __fd_ctx_del symbol) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4452 (quick-read: various fixes) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4406 (open-behind: translator to perform open calls in background) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4548 (cluster/afr: do complete split-brain check in all the fd based fops) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4483 (performance/open-behind: use anonymous fd for doing fstat and readv) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4520 (call-stub: internal refactor) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4521 (open-behind: propagate errors from ob_wake_cbk) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4553 (testcase for the open-behind xlator when open fails) merged in master by Anand Avati (avati) CHANGE: http://review.gluster.org/4620 (cluster/afr: do complete split-brain check in all the fd based fops) merged in release-3.4 by Vijay Bellur (vbellur) CHANGE: http://review.gluster.org/4634 (testcase for the open-behind xlator when open fails) merged in release-3.4 by Anand Avati (avati) REVIEW: http://review.gluster.org/5150 (tests: fix open-behind test) posted (#1) for review on master by Jeff Darcy (jdarcy) REVIEW: http://review.gluster.org/5807 (performance/open-behind: use anonymous fd for doing fstat and readv) posted (#1) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5808 (open-behind: propagate errors from ob_wake_cbk) posted (#1) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5820 (call-stub: internal refactor) posted (#1) for review on release-3.4 by Vijay Bellur (vbellur) COMMIT: http://review.gluster.org/5820 committed in release-3.4 by Vijay Bellur (vbellur) ------ commit 285de41f1105e18fa7bcf309fe43fb9e6d08508e Author: Anand Avati <avati> Date: Fri Feb 15 17:14:50 2013 -0800 call-stub: internal refactor - re-structure members of call_stub_t with new simpler layout - easier to inspect call_stub_t contents in gdb now - fix a bunch of double unrefs and double frees in cbk stub - change all STACK_UNWIND to STACK_UNWIND_STRICT and thereby fixed a lot of bad params - implement new API call_unwind_error() which can even be called on fop_XXX_stub(), and not necessarily fop_XXX_cbk_stub() Change-Id: Idf979f14d46256af0afb9658915cc79de157b2d7 BUG: 846240 Signed-off-by: Anand Avati <avati> Reviewed-on: http://review.gluster.org/4520 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Jeff Darcy <jdarcy> Reviewed-on: http://review.gluster.org/5820 Reviewed-by: Raghavendra Bhat <raghavendra> REVIEW: http://review.gluster.org/5807 (performance/open-behind: use anonymous fd for doing fstat and readv) posted (#2) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5808 (open-behind: propagate errors from ob_wake_cbk) posted (#2) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5807 (performance/open-behind: use anonymous fd for doing fstat and readv) posted (#3) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5808 (open-behind: propagate errors from ob_wake_cbk) posted (#3) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5807 (performance/open-behind: use anonymous fd for doing fstat and readv) posted (#4) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5808 (open-behind: propagate errors from ob_wake_cbk) posted (#4) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5807 (performance/open-behind: use anonymous fd for doing fstat and readv) posted (#5) for review on release-3.4 by Vijay Bellur (vbellur) REVIEW: http://review.gluster.org/5808 (open-behind: propagate errors from ob_wake_cbk) posted (#5) for review on release-3.4 by Vijay Bellur (vbellur) COMMIT: http://review.gluster.org/5807 committed in release-3.4 by Anand Avati (avati) ------ commit 3438475cfda277fac0de6758ea9f85e3587596b5 Author: Raghavendra Bhat <raghavendra> Date: Tue Feb 19 12:37:04 2013 +0530 performance/open-behind: use anonymous fd for doing fstat and readv Change-Id: I61a3c221e0a15736ab6315e2538c03dac27480a5 BUG: 846240 Signed-off-by: Raghavendra Bhat <raghavendra> Reviewed-on: http://review.gluster.org/4483 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Anand Avati <avati> Reviewed-on: http://review.gluster.org/5807 Reviewed-by: Kaleb KEITHLEY <kkeithle> COMMIT: http://review.gluster.org/5808 committed in release-3.4 by Anand Avati (avati) ------ commit 8e5e836e434889d4b2d37c9caf0abf2393dc198c Author: Anand Avati <avati> Date: Fri Feb 15 17:30:56 2013 -0800 open-behind: propagate errors from ob_wake_cbk If opening fd in background fails, then remember the error and fail all further calls on the fd. Use the newly introduced call_unwind_error() function from call-stub cleanup to fail the future calls. Change-Id: I3b09b7969c98d915abd56590a2777ce833b81813 BUG: 846240 Signed-off-by: Anand Avati <avati> Reviewed-on: http://review.gluster.org/4521 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Raghavendra Bhat <raghavendra> Reviewed-on: http://review.gluster.org/5808 REVIEW: http://review.gluster.org/6505 (backport call-stub: internal refactor) posted (#1) for review on release-3.3 by inevity (roidinev) *** Bug 843075 has been marked as a duplicate of this bug. *** *** Bug 903723 has been marked as a duplicate of this bug. *** |