RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-26 Thread Dev, Vasu
V; Vick, > Matthew; Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- > de...@lists.sourceforge.net; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; brian.m...@oracle.com > Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF > reset > >

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-26 Thread Dev, Vasu
; Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- de...@lists.sourceforge.net; net...@vger.kernel.org; linux- ker...@vger.kernel.org; brian.m...@oracle.com Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset On 2015/1/20 5:10, Dev, Vasu wrote: -Original

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-19 Thread ethan zhao
, Gregory V; Vick, Matthew; Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- de...@lists.sourceforge.net; net...@vger.kernel.org; linux- ker...@vger.kernel.org; brian.m...@oracle.com Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset Vasu, What' your idea about

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-19 Thread Dev, Vasu
y V; Vick, Matthew; > Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- > de...@lists.sourceforge.net; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; brian.m...@oracle.com > Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF > reset > > Vas

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-19 Thread ethan zhao
, Gregory V; Vick, Matthew; Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- de...@lists.sourceforge.net; net...@vger.kernel.org; linux- ker...@vger.kernel.org; brian.m...@oracle.com Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset Vasu, What' your idea about

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-19 Thread Dev, Vasu
; Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- de...@lists.sourceforge.net; net...@vger.kernel.org; linux- ker...@vger.kernel.org; brian.m...@oracle.com Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset Vasu, What' your idea about the v2, any suggestion

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-16 Thread ethan zhao
FCOE by default when do PF reset Vasu, On 2015/1/14 3:38, Dev, Vasu wrote: -Original Message- diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index a5f2660..a2572cc 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-16 Thread Jeff Kirsher
; >> Bruce W; Wyborny, Carolyn; Skidmore, Donald C; Rose, Gregory V; Vick, > >> Matthew; Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- > >> de...@lists.sourceforge.net; net...@vger.kernel.org; linux- > >> ker...@vger.kernel.org; brian.m...@oracle.com > >>

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-16 Thread ethan zhao
FCOE by default when do PF reset Vasu, On 2015/1/14 3:38, Dev, Vasu wrote: -Original Message- diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index a5f2660..a2572cc 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-16 Thread Jeff Kirsher
; Linux NICS; e1000- de...@lists.sourceforge.net; net...@vger.kernel.org; linux- ker...@vger.kernel.org; brian.m...@oracle.com Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset Vasu, On 2015/1/14 3:38, Dev, Vasu wrote: -Original Message- diff

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-15 Thread ethan zhao
Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset Vasu, On 2015/1/14 3:38, Dev, Vasu wrote: -Original Message- diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index a5f2660..a2572cc 100644 --- a/drivers/net

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-15 Thread Dev, Vasu
V; Vick, > Matthew; Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- > de...@lists.sourceforge.net; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; brian.m...@oracle.com > Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF > reset > >

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-15 Thread Dev, Vasu
; Williams, Mitch A; Parikh, Neerav; Linux NICS; e1000- de...@lists.sourceforge.net; net...@vger.kernel.org; linux- ker...@vger.kernel.org; brian.m...@oracle.com Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset Vasu, On 2015/1/14 3:38, Dev, Vasu wrote

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-15 Thread ethan zhao
Subject: Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset Vasu, On 2015/1/14 3:38, Dev, Vasu wrote: -Original Message- diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index a5f2660..a2572cc 100644 --- a/drivers/net

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-13 Thread ethan zhao
Vasu, On 2015/1/14 3:38, Dev, Vasu wrote: -Original Message- diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index a5f2660..a2572cc 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-13 Thread Dev, Vasu
> -Original Message- > >> > diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c > >> > b/drivers/net/ethernet/intel/i40e/i40e_main.c > >> > index a5f2660..a2572cc 100644 > >> > --- a/drivers/net/ethernet/intel/i40e/i40e_main.c > >> > +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c >

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-13 Thread Dev, Vasu
-Original Message- diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index a5f2660..a2572cc 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c @@ -6180,9 +6180,12

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-13 Thread ethan zhao
Vasu, On 2015/1/14 3:38, Dev, Vasu wrote: -Original Message- diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c index a5f2660..a2572cc 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_main.c +++

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-12 Thread Ethan Zhao
Rose, Gregory V; Vick, Matthew; >> Williams, Mitch A; Dev, Vasu; Parikh, Neerav >> Cc: Linux NICS; e1000-de...@lists.sourceforge.net; net...@vger.kernel.org; >> linux-kernel@vger.kernel.org; ethan.ker...@gmail.com; >> brian.m...@oracle.com >> Subject: RE: [PATCH] i40

Re: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-12 Thread Ethan Zhao
, Matthew; Williams, Mitch A; Dev, Vasu; Parikh, Neerav Cc: Linux NICS; e1000-de...@lists.sourceforge.net; net...@vger.kernel.org; linux-kernel@vger.kernel.org; ethan.ker...@gmail.com; brian.m...@oracle.com Subject: RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-09 Thread Dev, Vasu
Neerav > Cc: Linux NICS; e1000-de...@lists.sourceforge.net; net...@vger.kernel.org; > linux-kernel@vger.kernel.org; ethan.ker...@gmail.com; > brian.m...@oracle.com > Subject: RE: [PATCH] i40e: don't enable and init FCOE by default when do PF > reset > > Adding Vasu and Neerav &

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-09 Thread Ronciak, John
atthew; Ronciak, John; > Williams, Mitch A > Cc: Linux NICS; e1000-de...@lists.sourceforge.net; net...@vger.kernel.org; > linux-kernel@vger.kernel.org; ethan.ker...@gmail.com; > brian.m...@oracle.com; Ethan Zhao > Subject: [PATCH] i40e: don't enable and init FCOE by default when do PF

[PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-09 Thread Ethan Zhao
While do PF reset with function i40e_reset_and_rebuild(), it will call i40e_init_pf_fcoe() by default if FCOE is defined, thus if the PF is resetted, FCOE will be enabled whatever it was - enabled or not. Such bug might be hit when PF resumes from suspend, run diagnostic test with ethtool, setup

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-09 Thread Ronciak, John
; Williams, Mitch A Cc: Linux NICS; e1000-de...@lists.sourceforge.net; net...@vger.kernel.org; linux-kernel@vger.kernel.org; ethan.ker...@gmail.com; brian.m...@oracle.com; Ethan Zhao Subject: [PATCH] i40e: don't enable and init FCOE by default when do PF reset While do PF reset

[PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-09 Thread Ethan Zhao
While do PF reset with function i40e_reset_and_rebuild(), it will call i40e_init_pf_fcoe() by default if FCOE is defined, thus if the PF is resetted, FCOE will be enabled whatever it was - enabled or not. Such bug might be hit when PF resumes from suspend, run diagnostic test with ethtool, setup

RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset

2015-01-09 Thread Dev, Vasu
; e1000-de...@lists.sourceforge.net; net...@vger.kernel.org; linux-kernel@vger.kernel.org; ethan.ker...@gmail.com; brian.m...@oracle.com Subject: RE: [PATCH] i40e: don't enable and init FCOE by default when do PF reset Adding Vasu and Neerav Cheers, John -Original Message- From