diff options
| author | Nithya Balachandran <nbalacha@redhat.com> | 2015-05-19 23:27:35 +0530 |
|---|---|---|
| committer | Raghavendra G <rgowdapp@redhat.com> | 2015-06-02 22:36:17 -0700 |
| commit | 9684b90526d03a15d451e341521d7df44adae73e (patch) | |
| tree | 4a4f342391a111d3c3fe1782bdc8ecd8dea1a903 /xlators/cluster/dht/src/dht-mem-types.h | |
| parent | b6eda067d2e2a0b56718ea71522f6c7b06a09f13 (diff) | |
cluster/dht: fix incorrect dst subvol info in inode_ctx
Stashing additional information in the inode_ctx to help
decide whether the migration information is stale, which could
happen if a file was migrated several times but FOPs only detected
the P1 migration phase. If no FOP detects the P2 phase, the inode
ctx1 is never reset.
We now save the src subvol as well as the dst subvol in the
inode ctx. The src subvol is the subvol on which the FOP was sent
when the mig info was set in the inode ctx. This information is
considered stale if:
1. The subvol on which the current FOP is sent is the same as
the dst subvol in the ctx
2. The subvol on which the current FOP is sent is not the same
as the src subvol in the ctx
This does not handle the case where the same file might have been
renamed such that the src subvol is the same but the dst subvol
is different. However, that is unlikely to happen very often.
Change-Id: I05a2e9b107ee64750c7ca629aee03b03a02ef75f
BUG: 1142423
Signed-off-by: Nithya Balachandran <nbalacha@redhat.com>
Reviewed-on: http://review.gluster.org/10834
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Raghavendra G <rgowdapp@redhat.com>
Tested-by: Raghavendra G <rgowdapp@redhat.com>
Diffstat (limited to 'xlators/cluster/dht/src/dht-mem-types.h')
| -rw-r--r-- | xlators/cluster/dht/src/dht-mem-types.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/xlators/cluster/dht/src/dht-mem-types.h b/xlators/cluster/dht/src/dht-mem-types.h index 46028e6d9e0..85e5baed62c 100644 --- a/xlators/cluster/dht/src/dht-mem-types.h +++ b/xlators/cluster/dht/src/dht-mem-types.h @@ -33,6 +33,7 @@ enum gf_dht_mem_types_ { gf_dht_mt_dirent_t, gf_dht_mt_container_t, gf_dht_mt_octx_t, + gf_dht_mt_miginfo_t, gf_dht_mt_end }; #endif |