On Mon, Sep 14, 2015 at 19:08:34 +0530, Hariprasad Shenai wrote:
> The issue is the for the Virtual Function Driver, the only way to get the
> Virtual Interface statistics is to issue mailbox commands to ask the
> firmware for the VI Stats. And, because the VI Stats command can only
> retrieve a smallish number of stats per mailbox command, we have to issue
> three mailbox commands in quick succession. What we ran into was irqbalance
> coming in every 10 seconds and interrogating every network interface in the
> system.
> 
> Signed-off-by: Hariprasad Shenai <haripra...@chelsio.com>
> ---
> V2: Updated description and using linux completion API's instead of
>     for loop based on review comments by David Miller
> 
>  drivers/net/ethernet/chelsio/cxgb4vf/adapter.h     |  9 +++++
>  .../net/ethernet/chelsio/cxgb4vf/cxgb4vf_main.c    |  4 ++
>  drivers/net/ethernet/chelsio/cxgb4vf/t4vf_hw.c     | 46 
> +++++++++++++++++++++-
>  3 files changed, 58 insertions(+), 1 deletion(-)
> 


Hi David,

There is an issue with this patch. Can you please drop it.
Will send a V3, with the fixes. 

The below one should be a while loop, instead of if condition.
        /* If we're at the head, break out and start the mailbox
         * protocol.
         */
        if (list_first_entry(&adapter->mlist.list,
                             struct mbox_list, list) != &entry) {
                int ret;

Thanks
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to