* [dpdk-stable] [PATCH 1/2] net/i40e: fix negative check on unsigned queue pairs
@ 2019-03-05 16:30 Kevin Traynor
2019-03-07 13:42 ` Zhang, Qi Z
0 siblings, 1 reply; 2+ messages in thread
From: Kevin Traynor @ 2019-03-05 16:30 UTC (permalink / raw)
To: qi.z.zhang; +Cc: dev, Kevin Traynor, stable, zhirun.yan
Fix the check and associated log. Also, fix a typo in other log.
Fixes: 03d478e9609d ("net/i40e: support PF respond VF request more queues")
Cc: stable@dpdk.org
Cc: zhirun.yan@intel.com
Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
---
drivers/net/i40e/i40e_pf.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/net/i40e/i40e_pf.c b/drivers/net/i40e/i40e_pf.c
index 585b21cbc..0c4bdbcd1 100644
--- a/drivers/net/i40e/i40e_pf.c
+++ b/drivers/net/i40e/i40e_pf.c
@@ -1251,9 +1251,7 @@ i40e_pf_host_process_cmd_request_queues(struct i40e_pf_vf *vf, uint8_t *msg)
pf = vf->pf;
- if (req_pairs <= 0) {
- PMD_DRV_LOG(ERR,
- "VF %d tried to request %d queues. Ignoring.\n",
- vf->vf_idx,
- I40E_MAX_QP_NUM_PER_VF);
+ if (req_pairs == 0) {
+ PMD_DRV_LOG(ERR, "VF %d tried to request 0 queues. Ignoring.\n",
+ vf->vf_idx);
} else if (req_pairs > I40E_MAX_QP_NUM_PER_VF) {
PMD_DRV_LOG(ERR,
@@ -1264,5 +1262,5 @@ i40e_pf_host_process_cmd_request_queues(struct i40e_pf_vf *vf, uint8_t *msg)
} else if (req_pairs > cur_pairs + pf->qp_pool.num_free) {
PMD_DRV_LOG(ERR,
- "VF %d requested %d more queues, but noly %d left\n",
+ "VF %d requested %d more queues, but only %d left\n",
vf->vf_idx,
req_pairs - cur_pairs,
--
2.20.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [dpdk-stable] [PATCH 1/2] net/i40e: fix negative check on unsigned queue pairs
2019-03-05 16:30 [dpdk-stable] [PATCH 1/2] net/i40e: fix negative check on unsigned queue pairs Kevin Traynor
@ 2019-03-07 13:42 ` Zhang, Qi Z
0 siblings, 0 replies; 2+ messages in thread
From: Zhang, Qi Z @ 2019-03-07 13:42 UTC (permalink / raw)
To: Kevin Traynor; +Cc: dev, stable, Yan, Zhirun
> -----Original Message-----
> From: Kevin Traynor [mailto:ktraynor@redhat.com]
> Sent: Wednesday, March 6, 2019 12:31 AM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: dev@dpdk.org; Kevin Traynor <ktraynor@redhat.com>; stable@dpdk.org;
> Yan, Zhirun <zhirun.yan@intel.com>
> Subject: [PATCH 1/2] net/i40e: fix negative check on unsigned queue pairs
>
> Fix the check and associated log. Also, fix a typo in other log.
>
> Fixes: 03d478e9609d ("net/i40e: support PF respond VF request more queues")
> Cc: stable@dpdk.org
>
> Cc: zhirun.yan@intel.com
> Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
Applied to dpdk-next-net-intel.
Thanks
Qi
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-03-07 13:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-05 16:30 [dpdk-stable] [PATCH 1/2] net/i40e: fix negative check on unsigned queue pairs Kevin Traynor
2019-03-07 13:42 ` Zhang, Qi Z
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).