Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

Commit

Permalink
CORTX-34311 motr:converted logs from WARN to DEBUG related to DI (#2162)
Browse files Browse the repository at this point in the history
DI logs are creating lot of unnecessary warnigs,
so converted them from WARN to DEBUG

Signed-off-by: Rajat Patil <rajat.r.patil@seagate.com>
  • Loading branch information
rajatpatil98 authored Sep 19, 2022
1 parent 33e87a1 commit c778392
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions ioservice/io_foms.c
Original file line number Diff line number Diff line change
Expand Up @@ -2168,14 +2168,14 @@ static int stob_io_create(struct m0_fom *fom)
rwfop->crw_cksum_size) {
v_nr = fom_obj->fcrw_io.si_stob.iv_vec.v_nr;
c_nr = rwfop->crw_ivec.ci_nr;
M0_LOG(M0_WARN,"Write Disabling DI for Ext0: %"PRIi64
M0_LOG(M0_DEBUG,"Write Disabling DI for Ext0: %"PRIi64
" ExtNr: %"PRIi64" Count0: %"PRIi64
" Vnr: %"PRIi32" CountEnd: %"PRIi64,
fom_obj->fcrw_io.si_stob.iv_index[0],
fom_obj->fcrw_io.si_stob.iv_index[v_nr-1],
fom_obj->fcrw_io.si_stob.iv_vec.v_count[0], v_nr,
fom_obj->fcrw_io.si_stob.iv_vec.v_count[v_nr-1]);
M0_LOG(M0_WARN,"CRW IVEC for Ext0: %"PRIi64 " %"PRIi64
M0_LOG(M0_DEBUG,"CRW IVEC for Ext0: %"PRIi64 " %"PRIi64
" ExtNr: %d Count0: %"PRIi64
" CountEnd: %"PRIi64,
rwfop->crw_ivec.ci_iosegs[0].ci_index,
Expand Down
8 changes: 4 additions & 4 deletions motr/io_req_fop.c
Original file line number Diff line number Diff line change
Expand Up @@ -119,15 +119,15 @@ static void di_debug_log_print(struct target_ioreq *tioreq,
{
int cdi;
uint32_t v_nr = tioreq->ti_goff_ivec.iv_vec.v_nr;
M0_LOG(M0_WARN,"- FOP details Ext0: %" PRIi64
M0_LOG(M0_DEBUG,"- FOP details Ext0: %" PRIi64
" ExtN: %"PRIi64 " Count0: %" PRIi64
" Vnr: %"PRIi32" CountEnd: %" PRIi64 ,
tioreq->ti_goff_ivec.iv_index[0],
tioreq->ti_goff_ivec.iv_index[v_nr-1],
tioreq->ti_goff_ivec.iv_vec.v_count[0], v_nr,
tioreq->ti_goff_ivec.iv_vec.v_count[v_nr-1]);
for (cdi = 0; cdi < irfop->irf_cksum_data.cd_num_units; cdi++)
M0_LOG(M0_WARN,"- %d. FOP DU details [%s] [PG Idx:%" PRIu64
M0_LOG(M0_DEBUG,"- %d. FOP DU details [%s] [PG Idx:%" PRIu64
"][Unit Idx:%" PRIu64 "]", cdi + 1,
irfop->irf_pattr == PA_DATA ? "D" : "P",
(uint64_t)(irfop->irf_cksum_data.cd_idx[cdi].ci_pg_idx +
Expand Down Expand Up @@ -165,7 +165,7 @@ static int application_checksum_process(struct m0_op_io *ioo,
}
cksum_size = m0_cksum_get_size(cksum_type);
if (cksum_size == 0) {
M0_LOG(M0_WARN, "Skipping DI for PI Type: %d Size: %d",
M0_LOG(M0_DEBUG, "Skipping DI for PI Type: %d Size: %d",
cksum_type,cksum_size);
di_debug_log_print(ti, irfop, ioo);
return rc;
Expand Down Expand Up @@ -344,7 +344,7 @@ static void io_bottom_half(struct m0_sm_group *grp, struct m0_sm_ast *ast)
else if (m0__obj_is_di_enabled(ioo) &&
irfop->irf_cksum_data.cd_num_units &&
(ioo->ioo_oo.oo_oc.oc_op.op_code == M0_OC_READ)) {
M0_LOG(M0_WARN,"No DI data received for :");
M0_LOG(M0_DEBUG,"No DI data received for :");
di_debug_log_print(tioreq, irfop, ioo);
}
}
Expand Down

0 comments on commit c778392

Please sign in to comment.