mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Shrikrishna Khare <skhare@shri-linux.eng.vmware.com>
To: Benjamin Poirier <benjamin.poirier@gmail.com>
Cc: Shrikrishna Khare <skhare@vmware.com>, <pv-drivers@vmware.com>,
	"Jeff Kirsher" <jeffrey.t.kirsher@intel.com>,
	<intel-wired-lan@lists.osuosl.org>, <netdev@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH] e1000e: Remove Other from EIAC.
Date: Thu, 18 Jan 2018 18:42:05 -0800	[thread overview]
Message-ID: <alpine.DEB.2.10.1801181835490.23969@shri-linux.eng.vmware.com> (raw)
In-Reply-To: <20180118072741.hlkepaeibppz7uih@f1.synalogic.ca>



On Thu, 18 Jan 2018, Benjamin Poirier wrote:

> On 2018/01/18 15:50, Benjamin Poirier wrote:
> > It was reported that emulated e1000e devices in vmware esxi 6.5 Build
> > 7526125 do not link up after commit 4aea7a5c5e94 ("e1000e: Avoid receiver
> > overrun interrupt bursts", v4.15-rc1). Some tracing shows that after
> > e1000e_trigger_lsc() is called, ICR reads out as 0x0 in e1000_msix_other()
> > on emulated e1000e devices. In comparison, on real e1000e 82574 hardware,
> > icr=0x80000004 (_INT_ASSERTED | _OTHER) in the same situation.
> > 
> > Some experimentation showed that this flaw in vmware e1000e emulation can
> > be worked around by not setting Other in EIAC. This is how it was before
> > 16ecba59bc33 ("e1000e: Do not read ICR in Other interrupt", v4.5-rc1).
> 
> vmware folks, please comment.

Thank you for bringing this to our attention.

Using the reported build (ESX 6.5, 7526125) and 4.15.0-rc8+ kernel (which 
has the said patch), I could bring up e1000e interface (version: 3.2.6-k),
get dhcp address and even do large file downloads without difficulty.

Could you give us more pointers on how we may be able to reproduce this 
locally? Was there anything different with the configuration when the 
issue was observed? Is the issue consistently reproducible?

Thanks,
Shri

  reply	other threads:[~2018-01-19  2:57 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-18  6:50 Benjamin Poirier
2018-01-18  7:27 ` Benjamin Poirier
2018-01-19  2:42   ` Shrikrishna Khare [this message]
2018-01-19  5:36     ` Benjamin Poirier
2018-01-18  7:41 ` Benjamin Poirier
2018-01-18 15:51 ` [Intel-wired-lan] " Alexander Duyck
2018-01-19  8:59   ` Benjamin Poirier
2018-01-19 13:36     ` Benjamin Poirier
2018-01-19 16:22       ` Alexander Duyck
2018-01-19 22:45         ` Benjamin Poirier
2018-01-19 22:55           ` Benjamin Poirier
2018-01-20 17:21             ` Alexander Duyck
2018-01-22  7:12               ` Benjamin Poirier
2018-01-22 18:01                 ` Alexander Duyck
2018-01-24  8:35                   ` Benjamin Poirier
2018-01-24 16:01                     ` Alexander Duyck
2018-01-30 19:46 ` Alexander Duyck
2018-01-31  7:31   ` Benjamin Poirier
2018-02-02  4:29     ` Brown, Aaron F
2018-02-02  4:31     ` Brown, Aaron F

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=alpine.DEB.2.10.1801181835490.23969@shri-linux.eng.vmware.com \
    --to=skhare@shri-linux.eng.vmware.com \
    --cc=benjamin.poirier@gmail.com \
    --cc=intel-wired-lan@lists.osuosl.org \
    --cc=jeffrey.t.kirsher@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pv-drivers@vmware.com \
    --cc=skhare@vmware.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®