Intel-Wired-Lan Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
To: intel-wired-lan@osuosl.org
Subject: [Intel-wired-lan] [next PATCH S21 07/14] i40e: propagate properly
Date: Sun, 08 Nov 2015 04:46:26 -0800	[thread overview]
Message-ID: <1446986786.4094.9.camel@intel.com> (raw)
In-Reply-To: <1446852372-25480-8-git-send-email-joshua.a.hay@intel.com>

On Fri, 2015-11-06 at 15:26 -0800, Joshua Hay wrote:
> From: Mitch Williams <mitch.a.williams@intel.com>
> 
> i40e_sync_vsi_filters() is the surly teenager of this driver. It says
> it's going to report errors, but it doesn't actually do that most of
> the
> time. And when it does, it leaves a mess.
> 
> Change this function to have a common exit point so it will properly
> release the busy lock on the VSI. Propagate errors to the callers.
> Finally, adjust a few callers to check for and deal with errors from
> this function.
> 
> Signed-off-by: Mitch Williams <mitch.a.williams@intel.com>
> Change-ID: Ic6af4956491e72402ebb3c538a3c31a0ad7f8667
> ---
> ?drivers/net/ethernet/intel/i40e/i40e_main.c??????? | 109
> +++++++++++++--------
> ?drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c |? 14 +--
> ?2 files changed, 75 insertions(+), 48 deletions(-)

This does not apply cleanly, so I am not applying this patch (or the
remaining patches in the series). ?Please send me an update to this
patch, which applies cleanly to me next-queue tree (dev-queue branch).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.osuosl.org/pipermail/intel-wired-lan/attachments/20151108/9e4bc7c5/attachment.asc>

  reply	other threads:[~2015-11-08 12:46 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-06 23:25 [Intel-wired-lan] [next PATCH S21 00/14] i40e/i40evf updates Joshua Hay
2015-11-06 23:25 ` [Intel-wired-lan] [next PATCH S21 01/14] i40evf: increase max number of queues Joshua Hay
2015-11-09 17:48   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 02/14] i40evf: set real num queues Joshua Hay
2015-11-09 18:02   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 03/14] i40evf: remove duplicate string Joshua Hay
2015-11-09 18:06   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 04/14] i40e: Detection and recovery of TX queue hung logic moved to service_task from tx_timeout Joshua Hay
2015-11-09 18:47   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 05/14] i40e: Fix memory leaks, sideband filter programming Joshua Hay
2015-11-09 19:02   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 06/14] i40evf: don't use atomic allocation Joshua Hay
2015-11-09 19:05   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 07/14] i40e: propagate properly Joshua Hay
2015-11-08 12:46   ` Jeff Kirsher [this message]
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 08/14] i40evf: use correct types Joshua Hay
2015-11-10 20:43   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 09/14] i40e: use priv flags to control packet split Joshua Hay
2015-11-10 20:48   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 10/14] i40e: remove CONFIG_I40E_VXLAN Joshua Hay
2015-11-10 18:04   ` Keller, Jacob E
2015-11-10 21:00   ` Bowers, AndrewX
2015-11-11  0:24   ` Singhai, Anjali
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 11/14] i40e: Remove separate functions gathering XOFF Rx stats Joshua Hay
2015-11-10 21:08   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 12/14] i40e: fix whitespace Joshua Hay
2015-11-10 21:09   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 13/14] i40e/i40evf: use logical operator Joshua Hay
2015-11-10 21:14   ` Bowers, AndrewX
2015-11-06 23:26 ` [Intel-wired-lan] [next PATCH S21 14/14] i40e/i40evf: Bump version to 1.4.7 for i40e and 1.4.3 for i40evf Joshua Hay
2015-11-10 21:15   ` Bowers, AndrewX

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=1446986786.4094.9.camel@intel.com \
    --to=jeffrey.t.kirsher@intel.com \
    --cc=intel-wired-lan@osuosl.org \
    /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