app/crypto-perf: overwrite mbuf when verifying
authorPablo de Lara <pablo.de.lara.guarch@intel.com>
Wed, 4 Oct 2017 03:46:10 +0000 (04:46 +0100)
committerPablo de Lara <pablo.de.lara.guarch@intel.com>
Thu, 12 Oct 2017 14:14:45 +0000 (15:14 +0100)
When running the verify test, mbufs in the pool were
populated with the test vector loaded from a file.
To avoid limiting the number of operations to the pool size,
mbufs will be rewritten with the test vector, before
linking them to the crypto operations.

Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
app/test-crypto-perf/cperf_options_parsing.c
app/test-crypto-perf/cperf_test_verify.c

index 0d4d679..3974c81 100644 (file)
@@ -955,13 +955,6 @@ cperf_options_check(struct cperf_options *options)
                return -EINVAL;
        }
 
-       if (options->test == CPERF_TEST_TYPE_VERIFY &&
-                       options->total_ops > options->pool_sz) {
-               RTE_LOG(ERR, USER1, "Total number of ops must be less than or"
-                               " equal to the pool size.\n");
-               return -EINVAL;
-       }
-
        if (options->test == CPERF_TEST_TYPE_VERIFY &&
                        (options->inc_buffer_size != 0 ||
                        options->buffer_size_count > 1)) {
index 5440ad5..c57d672 100644 (file)
@@ -229,6 +229,34 @@ out:
        return !!res;
 }
 
+static void
+cperf_mbuf_set(struct rte_mbuf *mbuf,
+               const struct cperf_options *options,
+               const struct cperf_test_vector *test_vector)
+{
+       uint32_t segment_sz = options->segment_sz;
+       uint8_t *mbuf_data;
+       uint8_t *test_data =
+                       (options->cipher_op == RTE_CRYPTO_CIPHER_OP_ENCRYPT) ?
+                                       test_vector->plaintext.data :
+                                       test_vector->ciphertext.data;
+       uint32_t remaining_bytes = options->max_buffer_size;
+
+       while (remaining_bytes) {
+               mbuf_data = rte_pktmbuf_mtod(mbuf, uint8_t *);
+
+               if (remaining_bytes <= segment_sz) {
+                       memcpy(mbuf_data, test_data, remaining_bytes);
+                       return;
+               }
+
+               memcpy(mbuf_data, test_data, segment_sz);
+               remaining_bytes -= segment_sz;
+               test_data += segment_sz;
+               mbuf = mbuf->next;
+       }
+}
+
 int
 cperf_verify_test_runner(void *test_ctx)
 {
@@ -299,6 +327,13 @@ cperf_verify_test_runner(void *test_ctx)
                                ops_needed, ctx->sess, ctx->options,
                                ctx->test_vector, iv_offset);
 
+
+               /* Populate the mbuf with the test vector, for verification */
+               for (i = 0; i < ops_needed; i++)
+                       cperf_mbuf_set(ops[i]->sym->m_src,
+                                       ctx->options,
+                                       ctx->test_vector);
+
 #ifdef CPERF_LINEARIZATION_ENABLE
                if (linearize) {
                        /* PMD doesn't support scatter-gather and source buffer