Skip to content

Commit 2fdc824

Browse files
committed
DAOS-19116 object: do not log IO RPC DER_TIMEDOUT as ERROR - b28
DAOS client logic will resend IO request if hit DER_TIMEDOUT. That is invisible to the application and not fatal. Logging it as ERROR is confused and may fill the log file under some bad case, such as server overloaded for very long time. Signed-off-by: Fan Yong <fan.yong@hpe.com>
1 parent 0d6cc6e commit 2fdc824

1 file changed

Lines changed: 8 additions & 8 deletions

File tree

src/object/cli_shard.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -787,14 +787,14 @@ dc_rw_cb(tse_task_t *task, void *arg)
787787
* If any failure happens inside Cart, let's reset failure to
788788
* TIMEDOUT, so the upper layer can retry.
789789
*/
790-
D_ERROR(DF_UOID
791-
" (%s) RPC %p (%d) to %d/%d, flags %lx/%x, task %p failed, %s, TX " DF_DTI
792-
": " DF_RC "\n",
793-
DP_UOID(orw->orw_oid), is_ec_obj ? "EC" : "non-EC", rw_args->rpc, opc,
794-
rw_args->rpc->cr_ep.ep_rank, rw_args->rpc->cr_ep.ep_tag,
795-
(unsigned long)orw->orw_api_flags, orw->orw_flags, task,
796-
orw->orw_bulks.ca_arrays || orw->orw_bulks.ca_count ? "DMA" : "non-DMA",
797-
DP_DTI(&orw->orw_dti), DP_RC(ret));
790+
DL_CDEBUG(ret != -DER_TIMEDOUT, DLOG_ERR, DB_IO, ret,
791+
DF_UOID " (%s) RPC %p (%d) "
792+
" to %d/%d, flags %lx/%x, task %p failed, %s, TX " DF_DTI,
793+
DP_UOID(orw->orw_oid), is_ec_obj ? "EC" : "non-EC", rw_args->rpc, opc,
794+
rw_args->rpc->cr_ep.ep_rank, rw_args->rpc->cr_ep.ep_tag,
795+
(unsigned long)orw->orw_api_flags, orw->orw_flags, task,
796+
orw->orw_bulks.ca_arrays || orw->orw_bulks.ca_count ? "DMA" : "non-DMA",
797+
DP_DTI(&orw->orw_dti));
798798

799799
D_GOTO(out, ret);
800800
}

0 commit comments

Comments
 (0)