Re: [E1000-devel] [next-net PATCH]drivers/net/ethernet/intel/e1000e: fix unregistered net_device ethX nameoutput by e1000e

From: Jeff Kirsher
Date: Wed Jun 06 2012 - 20:41:09 EST


On Wed, 2012-06-06 at 22:57 +0800, Ethan Zhao wrote:
> commit ca3ccc6835943287b6f69e973c126a02bc4de409
> Author: ethan.zhao <ethan.kernel@xxxxxxxxx>
> 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: signature.asc
Description: This is a digitally signed message part