Re: [dpdk-dev] [PATCH v3 06/14] test-bbdev: support HARQ validation

2020-03-25 Thread Chautru, Nicolas
From: Akhil Goyal >> +/* ignore parity mismatch false alarms for long iterations */ >> +{ >> +if (!(expected_status & (1 << RTE_BBDEV_SYNDROME_ERROR)) > >What is the need of these extra braces. > Missing check added now on new patchset. Thanks. >> + >> +/* Compute K0 for a

Re: [dpdk-dev] [PATCH v3 06/14] test-bbdev: support HARQ validation

2020-03-25 Thread Akhil Goyal
rte_bbdev_dec_op *op, > unsigned int order_idx, const int expected_status) > { > - TEST_ASSERT(op->status == expected_status, > + int status = op->status; > + /* ignore parity mismatch false alarms for long iterations */ > + { > + if (!(expected_status & (