From fade46a42bbc0abb7f1db107f6ad0827d4442cc3 Mon Sep 17 00:00:00 2001 From: Alex Zhuravlev Date: Mon, 14 Dec 2015 18:53:10 +0300 Subject: [PATCH] LU-7408 target: comment as per inspection add a comment as per the original patch's inspection. Change-Id: I391e5cefa6384f5c2c1c29efc958c9a873a5e7bc Signed-off-by: Alex Zhuravlev Reviewed-on: http://review.whamcloud.com/17578 Tested-by: Jenkins Reviewed-by: Andreas Dilger Tested-by: Maloo Reviewed-by: James Simmons --- lustre/target/tgt_lastrcvd.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/lustre/target/tgt_lastrcvd.c b/lustre/target/tgt_lastrcvd.c index bf44695..74e28f8 100644 --- a/lustre/target/tgt_lastrcvd.c +++ b/lustre/target/tgt_lastrcvd.c @@ -1729,8 +1729,15 @@ int tgt_txn_start_cb(const struct lu_env *env, struct thandle *th, return rc; if (tgt_is_multimodrpcs_client(tsi->tsi_exp)) { - tti->tti_off = atomic_read(&tgt->lut_num_clients) * 8 - * sizeof(struct lsd_reply_data); + /* + * Use maximum possible file offset for declaration to ensure + * ZFS will reserve enough credits for a write anywhere in this + * file, since we don't know where in the file the write will be + * because a replay slot has not been assigned. This should be + * replaced by dmu_tx_hold_append() when available. + */ + tti->tti_off = atomic_read(&tgt->lut_num_clients) * 8 * + sizeof(struct lsd_reply_data); tti->tti_buf.lb_buf = NULL; tti->tti_buf.lb_len = sizeof(struct lsd_reply_data); dto = dt_object_locate(tgt->lut_reply_data, th->th_dev); -- 1.8.3.1