lkml.org 
[lkml]   [2021]   [Nov]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 060/121] iavf: Prevent changing static ITR values if adaptive moderation is on
    Date
    From: Nitesh B Venkatesh <nitesh.b.venkatesh@intel.com>

    [ Upstream commit e792779e6b639c182df91b46ac1e5803460b0b15 ]

    Resolve being able to change static values on VF when adaptive interrupt
    moderation is enabled.

    This problem is fixed by checking the interrupt settings is not
    a combination of change of static value while adaptive interrupt
    moderation is turned on.

    Without this fix, the user would be able to change static values
    on VF with adaptive moderation enabled.

    Fixes: 65e87c0398f5 ("i40evf: support queue-specific settings for interrupt moderation")
    Signed-off-by: Nitesh B Venkatesh <nitesh.b.venkatesh@intel.com>
    Tested-by: George Kuruvinakunnel <george.kuruvinakunnel@intel.com>
    Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    .../net/ethernet/intel/iavf/iavf_ethtool.c | 30 ++++++++++++++++---
    1 file changed, 26 insertions(+), 4 deletions(-)

    diff --git a/drivers/net/ethernet/intel/iavf/iavf_ethtool.c b/drivers/net/ethernet/intel/iavf/iavf_ethtool.c
    index ea85b06857fa2..90f5ec982d513 100644
    --- a/drivers/net/ethernet/intel/iavf/iavf_ethtool.c
    +++ b/drivers/net/ethernet/intel/iavf/iavf_ethtool.c
    @@ -719,12 +719,31 @@ static int iavf_get_per_queue_coalesce(struct net_device *netdev, u32 queue,
    *
    * Change the ITR settings for a specific queue.
    **/
    -static void iavf_set_itr_per_queue(struct iavf_adapter *adapter,
    - struct ethtool_coalesce *ec, int queue)
    +static int iavf_set_itr_per_queue(struct iavf_adapter *adapter,
    + struct ethtool_coalesce *ec, int queue)
    {
    struct iavf_ring *rx_ring = &adapter->rx_rings[queue];
    struct iavf_ring *tx_ring = &adapter->tx_rings[queue];
    struct iavf_q_vector *q_vector;
    + u16 itr_setting;
    +
    + itr_setting = rx_ring->itr_setting & ~IAVF_ITR_DYNAMIC;
    +
    + if (ec->rx_coalesce_usecs != itr_setting &&
    + ec->use_adaptive_rx_coalesce) {
    + netif_info(adapter, drv, adapter->netdev,
    + "Rx interrupt throttling cannot be changed if adaptive-rx is enabled\n");
    + return -EINVAL;
    + }
    +
    + itr_setting = tx_ring->itr_setting & ~IAVF_ITR_DYNAMIC;
    +
    + if (ec->tx_coalesce_usecs != itr_setting &&
    + ec->use_adaptive_tx_coalesce) {
    + netif_info(adapter, drv, adapter->netdev,
    + "Tx interrupt throttling cannot be changed if adaptive-tx is enabled\n");
    + return -EINVAL;
    + }

    rx_ring->itr_setting = ITR_REG_ALIGN(ec->rx_coalesce_usecs);
    tx_ring->itr_setting = ITR_REG_ALIGN(ec->tx_coalesce_usecs);
    @@ -747,6 +766,7 @@ static void iavf_set_itr_per_queue(struct iavf_adapter *adapter,
    * the Tx and Rx ITR values based on the values we have entered
    * into the q_vector, no need to write the values now.
    */
    + return 0;
    }

    /**
    @@ -788,9 +808,11 @@ static int __iavf_set_coalesce(struct net_device *netdev,
    */
    if (queue < 0) {
    for (i = 0; i < adapter->num_active_queues; i++)
    - iavf_set_itr_per_queue(adapter, ec, i);
    + if (iavf_set_itr_per_queue(adapter, ec, i))
    + return -EINVAL;
    } else if (queue < adapter->num_active_queues) {
    - iavf_set_itr_per_queue(adapter, ec, queue);
    + if (iavf_set_itr_per_queue(adapter, ec, queue))
    + return -EINVAL;
    } else {
    netif_info(adapter, drv, netdev, "Invalid queue value, queue range is 0 - %d\n",
    adapter->num_active_queues - 1);
    --
    2.33.0


    \
     
     \ /
      Last update: 2021-11-29 19:35    [W:2.786 / U:0.048 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site