mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
To: Ethan Zhao <ethan.kernel@gmail.com>
Cc: eric.dumazet@gmail.com, linux.nics@intel.com,
	e1000-devel@lists.sourceforge.net,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [E1000-devel] [next-net PATCH] drivers/net/ethernet/intel/e1000e: fix unregistered net_device ethX name output by e1000e
Date: Wed, 06 Jun 2012 17:41:11 -0700	[thread overview]
Message-ID: <1339029671.10602.81.camel@jtkirshe-mobl> (raw)
In-Reply-To: <CABawtvNv7LwT_awXscXGEid-1wes-+ZELc7YMLZqyo2h=h98UA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1076 bytes --]

On Wed, 2012-06-06 at 22:57 +0800, Ethan Zhao wrote:
> commit ca3ccc6835943287b6f69e973c126a02bc4de409
> Author: ethan.zhao <ethan.kernel@gmail.com>
> Date:   Wed Jun 6 07:32:11 2012 -0700
> 
>         modified:   drivers/net/ethernet/intel/e1000e/param.c
> 
>     While e1000e_check_options() is called, netdev is not registered,
> so the
>     e1000e driver will print out confused ethernet interface name
> (unregistered net_device) :
> 
>     e1000e 0000:04:00.0:(unregistered net_device): Interrupt
> Throttling Rate (ints/sec) set to dynamic conservative mode
> 
>     So change e_info() back to dev_printk() by simply redefine the
> e_info macro used by
>     e1000e_check_options() and e1000_validate_option
> 
>     after applied this patch, we got:
> 
>     e1000e 0000:04:00.0: Interrupt Throttling Rate (ints/sec) set to
> dynamic conservative mode
>     e1000e 0000:04:00.0: irq 95 for MSI/MSI-X 

NACK, I currently have a patch in my queue from Bruce Allan to resolve
this issue.  I should be pushing the patch upstream in a few days.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2012-06-07  0:41 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-06 14:57 Ethan Zhao
2012-06-07  0:41 ` Jeff Kirsher [this message]
2012-06-07  0:56   ` [E1000-devel] " Allan, Bruce W
     [not found]     ` <CABawtvNAtJ=W=rztnizPz=ZXGcLoGHPNJn4XUZMQyQJQ46nooQ@mail.gmail.com>
     [not found]       ` <804857E1F29AAC47BF68C404FC60A18428444C0A@ORSMSX102.amr.corp.intel.com>
     [not found]         ` <9DA1E3A1-025A-474C-811A-957B57E74911@gmail.com>
     [not found]           ` <1339108741.21222.21.camel@jtkirshe-mobl>
2012-06-08  0:55             ` Ethan Zhao

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=1339029671.10602.81.camel@jtkirshe-mobl \
    --to=jeffrey.t.kirsher@intel.com \
    --cc=e1000-devel@lists.sourceforge.net \
    --cc=eric.dumazet@gmail.com \
    --cc=ethan.kernel@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux.nics@intel.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®