From f27a9e92b2507a18872008f49f9bf0e09009f1cc Mon Sep 17 00:00:00 2001 From: Frank Zago Date: Mon, 26 Jan 2015 11:53:59 -0600 Subject: [PATCH] LU-6159 hsm: add CL_CLOSE to default changelog mask There's no point in ignoring CL_CLOSE by default in changelogs. Robinhood needs these events else the database quickly becomes out of sync. So let's have it by default. Note that CL_CLOSE is not issued when the file was opened in read-only mode. Signed-off-by: frank zago Change-Id: Ie5f42bc4413259e5079801a204e15125cde0c48b Reviewed-on: http://review.whamcloud.com/13526 Tested-by: Jenkins Reviewed-by: Henri Doreau Tested-by: Maloo Reviewed-by: Andreas Dilger Reviewed-by: Oleg Drokin --- lustre/include/lustre/lustre_idl.h | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lustre/include/lustre/lustre_idl.h b/lustre/include/lustre/lustre_idl.h index 2ccb7ce..b1e5284 100644 --- a/lustre/include/lustre/lustre_idl.h +++ b/lustre/include/lustre/lustre_idl.h @@ -3354,8 +3354,10 @@ struct llog_size_change_rec { #define CHANGELOG_MINMASK (1 << CL_MARK) /** bits covering all \a changelog_rec_type's */ #define CHANGELOG_ALLMASK 0XFFFFFFFF -/** default \a changelog_rec_type mask */ -#define CHANGELOG_DEFMASK CHANGELOG_ALLMASK & ~(1 << CL_ATIME | 1 << CL_CLOSE) +/** default \a changelog_rec_type mask. Allow all of them, except + * CL_ATIME since it can really be time consuming, and not necessary + * under normal use. */ +#define CHANGELOG_DEFMASK (CHANGELOG_ALLMASK & ~(1 << CL_ATIME)) /* changelog llog name, needed by client replicators */ #define CHANGELOG_CATALOG "changelog_catalog" -- 1.8.3.1