Hello.
On 12/21/2014 3:52 AM, Jia-Ju Bai wrote:
> Th driver lacks netif_napi_del in the normal path and error path to match the
> call of netif_napi_add in e1000_probe.
Please wrap your change log lines at 80 columns (or less).
> This patch fixes this problem, and it has been tested in runtime.
> Signed-off-by: Jia-Ju Bai<baijiaju1...@163.com>
> ---
> drivers/net/ethernet/intel/e1000/e1000_main.c | 6 +-
> 1 file changed, 4 insertions(+), 2 deletions(-)
> diff --git a/drivers/net/ethernet/intel/e1000/e1000_main.c
> b/drivers/net/ethernet/intel/e1000/e1000_main.c
>
> index 24f3986..f6def7b 100644
>
> --- a/drivers/net/ethernet/intel/e1000/e1000_main.c
>
> +++ b/drivers/net/ethernet/intel/e1000/e1000_main.c
>
> @@ -1004,7 +1004,7 @@ static int e1000_probe(struct pci_dev *pdev, const
> struct pci_device_id *ent)
>
> /* make ready for any if (hw->...) below */
>
> err = e1000_init_hw_struct(adapter, hw);
>
> if (err)
>
> - goto err_sw_init;
>
> + goto err_dma;
>
> /* there is a workaround being applied below that limits
>
> * 64-bit DMA addresses to 64-bit hardware. There are some
Somehow your mailer inserted empty lines after each actual line of the
patch. :-(
WBR, Sergei
------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel® Ethernet, visit
http://communities.intel.com/community/wired
