[Intel-wired-lan] [next PATCH S65 07/13] i40e: remove extraneous loop in i40e_vsi_wait_queues_disabled
Keller, Jacob E
jacob.e.keller at intel.com
Mon Mar 27 23:41:55 UTC 2017
ACK
> -----Original Message-----
> From: Michael, Alice
> Sent: Monday, March 27, 2017 2:48 AM
> To: intel-wired-lan at lists.osuosl.org
> Cc: Keller, Jacob E <jacob.e.keller at intel.com>
> Subject: [next PATCH S65 07/13] i40e: remove extraneous loop in
> i40e_vsi_wait_queues_disabled
>
> From: Jacob Keller <jacob.e.keller at intel.com>
>
> We can simply check both Tx and Rx queues in a single loop, rather than
> repeating the loop twice.
>
> Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
> Change-ID: Ic06f26b0e3c2620e0e33c1a2999edda488e647ad
> ---
> drivers/net/ethernet/intel/i40e/i40e_main.c | 10 +++-------
> 1 file changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c
> b/drivers/net/ethernet/intel/i40e/i40e_main.c
> index 992dec1..cecbb5d 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
> @@ -4431,7 +4431,7 @@ static void i40e_pf_unquiesce_all_vsi(struct i40e_pf
> *pf)
> * i40e_vsi_wait_queues_disabled - Wait for VSI's queues to be disabled
> * @vsi: the VSI being configured
> *
> - * This function waits for the given VSI's queues to be disabled.
> + * Wait until all queues on a given VSI have been disabled.
> **/
> static int i40e_vsi_wait_queues_disabled(struct i40e_vsi *vsi)
> {
> @@ -4440,7 +4440,7 @@ static int i40e_vsi_wait_queues_disabled(struct
> i40e_vsi *vsi)
>
> pf_q = vsi->base_queue;
> for (i = 0; i < vsi->num_queue_pairs; i++, pf_q++) {
> - /* Check and wait for the disable status of the queue */
> + /* Check and wait for the Tx queue */
> ret = i40e_pf_txq_wait(pf, pf_q, false);
> if (ret) {
> dev_info(&pf->pdev->dev,
> @@ -4448,11 +4448,7 @@ static int i40e_vsi_wait_queues_disabled(struct
> i40e_vsi *vsi)
> vsi->seid, pf_q);
> return ret;
> }
> - }
> -
> - pf_q = vsi->base_queue;
> - for (i = 0; i < vsi->num_queue_pairs; i++, pf_q++) {
> - /* Check and wait for the disable status of the queue */
> + /* Check and wait for the Tx queue */
> ret = i40e_pf_rxq_wait(pf, pf_q, false);
> if (ret) {
> dev_info(&pf->pdev->dev,
> --
> 2.9.3
More information about the Intel-wired-lan
mailing list