Re: [net-next 10/16] i40e: add check for null VSI

2016-02-19 Thread David Miller
From: "Underwood, JohnX" Date: Fri, 19 Feb 2016 19:51:39 + > Sorry, folks. I was confused. The patch looked familiar, but I > wasn't able to find the email where I originally sent it out. Please stop top-posting Quote the relevant material, then add your

RE: [net-next 10/16] i40e: add check for null VSI

2016-02-19 Thread Underwood, JohnX
...@redhat.com; sassm...@redhat.com; jogre...@redhat.com Subject: Re: [net-next 10/16] i40e: add check for null VSI On Fri, 2016-02-19 at 14:41 -0500, David Miller wrote: > From: "Underwood, JohnX" <johnx.underw...@intel.com> > Date: Fri, 19 Feb 2016 19:15:53 + > > >

Re: [net-next 10/16] i40e: add check for null VSI

2016-02-19 Thread Jeff Kirsher
On Fri, 2016-02-19 at 14:41 -0500, David Miller wrote: > From: "Underwood, JohnX" > Date: Fri, 19 Feb 2016 19:15:53 + > > > ACK > > You should never top-post on this mailing list. > > But in this specific case it is even more important. > > So just in case it is

Re: [net-next 10/16] i40e: add check for null VSI

2016-02-19 Thread David Miller
From: "Underwood, JohnX" Date: Fri, 19 Feb 2016 19:15:53 + > ACK You should never top-post on this mailing list. But in this specific case it is even more important. So just in case it is not clear: Please, pretty please, DO NOT top-post ACKs to patches

RE: [net-next 10/16] i40e: add check for null VSI

2016-02-19 Thread Underwood, JohnX
ACK -Original Message- From: Kirsher, Jeffrey T Sent: Friday, February 19, 2016 1:54 AM To: da...@davemloft.net Cc: Underwood, JohnX; netdev@vger.kernel.org; nhor...@redhat.com; sassm...@redhat.com; jogre...@redhat.com; Kirsher, Jeffrey T Subject: [net-next 10/16] i40e: add check

[net-next 10/16] i40e: add check for null VSI

2016-02-19 Thread Jeff Kirsher
From: John Underwood Return from i40e_vsi_reinit_setup() if vsi param is NULL. This makes this code consistent with all the other code that checks for NULL before using one of the VSI pointers accessed with an indexed variable. (Indexed VSI pointers are intentionally