From patchwork Mon Jul 18 13:44:49 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Zhu Yanjun X-Patchwork-Id: 649540 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3rtPhD5g7Sz9sCY for ; Mon, 18 Jul 2016 23:50:36 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751440AbcGRNud (ORCPT ); Mon, 18 Jul 2016 09:50:33 -0400 Received: from [111.193.68.65] ([111.193.68.65]:45270 "EHLO mail.zhuyj.com" rhost-flags-FAIL-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1751251AbcGRNuc (ORCPT ); Mon, 18 Jul 2016 09:50:32 -0400 X-Greylist: delayed 321 seconds by postgrey-1.27 at vger.kernel.org; Mon, 18 Jul 2016 09:50:31 EDT Received: from mail.zhuyj.com (unknown [192.168.2.102]) by mail.zhuyj.com (Postfix) with ESMTPA id 93ADA5C3080; Mon, 18 Jul 2016 21:44:52 +0800 (CST) From: zyjzyj2000@gmail.com To: e1000-devel@lists.sourceforge.net, netdev@vger.kernel.org, jeffrey.t.kirsher@intel.com Cc: Zhu Yanjun Subject: [PATCH 1/1] ixgbevf: avoid checking hang when performing hardware reset Date: Mon, 18 Jul 2016 21:44:49 +0800 Message-Id: <1468849489-8407-1-git-send-email-zyjzyj2000@gmail.com> X-Mailer: git-send-email 2.7.4 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Zhu Yanjun When performing hardware reset, it is not necessary to check hang. Or else, the call trace will appear. Signed-off-by: Zhu Yanjun --- drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c index acc2401..d563d24 100644 --- a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c +++ b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c @@ -2792,9 +2792,14 @@ static void ixgbevf_reset_subtask(struct ixgbevf_adapter *adapter) static void ixgbevf_check_hang_subtask(struct ixgbevf_adapter *adapter) { struct ixgbe_hw *hw = &adapter->hw; + struct ixgbe_mbx_info *mbx = &hw->mbx; u32 eics = 0; int i; + /* When performing hardware reset, unnecessary to check hang. */ + if (mbx->ops.check_for_rst(hw)) + return; + /* If we're down or resetting, just bail */ if (test_bit(__IXGBEVF_DOWN, &adapter->state) || test_bit(__IXGBEVF_RESETTING, &adapter->state))