All of 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.