[v4] eventdev/rx-adapter: fix segfault in queue conf get
Checks
Commit Message
rte_event_eth_rx_adapter_queue_conf_get() segfaults if called
without queue added to the Rx adapter.
Added check to no queues in Rx adapter and error out on being
called with no queue in Rx adapter.
Added test case to call queue conf get without queues in
Rx adapter.
Fixes: b36879759b7f3ce ("eventdev/rx_adapter: support Rx queue config get")
Signed-off-by: Ganapati Kundapura <ganapati.kundapura@intel.com>
---
v4:
* Fixed checkpatch warning
v3:
* Added fixes line
v2:
* Corrected typo in the comment
---
Comments
> -----Original Message-----
> From: Kundapura, Ganapati <ganapati.kundapura@intel.com>
> Sent: Friday, October 1, 2021 11:00 AM
> To: jerinjacobk@gmail.com; dev@dpdk.org
> Cc: Jayatheerthan, Jay <jay.jayatheerthan@intel.com>
> Subject: [PATCH v4] eventdev/rx-adapter: fix segfault in queue conf get
>
> rte_event_eth_rx_adapter_queue_conf_get() segfaults if called
> without queue added to the Rx adapter.
>
> Added check to no queues in Rx adapter and error out on being
> called with no queue in Rx adapter.
>
> Added test case to call queue conf get without queues in
> Rx adapter.
>
> Fixes: b36879759b7f3ce ("eventdev/rx_adapter: support Rx queue config get")
> Signed-off-by: Ganapati Kundapura <ganapati.kundapura@intel.com>
>
> ---
> v4:
> * Fixed checkpatch warning
>
> v3:
> * Added fixes line
>
> v2:
> * Corrected typo in the comment
> ---
>
> diff --git a/app/test/test_event_eth_rx_adapter.c b/app/test/test_event_eth_rx_adapter.c
> index 13664a3..d0dc552 100644
> --- a/app/test/test_event_eth_rx_adapter.c
> +++ b/app/test/test_event_eth_rx_adapter.c
> @@ -751,20 +751,48 @@ static int
> adapter_queue_conf(void)
> {
> int err;
> - struct rte_event_eth_rx_adapter_queue_conf queue_conf;
> + struct rte_event_eth_rx_adapter_queue_conf queue_conf = {0};
>
> - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
> + /* Case 1: queue conf get without any queues in Rx adapter */
> + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> + 0, &queue_conf);
> + TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
> +
> + /* Add queue to Rx adapter */
> + queue_conf.ev.queue_id = 0;
> + queue_conf.ev.sched_type = RTE_SCHED_TYPE_ATOMIC;
> + queue_conf.ev.priority = RTE_EVENT_DEV_PRIORITY_NORMAL;
> +
> + err = rte_event_eth_rx_adapter_queue_add(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> + 0, &queue_conf);
> + TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> +
> + /* Case 2: queue conf get with queue added to Rx adapter */
> + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> 0, &queue_conf);
> TEST_ASSERT(err == 0, "Expected 0 got %d", err);
>
> - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
> + /* Case 3: queue conf get with invalid rx queue id */
> + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> -1, &queue_conf);
rx_queue_id param is uint16_t. Would using non-configured queue id (for e.g. rx_queue_id = 2) suffice instead of using -1 ?
> TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
>
> - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
> + /* Case 4: queue conf get with NULL queue conf struct */
> + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> 0, NULL);
> TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
>
> + /* Delete queue from the Rx adapter */
> + err = rte_event_eth_rx_adapter_queue_del(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> + 0);
> + TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> +
> return TEST_SUCCESS;
> }
>
> diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c b/lib/eventdev/rte_event_eth_rx_adapter.c
> index 10491ca..2a84490 100644
> --- a/lib/eventdev/rte_event_eth_rx_adapter.c
> +++ b/lib/eventdev/rte_event_eth_rx_adapter.c
> @@ -2844,12 +2844,13 @@ rte_event_eth_rx_adapter_queue_conf_get(uint8_t id,
> return -EINVAL;
>
> dev_info = &rx_adapter->eth_devices[eth_dev_id];
> - queue_info = &dev_info->rx_queue[rx_queue_id];
> - if (!queue_info->queue_enabled) {
> + if (dev_info->rx_queue == NULL ||
> + !dev_info->rx_queue[rx_queue_id].queue_enabled) {
> RTE_EDEV_LOG_ERR("Rx queue %u not added", rx_queue_id);
> return -EINVAL;
> }
>
> + queue_info = &dev_info->rx_queue[rx_queue_id];
> qi_ev = (struct rte_event *)&queue_info->event;
>
> memset(queue_conf, 0, sizeof(*queue_conf));
> --
> 2.6.4
Hi Jay,
> -----Original Message-----
> From: Jayatheerthan, Jay <jay.jayatheerthan@intel.com>
> Sent: 04 October 2021 22:49
> To: Kundapura, Ganapati <ganapati.kundapura@intel.com>;
> jerinjacobk@gmail.com; dev@dpdk.org
> Subject: RE: [PATCH v4] eventdev/rx-adapter: fix segfault in queue conf get
>
> > -----Original Message-----
> > From: Kundapura, Ganapati <ganapati.kundapura@intel.com>
> > Sent: Friday, October 1, 2021 11:00 AM
> > To: jerinjacobk@gmail.com; dev@dpdk.org
> > Cc: Jayatheerthan, Jay <jay.jayatheerthan@intel.com>
> > Subject: [PATCH v4] eventdev/rx-adapter: fix segfault in queue conf
> > get
> >
> > rte_event_eth_rx_adapter_queue_conf_get() segfaults if called without
> > queue added to the Rx adapter.
> >
> > Added check to no queues in Rx adapter and error out on being called
> > with no queue in Rx adapter.
> >
> > Added test case to call queue conf get without queues in Rx adapter.
> >
> > Fixes: b36879759b7f3ce ("eventdev/rx_adapter: support Rx queue config
> > get")
> > Signed-off-by: Ganapati Kundapura <ganapati.kundapura@intel.com>
> >
> > ---
> > v4:
> > * Fixed checkpatch warning
> >
> > v3:
> > * Added fixes line
> >
> > v2:
> > * Corrected typo in the comment
> > ---
> >
> > diff --git a/app/test/test_event_eth_rx_adapter.c
> > b/app/test/test_event_eth_rx_adapter.c
> > index 13664a3..d0dc552 100644
> > --- a/app/test/test_event_eth_rx_adapter.c
> > +++ b/app/test/test_event_eth_rx_adapter.c
> > @@ -751,20 +751,48 @@ static int
> > adapter_queue_conf(void)
> > {
> > int err;
> > - struct rte_event_eth_rx_adapter_queue_conf queue_conf;
> > + struct rte_event_eth_rx_adapter_queue_conf queue_conf = {0};
> >
> > - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> TEST_DEV_ID,
> > + /* Case 1: queue conf get without any queues in Rx adapter */
> > + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > + TEST_ETHDEV_ID,
> > + 0, &queue_conf);
> > + TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
> > +
> > + /* Add queue to Rx adapter */
> > + queue_conf.ev.queue_id = 0;
> > + queue_conf.ev.sched_type = RTE_SCHED_TYPE_ATOMIC;
> > + queue_conf.ev.priority = RTE_EVENT_DEV_PRIORITY_NORMAL;
> > +
> > + err = rte_event_eth_rx_adapter_queue_add(TEST_INST_ID,
> > + TEST_ETHDEV_ID,
> > + 0, &queue_conf);
> > + TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> > +
> > + /* Case 2: queue conf get with queue added to Rx adapter */
> > + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > + TEST_ETHDEV_ID,
> > 0, &queue_conf);
> > TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> >
> > - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> TEST_DEV_ID,
> > + /* Case 3: queue conf get with invalid rx queue id */
> > + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > + TEST_ETHDEV_ID,
> > -1, &queue_conf);
>
> rx_queue_id param is uint16_t. Would using non-configured queue id (for
> e.g. rx_queue_id = 2) suffice instead of using -1 ?
>
When -1 is promoted/demoted to uint16_t, it'll be converted to large value i.e UINT_MAX and guaranteed
to be invalid rx queue id and it's future proof.
In case, some queues are added in that test case in future, this invalid rx_queue_id also needs to be updated.
Yes, for this case rx_queue_id=2 can be used, but it's not future proof.
> > TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
> >
> > - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> TEST_DEV_ID,
> > + /* Case 4: queue conf get with NULL queue conf struct */
> > + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > + TEST_ETHDEV_ID,
> > 0, NULL);
> > TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
> >
> > + /* Delete queue from the Rx adapter */
> > + err = rte_event_eth_rx_adapter_queue_del(TEST_INST_ID,
> > + TEST_ETHDEV_ID,
> > + 0);
> > + TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> > +
> > return TEST_SUCCESS;
> > }
> >
> > diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c
> > b/lib/eventdev/rte_event_eth_rx_adapter.c
> > index 10491ca..2a84490 100644
> > --- a/lib/eventdev/rte_event_eth_rx_adapter.c
> > +++ b/lib/eventdev/rte_event_eth_rx_adapter.c
> > @@ -2844,12 +2844,13 @@
> rte_event_eth_rx_adapter_queue_conf_get(uint8_t id,
> > return -EINVAL;
> >
> > dev_info = &rx_adapter->eth_devices[eth_dev_id];
> > - queue_info = &dev_info->rx_queue[rx_queue_id];
> > - if (!queue_info->queue_enabled) {
> > + if (dev_info->rx_queue == NULL ||
> > + !dev_info->rx_queue[rx_queue_id].queue_enabled) {
> > RTE_EDEV_LOG_ERR("Rx queue %u not added",
> rx_queue_id);
> > return -EINVAL;
> > }
> >
> > + queue_info = &dev_info->rx_queue[rx_queue_id];
> > qi_ev = (struct rte_event *)&queue_info->event;
> >
> > memset(queue_conf, 0, sizeof(*queue_conf));
> > --
> > 2.6.4
> -----Original Message-----
> From: Kundapura, Ganapati <ganapati.kundapura@intel.com>
> Sent: Monday, October 4, 2021 11:55 PM
> To: Jayatheerthan, Jay <jay.jayatheerthan@intel.com>; jerinjacobk@gmail.com; dev@dpdk.org
> Subject: RE: [PATCH v4] eventdev/rx-adapter: fix segfault in queue conf get
>
> Hi Jay,
>
> > -----Original Message-----
> > From: Jayatheerthan, Jay <jay.jayatheerthan@intel.com>
> > Sent: 04 October 2021 22:49
> > To: Kundapura, Ganapati <ganapati.kundapura@intel.com>;
> > jerinjacobk@gmail.com; dev@dpdk.org
> > Subject: RE: [PATCH v4] eventdev/rx-adapter: fix segfault in queue conf get
> >
> > > -----Original Message-----
> > > From: Kundapura, Ganapati <ganapati.kundapura@intel.com>
> > > Sent: Friday, October 1, 2021 11:00 AM
> > > To: jerinjacobk@gmail.com; dev@dpdk.org
> > > Cc: Jayatheerthan, Jay <jay.jayatheerthan@intel.com>
> > > Subject: [PATCH v4] eventdev/rx-adapter: fix segfault in queue conf
> > > get
> > >
> > > rte_event_eth_rx_adapter_queue_conf_get() segfaults if called without
> > > queue added to the Rx adapter.
> > >
> > > Added check to no queues in Rx adapter and error out on being called
> > > with no queue in Rx adapter.
> > >
> > > Added test case to call queue conf get without queues in Rx adapter.
> > >
> > > Fixes: b36879759b7f3ce ("eventdev/rx_adapter: support Rx queue config
> > > get")
> > > Signed-off-by: Ganapati Kundapura <ganapati.kundapura@intel.com>
> > >
> > > ---
> > > v4:
> > > * Fixed checkpatch warning
> > >
> > > v3:
> > > * Added fixes line
> > >
> > > v2:
> > > * Corrected typo in the comment
> > > ---
> > >
> > > diff --git a/app/test/test_event_eth_rx_adapter.c
> > > b/app/test/test_event_eth_rx_adapter.c
> > > index 13664a3..d0dc552 100644
> > > --- a/app/test/test_event_eth_rx_adapter.c
> > > +++ b/app/test/test_event_eth_rx_adapter.c
> > > @@ -751,20 +751,48 @@ static int
> > > adapter_queue_conf(void)
> > > {
> > > int err;
> > > - struct rte_event_eth_rx_adapter_queue_conf queue_conf;
> > > + struct rte_event_eth_rx_adapter_queue_conf queue_conf = {0};
> > >
> > > - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > TEST_DEV_ID,
> > > + /* Case 1: queue conf get without any queues in Rx adapter */
> > > + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > > + TEST_ETHDEV_ID,
> > > + 0, &queue_conf);
> > > + TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
> > > +
> > > + /* Add queue to Rx adapter */
> > > + queue_conf.ev.queue_id = 0;
> > > + queue_conf.ev.sched_type = RTE_SCHED_TYPE_ATOMIC;
> > > + queue_conf.ev.priority = RTE_EVENT_DEV_PRIORITY_NORMAL;
> > > +
> > > + err = rte_event_eth_rx_adapter_queue_add(TEST_INST_ID,
> > > + TEST_ETHDEV_ID,
> > > + 0, &queue_conf);
> > > + TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> > > +
> > > + /* Case 2: queue conf get with queue added to Rx adapter */
> > > + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > > + TEST_ETHDEV_ID,
> > > 0, &queue_conf);
> > > TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> > >
> > > - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > TEST_DEV_ID,
> > > + /* Case 3: queue conf get with invalid rx queue id */
> > > + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > > + TEST_ETHDEV_ID,
> > > -1, &queue_conf);
> >
> > rx_queue_id param is uint16_t. Would using non-configured queue id (for
> > e.g. rx_queue_id = 2) suffice instead of using -1 ?
> >
> When -1 is promoted/demoted to uint16_t, it'll be converted to large value i.e UINT_MAX and guaranteed
> to be invalid rx queue id and it's future proof.
> In case, some queues are added in that test case in future, this invalid rx_queue_id also needs to be updated.
>
> Yes, for this case rx_queue_id=2 can be used, but it's not future proof.
Acked-by: Jay Jayatheerthan <jay.jayatheerthan@intel.com>
>
> > > TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
> > >
> > > - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > TEST_DEV_ID,
> > > + /* Case 4: queue conf get with NULL queue conf struct */
> > > + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> > > + TEST_ETHDEV_ID,
> > > 0, NULL);
> > > TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
> > >
> > > + /* Delete queue from the Rx adapter */
> > > + err = rte_event_eth_rx_adapter_queue_del(TEST_INST_ID,
> > > + TEST_ETHDEV_ID,
> > > + 0);
> > > + TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> > > +
> > > return TEST_SUCCESS;
> > > }
> > >
> > > diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c
> > > b/lib/eventdev/rte_event_eth_rx_adapter.c
> > > index 10491ca..2a84490 100644
> > > --- a/lib/eventdev/rte_event_eth_rx_adapter.c
> > > +++ b/lib/eventdev/rte_event_eth_rx_adapter.c
> > > @@ -2844,12 +2844,13 @@
> > rte_event_eth_rx_adapter_queue_conf_get(uint8_t id,
> > > return -EINVAL;
> > >
> > > dev_info = &rx_adapter->eth_devices[eth_dev_id];
> > > - queue_info = &dev_info->rx_queue[rx_queue_id];
> > > - if (!queue_info->queue_enabled) {
> > > + if (dev_info->rx_queue == NULL ||
> > > + !dev_info->rx_queue[rx_queue_id].queue_enabled) {
> > > RTE_EDEV_LOG_ERR("Rx queue %u not added",
> > rx_queue_id);
> > > return -EINVAL;
> > > }
> > >
> > > + queue_info = &dev_info->rx_queue[rx_queue_id];
> > > qi_ev = (struct rte_event *)&queue_info->event;
> > >
> > > memset(queue_conf, 0, sizeof(*queue_conf));
> > > --
> > > 2.6.4
On Fri, Oct 1, 2021 at 11:00 AM Ganapati Kundapura
<ganapati.kundapura@intel.com> wrote:
>
> rte_event_eth_rx_adapter_queue_conf_get() segfaults if called
> without queue added to the Rx adapter.
>
> Added check to no queues in Rx adapter and error out on being
> called with no queue in Rx adapter.
>
> Added test case to call queue conf get without queues in
> Rx adapter.
>
> Fixes: b36879759b7f3ce ("eventdev/rx_adapter: support Rx queue config get")
Squashed to relevant patches to dpdk-next-net-eventdev/for-main. Thanks
> Signed-off-by: Ganapati Kundapura <ganapati.kundapura@intel.com>
>
> ---
> v4:
> * Fixed checkpatch warning
>
> v3:
> * Added fixes line
>
> v2:
> * Corrected typo in the comment
> ---
>
> diff --git a/app/test/test_event_eth_rx_adapter.c b/app/test/test_event_eth_rx_adapter.c
> index 13664a3..d0dc552 100644
> --- a/app/test/test_event_eth_rx_adapter.c
> +++ b/app/test/test_event_eth_rx_adapter.c
> @@ -751,20 +751,48 @@ static int
> adapter_queue_conf(void)
> {
> int err;
> - struct rte_event_eth_rx_adapter_queue_conf queue_conf;
> + struct rte_event_eth_rx_adapter_queue_conf queue_conf = {0};
>
> - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
> + /* Case 1: queue conf get without any queues in Rx adapter */
> + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> + 0, &queue_conf);
> + TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
> +
> + /* Add queue to Rx adapter */
> + queue_conf.ev.queue_id = 0;
> + queue_conf.ev.sched_type = RTE_SCHED_TYPE_ATOMIC;
> + queue_conf.ev.priority = RTE_EVENT_DEV_PRIORITY_NORMAL;
> +
> + err = rte_event_eth_rx_adapter_queue_add(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> + 0, &queue_conf);
> + TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> +
> + /* Case 2: queue conf get with queue added to Rx adapter */
> + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> 0, &queue_conf);
> TEST_ASSERT(err == 0, "Expected 0 got %d", err);
>
> - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
> + /* Case 3: queue conf get with invalid rx queue id */
> + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> -1, &queue_conf);
> TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
>
> - err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
> + /* Case 4: queue conf get with NULL queue conf struct */
> + err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> 0, NULL);
> TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
>
> + /* Delete queue from the Rx adapter */
> + err = rte_event_eth_rx_adapter_queue_del(TEST_INST_ID,
> + TEST_ETHDEV_ID,
> + 0);
> + TEST_ASSERT(err == 0, "Expected 0 got %d", err);
> +
> return TEST_SUCCESS;
> }
>
> diff --git a/lib/eventdev/rte_event_eth_rx_adapter.c b/lib/eventdev/rte_event_eth_rx_adapter.c
> index 10491ca..2a84490 100644
> --- a/lib/eventdev/rte_event_eth_rx_adapter.c
> +++ b/lib/eventdev/rte_event_eth_rx_adapter.c
> @@ -2844,12 +2844,13 @@ rte_event_eth_rx_adapter_queue_conf_get(uint8_t id,
> return -EINVAL;
>
> dev_info = &rx_adapter->eth_devices[eth_dev_id];
> - queue_info = &dev_info->rx_queue[rx_queue_id];
> - if (!queue_info->queue_enabled) {
> + if (dev_info->rx_queue == NULL ||
> + !dev_info->rx_queue[rx_queue_id].queue_enabled) {
> RTE_EDEV_LOG_ERR("Rx queue %u not added", rx_queue_id);
> return -EINVAL;
> }
>
> + queue_info = &dev_info->rx_queue[rx_queue_id];
> qi_ev = (struct rte_event *)&queue_info->event;
>
> memset(queue_conf, 0, sizeof(*queue_conf));
> --
> 2.6.4
>
@@ -751,20 +751,48 @@ static int
adapter_queue_conf(void)
{
int err;
- struct rte_event_eth_rx_adapter_queue_conf queue_conf;
+ struct rte_event_eth_rx_adapter_queue_conf queue_conf = {0};
- err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
+ /* Case 1: queue conf get without any queues in Rx adapter */
+ err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
+ TEST_ETHDEV_ID,
+ 0, &queue_conf);
+ TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
+
+ /* Add queue to Rx adapter */
+ queue_conf.ev.queue_id = 0;
+ queue_conf.ev.sched_type = RTE_SCHED_TYPE_ATOMIC;
+ queue_conf.ev.priority = RTE_EVENT_DEV_PRIORITY_NORMAL;
+
+ err = rte_event_eth_rx_adapter_queue_add(TEST_INST_ID,
+ TEST_ETHDEV_ID,
+ 0, &queue_conf);
+ TEST_ASSERT(err == 0, "Expected 0 got %d", err);
+
+ /* Case 2: queue conf get with queue added to Rx adapter */
+ err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
+ TEST_ETHDEV_ID,
0, &queue_conf);
TEST_ASSERT(err == 0, "Expected 0 got %d", err);
- err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
+ /* Case 3: queue conf get with invalid rx queue id */
+ err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
+ TEST_ETHDEV_ID,
-1, &queue_conf);
TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
- err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID, TEST_DEV_ID,
+ /* Case 4: queue conf get with NULL queue conf struct */
+ err = rte_event_eth_rx_adapter_queue_conf_get(TEST_INST_ID,
+ TEST_ETHDEV_ID,
0, NULL);
TEST_ASSERT(err == -EINVAL, "Expected -EINVAL got %d", err);
+ /* Delete queue from the Rx adapter */
+ err = rte_event_eth_rx_adapter_queue_del(TEST_INST_ID,
+ TEST_ETHDEV_ID,
+ 0);
+ TEST_ASSERT(err == 0, "Expected 0 got %d", err);
+
return TEST_SUCCESS;
}
@@ -2844,12 +2844,13 @@ rte_event_eth_rx_adapter_queue_conf_get(uint8_t id,
return -EINVAL;
dev_info = &rx_adapter->eth_devices[eth_dev_id];
- queue_info = &dev_info->rx_queue[rx_queue_id];
- if (!queue_info->queue_enabled) {
+ if (dev_info->rx_queue == NULL ||
+ !dev_info->rx_queue[rx_queue_id].queue_enabled) {
RTE_EDEV_LOG_ERR("Rx queue %u not added", rx_queue_id);
return -EINVAL;
}
+ queue_info = &dev_info->rx_queue[rx_queue_id];
qi_ev = (struct rte_event *)&queue_info->event;
memset(queue_conf, 0, sizeof(*queue_conf));