From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id AD712A2EEB for ; Fri, 13 Sep 2019 12:28:56 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C74841F0A8; Fri, 13 Sep 2019 12:28:55 +0200 (CEST) Received: from mail-io1-f66.google.com (mail-io1-f66.google.com [209.85.166.66]) by dpdk.org (Postfix) with ESMTP id 121461F09F for ; Fri, 13 Sep 2019 12:28:55 +0200 (CEST) Received: by mail-io1-f66.google.com with SMTP id f4so61210103ion.2 for ; Fri, 13 Sep 2019 03:28:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=VipGJqLvqPgswXUzGTfOJGHUpBHHq7YiTeFGIO3ROZU=; b=m6nPX+IPhgupQkWEiCykmXAo+6KMq92ItDUPAnntr42oJNOqyG7+km//1vckNdf0Po 1aQXAAX7BlxhCKExbpp/QiwmGNkxk9l+AsuNxWsmVBWrcxeG+12CB8MUFBOpygf/POVv Gql7pATE2tUWVGCk1CXyGCDrNZBOXbSITdLDkR/UCNVcaNVLoyDdoFeXkpj9wc5pskF4 xhDirFTppCQ7kwiHxlSVfSQWn1+EXJQASZAGi8haTObcQaNA0py1N1AZcEJd7A1KJrga X8xdm5XOIp8IS7yfhXi3+TGJNuvihFvyxkUP8t8uQQ0wNC9AhgD3Rd8+bUsKqTCpmPPS 422w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=VipGJqLvqPgswXUzGTfOJGHUpBHHq7YiTeFGIO3ROZU=; b=VMxUFeAbrt/7H67yITWrcc6A9jUKRjfZRF6il+b3cr4my0PgpVWiuvUoBQMckcwidi 50JnmWRVRFS3/PaekGtqKbpJPW0TxAuEa3gzDwmWgkm/JVb2iN2M3NZpDTfcMCwkZqHb aOSvVS/T5DVxLx+jl/qs8S9T+Ka7TFH2XYXzMut8R5IOBpN6Qex2UYgzSCNq1HylKP2Z wBFIlay+P5W60yQMVyyPoBZVC4fWtIAK4fUzGcigLxtkP3uKqsG5Gx0cTwtU8xUgnVZ2 P0HKrc1yixkS4aghvIIEjrAtIm8N2uOWh5v4MBHOYMk3EXq9em35sGD/V24Dyex0CuJo PrLA== X-Gm-Message-State: APjAAAXVOqOBIcT5zVNoNAgB2oo84CQPhDhLTTpKv+g16s8QSyq3bbZn +Q+dUm/Jos8ffBCM+WXy+l0fwFrtXUp442biv/Q= X-Google-Smtp-Source: APXvYqz8wgkwOv/XmpwmefeQCHDLtG4suBF65eUKS0v5KDXi7NApyYUsWtbLz1/G4VtrWoKGnXFb/eSmrBIk57yZOPE= X-Received: by 2002:a6b:c895:: with SMTP id y143mr7796569iof.271.1568370534073; Fri, 13 Sep 2019 03:28:54 -0700 (PDT) MIME-Version: 1.0 References: <20190906104228.15271-1-hemant.agrawal@nxp.com> <5612CB344B05EE4F95FC5B729939F78079540FFE@PGSMSX102.gar.corp.intel.com> In-Reply-To: <5612CB344B05EE4F95FC5B729939F78079540FFE@PGSMSX102.gar.corp.intel.com> From: Jerin Jacob Date: Fri, 13 Sep 2019 15:58:32 +0530 Message-ID: To: "Gujjar, Abhinandan S" Cc: Hemant Agrawal , "dev@dpdk.org" , "jerinj@marvell.com" Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH] test/event_crypto: add check for session init failure X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, Sep 12, 2019 at 8:40 PM Gujjar, Abhinandan S wrote: > > Acked-by: abhinandan.gujjar@intel.com Changed to Acked-by: Abhinandan Gujjar and Applied to dpdk-next-eventdev/master. Thanks. > > -----Original Message----- > > From: Hemant Agrawal > > Sent: Friday, September 6, 2019 4:12 PM > > To: dev@dpdk.org > > Cc: jerinj@marvell.com; Gujjar, Abhinandan S ; > > Hemant Agrawal > > Subject: [PATCH] test/event_crypto: add check for session init failure > > > > Mismatch in algo or sec capability can cause session to fail. > > This patch handle it and return error timely. > > > > Signed-off-by: Hemant Agrawal > > --- > > app/test/test_event_crypto_adapter.c | 7 +++++-- > > 1 file changed, 5 insertions(+), 2 deletions(-) > > > > diff --git a/app/test/test_event_crypto_adapter.c > > b/app/test/test_event_crypto_adapter.c > > index 73655020d..e7bf80978 100644 > > --- a/app/test/test_event_crypto_adapter.c > > +++ b/app/test/test_event_crypto_adapter.c > > @@ -199,8 +199,9 @@ test_op_forward_mode(uint8_t session_less) > > TEST_ASSERT_NOT_NULL(sess, "Session creation failed\n"); > > > > /* Create Crypto session*/ > > - rte_cryptodev_sym_session_init(TEST_CDEV_ID, sess, > > + ret = rte_cryptodev_sym_session_init(TEST_CDEV_ID, sess, > > &cipher_xform, params.session_priv_mpool); > > + TEST_ASSERT_SUCCESS(ret, "Failed to init session\n"); > > > > ret = rte_event_crypto_adapter_caps_get(TEST_ADAPTER_ID, > > evdev, &cap); > > @@ -398,8 +399,10 @@ test_op_new_mode(uint8_t session_less) > > rte_cryptodev_sym_session_set_user_data(sess, > > &m_data, sizeof(m_data)); > > } > > - rte_cryptodev_sym_session_init(TEST_CDEV_ID, sess, > > + ret = rte_cryptodev_sym_session_init(TEST_CDEV_ID, sess, > > &cipher_xform, params.session_priv_mpool); > > + TEST_ASSERT_SUCCESS(ret, "Failed to init session\n"); > > + > > rte_crypto_op_attach_sym_session(op, sess); > > } else { > > struct rte_crypto_sym_xform *first_xform; > > -- > > 2.17.1 >