From: Ankur Dwivedi <adwivedi@marvell.com> To: "Gujjar, Abhinandan S" <abhinandan.gujjar@intel.com>, "dev@dpdk.org" <dev@dpdk.org> Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>, Anoob Joseph <anoobj@marvell.com> Subject: Re: [dpdk-dev] [PATCH 1/3] test/event_crypto_adapter: return error with unsupported mode Date: Wed, 23 Sep 2020 05:31:35 +0000 Message-ID: <DM5PR1801MB1884A606CC3B91A84EAA92C4DD380@DM5PR1801MB1884.namprd18.prod.outlook.com> (raw) In-Reply-To: <DM5PR1801MB1884E2FE5A40FD0BBE3804E5DD3F0@DM5PR1801MB1884.namprd18.prod.outlook.com> Hi Abhinandan, Please see my comments inline. Thanks Ankur >-----Original Message----- >From: Ankur Dwivedi >Sent: Friday, September 18, 2020 8:34 PM >To: 'Gujjar, Abhinandan S' <abhinandan.gujjar@intel.com>; dev@dpdk.org >Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Anoob Joseph ><anoobj@marvell.com> >Subject: RE: [PATCH 1/3] test/event_crypto_adapter: return error with >unsupported mode > >Hi Abhinandan, > >Please see my comments inline. > >Thanks >Ankur > >>-----Original Message----- >>From: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com> >>Sent: Friday, September 18, 2020 8:13 PM >>To: Ankur Dwivedi <adwivedi@marvell.com>; dev@dpdk.org >>Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Anoob Joseph >><anoobj@marvell.com> >>Subject: [EXT] RE: [PATCH 1/3] test/event_crypto_adapter: return error >>with unsupported mode >> >>External Email >> >>---------------------------------------------------------------------- >>Hi Ankur, >> >>Please see comments inline. >> >>Regards >>Abhinandan >> >>> -----Original Message----- >>> From: Ankur Dwivedi <adwivedi@marvell.com> >>> Sent: Friday, September 18, 2020 8:05 PM >>> To: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>; dev@dpdk.org >>> Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Anoob Joseph >>> <anoobj@marvell.com> >>> Subject: RE: [PATCH 1/3] test/event_crypto_adapter: return error with >>> unsupported mode >>> >>> Hi Abhinandan, >>> >>> Please see my comments inline. >>> >>> Thanks >>> Ankur >>> >>> >-----Original Message----- >>> >From: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com> >>> >Sent: Friday, September 18, 2020 12:41 PM >>> >To: Ankur Dwivedi <adwivedi@marvell.com>; dev@dpdk.org >>> >Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Anoob Joseph >>> ><anoobj@marvell.com> >>> >Subject: [EXT] RE: [PATCH 1/3] test/event_crypto_adapter: return >>> >error with unsupported mode >>> > >>> >External Email >>> > >>> >-------------------------------------------------------------------- >>> >- >>> >- >>> >Hi Ankur, >>> > >>> >This patch requires minor change. Please find the comment inline. >>> > >>> >> -----Original Message----- >>> >> From: Gujjar, Abhinandan S >>> >> Sent: Friday, September 11, 2020 1:07 PM >>> >> To: Ankur Dwivedi <adwivedi@marvell.com>; dev@dpdk.org >>> >> Cc: jerinj@marvell.com; anoobj@marvell.com >>> >> Subject: RE: [PATCH 1/3] test/event_crypto_adapter: return error >>> >> with unsupported mode >>> >> >>> >> Acked-by: abhinandan.gujjar@intel.com (For series) >>> >> >>> >> > -----Original Message----- >>> >> > From: Ankur Dwivedi <adwivedi@marvell.com> >>> >> > Sent: Tuesday, September 8, 2020 2:46 PM >>> >> > To: dev@dpdk.org >>> >> > Cc: Gujjar, Abhinandan S <abhinandan.gujjar@intel.com>; >>> >> > jerinj@marvell.com; anoobj@marvell.com; Ankur Dwivedi >>> >> > <adwivedi@marvell.com> >>> >> > Subject: [PATCH 1/3] test/event_crypto_adapter: return error >>> >> > with unsupported mode >>> >> > >>> >> > The capability of a event device should be checked before >>> >> > creating a event crypto adapter in a particular mode. The test >>> >> > case returns error if the mode is not supported. >>> >> > >>> >> > Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com> >>> >> > --- >>> >> > app/test/test_event_crypto_adapter.c | 30 >>> >> > ++++++++++++++++++---------- >>> >> > 1 file changed, 20 insertions(+), 10 deletions(-) >>> >> > >>> >> > diff --git a/app/test/test_event_crypto_adapter.c >>> >> > b/app/test/test_event_crypto_adapter.c >>> >> > index 8d42462d8..930c2a9bc 100644 >>> >> > --- a/app/test/test_event_crypto_adapter.c >>> >> > +++ b/app/test/test_event_crypto_adapter.c >>> >> > @@ -750,15 +750,23 @@ configure_event_crypto_adapter(enum >>> >> > rte_event_crypto_adapter_mode mode) >>> >> > uint32_t cap; >>> >> > int ret; >>> >> > >>> >> > + ret = rte_event_crypto_adapter_caps_get(evdev, >TEST_CDEV_ID, >>> >> > &cap); >>> >> > + TEST_ASSERT_SUCCESS(ret, "Failed to get adapter >>> >> > +capabilities\n"); >>> >> > + >>> >> > + if ((mode == RTE_EVENT_CRYPTO_ADAPTER_OP_FORWARD) >&& >>> >> > + !(cap & >>> >> > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_OP_FWD)) >>> >> > + return -ENOTSUP; >>> >> > + >>> >> > + if ((mode == RTE_EVENT_CRYPTO_ADAPTER_OP_NEW) && >>> >> > + !(cap & >>> >> > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_OP_NEW)) >>> >> > + return -ENOTSUP; >>> >> > + >>> >SW adapter does not have *_INTERNAL_PORT_* caps. Just noticed test >>> >was failing here. >>> >Could you please bypass this check for "event_sw" and repost the patch? >>> [Ankur] I was thinking of handling this by reading >>> rte_eventdevs[evdev].data->name. >>> If the name matches event_sw0(which is the default device name), I >>> will bypass if checks. >>> Can the event_sw device name be anything else? >>> Or, if there is a better way of handling this please let me know. >>Yes. You can also use rte_event_dev_info_get(evdev, &info) and check >>info.driver_name for "event_sw". >[Ankur] Thanks for the suggestion. I will try this and make the required >changes in v2. [Ankur] I thought of an alternative to implement the above. If the capability does not have any *_INTERNAL_PORT_* caps, it will be a SW PMD. Is this correct? In that case capability will be used to skip the if checks instead of the pmd specific name comparison. Please let me know your thoughts on this. >> >>> > >>> >> > /* Create adapter with default port creation callback */ >>> >> > ret = rte_event_crypto_adapter_create(TEST_ADAPTER_ID, >>> >> > - TEST_CDEV_ID, >>> >> > + evdev, >>> >> > &conf, mode); >>> >> > TEST_ASSERT_SUCCESS(ret, "Failed to create event crypto >>> >> adapter\n"); >>> >> > >>> >> > - ret = rte_event_crypto_adapter_caps_get(TEST_ADAPTER_ID, >>> >> > evdev, &cap); >>> >> > - TEST_ASSERT_SUCCESS(ret, "Failed to get adapter >capabilities\n"); >>> >> > - >>> >> > if (cap & >>> >> > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) { >>> >> > ret = >>> >> > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID, >>> >> > TEST_CDEV_ID, TEST_CDEV_QP_ID, >>> &response_info); @@ -813,6 >>> >> > +821,8 @@ test_crypto_adapter_conf(enum >>> >> > rte_event_crypto_adapter_mode mode) >>> >> > TEST_ASSERT(ret >= 0, "Failed to link queue >%d " >>> >> > "port=%u\n", qid, >>> >> > > params.crypto_event_port_id); >>> >> > + } else { >>> >> > + return ret; >>> >> > } >>> >> > crypto_adapter_setup_done = 1; >>> >> > } >>> >> > @@ -845,24 +855,24 @@ static int >>> >> > test_crypto_adapter_conf_op_forward_mode(void) >>> >> > { >>> >> > enum rte_event_crypto_adapter_mode mode; >>> >> > + int ret; >>> >> > >>> >> > mode = RTE_EVENT_CRYPTO_ADAPTER_OP_FORWARD; >>> >> > - TEST_ASSERT_SUCCESS(test_crypto_adapter_conf(mode), >>> >> > - "Failed to config crypto adapter"); >>> >> > + ret = test_crypto_adapter_conf(mode); >>> >> > >>> >> > - return TEST_SUCCESS; >>> >> > + return ret; >>> >> > } >>> >> > >>> >> > static int >>> >> > test_crypto_adapter_conf_op_new_mode(void) >>> >> > { >>> >> > enum rte_event_crypto_adapter_mode mode; >>> >> > + int ret; >>> >> > >>> >> > mode = RTE_EVENT_CRYPTO_ADAPTER_OP_NEW; >>> >> > - TEST_ASSERT_SUCCESS(test_crypto_adapter_conf(mode), >>> >> > - "Failed to config crypto adapter"); >>> >> > + ret = test_crypto_adapter_conf(mode); >>> >> > >>> >> > - return TEST_SUCCESS; >>> >> > + return ret; >>> >> > } >>> >> > >>> >> > >>> >> > -- >>> >> > 2.28.0
next prev parent reply other threads:[~2020-09-23 5:31 UTC|newest] Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-09-08 9:15 [dpdk-dev] [PATCH 0/3] Bug fixes in event crypto adapter test application Ankur Dwivedi 2020-09-08 9:15 ` [dpdk-dev] [PATCH 1/3] test/event_crypto_adapter: return error with unsupported mode Ankur Dwivedi 2020-09-11 7:37 ` Gujjar, Abhinandan S 2020-09-18 7:10 ` Gujjar, Abhinandan S 2020-09-18 14:34 ` Ankur Dwivedi 2020-09-18 14:42 ` Gujjar, Abhinandan S 2020-09-18 15:03 ` Ankur Dwivedi 2020-09-23 5:31 ` Ankur Dwivedi [this message] 2020-09-23 7:17 ` Gujjar, Abhinandan S 2020-10-08 9:12 ` Jerin Jacob 2020-09-08 9:15 ` [dpdk-dev] [PATCH 2/3] test/event_crypto_adapter: fix function arguments Ankur Dwivedi 2020-09-08 9:15 ` [dpdk-dev] [PATCH 3/3] test/event_crypto_adapter: free resources during exit Ankur Dwivedi 2020-09-25 7:28 ` [dpdk-dev] [PATCH v2 0/3] Bug fixes in event crypto adapter test application Ankur Dwivedi 2020-09-25 7:28 ` [dpdk-dev] [PATCH v2 1/3] test/event_crypto_adapter: return error with unsupported mode Ankur Dwivedi 2020-09-25 7:28 ` [dpdk-dev] [PATCH v2 2/3] test/event_crypto_adapter: fix function arguments Ankur Dwivedi 2020-10-06 10:21 ` Gujjar, Abhinandan S 2020-10-06 12:57 ` Ankur Dwivedi 2020-09-25 7:28 ` [dpdk-dev] [PATCH v2 3/3] test/event_crypto_adapter: free resources during exit Ankur Dwivedi 2020-10-06 10:21 ` Gujjar, Abhinandan S 2020-10-06 10:29 ` Gujjar, Abhinandan S 2020-10-06 13:00 ` Ankur Dwivedi 2020-10-05 9:43 ` [dpdk-dev] [PATCH v2 0/3] Bug fixes in event crypto adapter test application Jerin Jacob 2020-10-05 9:48 ` Gujjar, Abhinandan S 2020-10-07 5:26 ` [dpdk-dev] [PATCH v3 0/3] Bug fixes in event crypto adapter test Ankur Dwivedi 2020-10-07 5:26 ` [dpdk-dev] [PATCH v3 1/3] test/event_crypto_adapter: return error with unsupported mode Ankur Dwivedi 2020-10-08 7:25 ` Gujjar, Abhinandan S 2020-10-07 5:26 ` [dpdk-dev] [PATCH v3 2/3] test/event_crypto_adapter: fix function arguments Ankur Dwivedi 2020-10-08 7:26 ` Gujjar, Abhinandan S 2020-10-07 5:26 ` [dpdk-dev] [PATCH v3 3/3] test/event_crypto_adapter: free resources during exit Ankur Dwivedi 2020-10-08 7:26 ` Gujjar, Abhinandan S 2020-10-08 12:52 ` [dpdk-dev] [PATCH v4 0/3] Bug fixes in event crypto adapter test Ankur Dwivedi 2020-10-08 12:52 ` [dpdk-dev] [PATCH v4 1/3] test/event_crypto_adapter: return error for unsupported mode Ankur Dwivedi 2020-10-08 12:52 ` [dpdk-dev] [PATCH v4 2/3] test/event_crypto_adapter: fix function arguments Ankur Dwivedi 2020-10-08 12:52 ` [dpdk-dev] [PATCH v4 3/3] test/event_crypto_adapter: free resources during exit Ankur Dwivedi 2020-10-08 15:00 ` [dpdk-dev] [PATCH v4 0/3] Bug fixes in event crypto adapter test Jerin Jacob
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=DM5PR1801MB1884A606CC3B91A84EAA92C4DD380@DM5PR1801MB1884.namprd18.prod.outlook.com \ --to=adwivedi@marvell.com \ --cc=abhinandan.gujjar@intel.com \ --cc=anoobj@marvell.com \ --cc=dev@dpdk.org \ --cc=jerinj@marvell.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
DPDK patches and discussions This inbox may be cloned and mirrored by anyone: git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \ dev@dpdk.org public-inbox-index dev Example config snippet for mirrors. Newsgroup available over NNTP: nntp://inbox.dpdk.org/inbox.dpdk.dev AGPL code for this site: git clone https://public-inbox.org/public-inbox.git