From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
To: Jeff Kirsher <jeffrey.t.kirsher@intel.com>, davem@davemloft.net
Cc: yzhu1 <yanjun.zhu@windriver.com>,
netdev@vger.kernel.org, gospo@redhat.com, sassmann@redhat.com
Subject: Re: [net v2 2/7] e1000: prevent oops when adapter is being closed and reset simultaneously
Date: Sat, 30 Nov 2013 20:37:48 +0400 [thread overview]
Message-ID: <529A145C.4030503@cogentembedded.com> (raw)
In-Reply-To: <1385803221-3752-3-git-send-email-jeffrey.t.kirsher@intel.com>
Hello.
On 30-11-2013 13:20, Jeff Kirsher wrote:
> From: yzhu1 <yanjun.zhu@windriver.com>
> This change is based on a similar change made to e1000e support in
> commit bb9e44d0d0f4 ("e1000e: prevent oops when adapter is being closed
> and reset simultaneously"). The same issue has also been observed
> on the older e1000 cards.
> Here, we have increased the RESET_COUNT value to 50 because there are too
> many accesses to e1000 nic on stress tests to e1000 nic, it is not enough
> to set RESET_COUT 25. Experimentation has shown that it is enough to set
> RESET_COUNT 50.
> Signed-off-by: yzhu1 <yanjun.zhu@windriver.com>
Jeff, why are you accepting patches without real name in the sign-off line?
> Tested-by: Aaron Brown <aaron.f.brown@intel.com>
> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
WBR, Sergei
next prev parent reply other threads:[~2013-11-30 16:37 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-30 9:20 [net v2 0/7][pull request] Intel Wired LAN Driver Updates Jeff Kirsher
2013-11-30 9:20 ` [net v2 1/7] igb: Fixed Wake On LAN support Jeff Kirsher
2013-11-30 9:20 ` [net v2 2/7] e1000: prevent oops when adapter is being closed and reset simultaneously Jeff Kirsher
2013-11-30 16:37 ` Sergei Shtylyov [this message]
2013-12-02 9:50 ` yzhu1
2013-12-02 16:12 ` David Miller
2013-12-03 2:02 ` yzhu1
2013-12-03 2:22 ` Jeff Kirsher
2013-12-03 2:25 ` yzhu1
2013-12-03 5:26 ` David Miller
2013-11-30 9:20 ` [net v2 3/7] e1000: fix lockdep warning in e1000_reset_task Jeff Kirsher
2013-11-30 9:20 ` [net v2 4/7] e1000: fix possible reset_task running after adapter down Jeff Kirsher
2013-11-30 9:20 ` [net v2 5/7] ixgbe: ixgbe_fwd_ring_down needs to be static Jeff Kirsher
2013-11-30 9:20 ` [net v2 6/7] ixgbe: turn NETIF_F_HW_L2FW_DOFFLOAD off by default Jeff Kirsher
2013-11-30 9:20 ` [net v2 7/7] ixgbe: Make ixgbe_identify_qsfp_module_generic static Jeff Kirsher
2013-11-30 17:46 ` [net v2 0/7][pull request] Intel Wired LAN Driver Updates David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=529A145C.4030503@cogentembedded.com \
--to=sergei.shtylyov@cogentembedded.com \
--cc=davem@davemloft.net \
--cc=gospo@redhat.com \
--cc=jeffrey.t.kirsher@intel.com \
--cc=netdev@vger.kernel.org \
--cc=sassmann@redhat.com \
--cc=yanjun.zhu@windriver.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.