From: Jesse Brandeburg <jesse.brandeburg@intel.com>
To: intel-wired-lan@osuosl.org
Subject: [Intel-wired-lan] [next PATCH S26 06/12] i40e: fix: do not sleep in netdev_ops
Date: Mon, 4 Jan 2016 12:02:05 -0800 [thread overview]
Message-ID: <20160104120205.00000b51@intel.com> (raw)
In-Reply-To: <1451932392-15538-7-git-send-email-deepthi.kavalur@intel.com>
On Mon, 4 Jan 2016 10:33:06 -0800
Deepthi Kavalur <deepthi.kavalur@intel.com> wrote:
> From: Jesse Brandeburg <jesse.brandeburg@intel.com>
>
> The driver was being called by vlan, bonding, teaming operations
> that expected to be able to hold locks like rcu_read_lock().
>
> This causes the driver to be held to the requirement to not sleep,
> and was found by the kernel debug options for checking sleep
> inside critical section, and the locking validator.
>
> Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
> Reviewed-by: Nelson, Shannon <shannon.nelson@intel.com>
> Change-ID: Ibc68c835f5ffa8ffe0638ffe910a66fc5649a7f7
I don't understand how you ever applied this in your testing and built
a kernel, as this change (the complete version) was already applied in
Jeff's tree back on 2-Dec:
https://git.kernel.org/cgit/linux/kernel/git/jkirsher/next-queue.git/commit/drivers/net/ethernet/intel/i40e?id=0e4425ed641f3eef67c892bc541949cd745a9ba9
Likely this patch can just be dropped, but we need a process check here.
next prev parent reply other threads:[~2016-01-04 20:02 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-04 18:33 [Intel-wired-lan] [next PATCH S26 00/12] i40e/i40evf updates Deepthi Kavalur
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 01/12] i40e: trivial: drop duplicate definition Deepthi Kavalur
2016-01-07 15:46 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 02/12] i40e: trivial: fix missing space Deepthi Kavalur
2016-01-07 15:48 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 03/12] i40e: fix bug in dma sync Deepthi Kavalur
2016-01-07 22:37 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 04/12] i40e: do TSO only if CHECKSUM_PARTIAL is set Deepthi Kavalur
2016-01-07 18:31 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 05/12] i40e: allocate memory safer Deepthi Kavalur
2016-01-07 22:40 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 06/12] i40e: fix: do not sleep in netdev_ops Deepthi Kavalur
2016-01-04 20:02 ` Jesse Brandeburg [this message]
2016-01-06 1:39 ` Kavalur, Deepthi
2016-01-06 2:25 ` Brandeburg, Jesse
2016-01-13 19:13 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 07/12] i40e: APIs to Add/remove port mirroring rules Deepthi Kavalur
2016-01-07 19:03 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 08/12] i40e: negate phy int mask bits Deepthi Kavalur
2016-01-07 19:20 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 09/12] i40e: drop unused function Deepthi Kavalur
2016-01-07 19:26 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 10/12] i40e: count allocation errors Deepthi Kavalur
2016-01-07 19:28 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 11/12] i40e: avoid large memcpy by assigning struct Deepthi Kavalur
2016-01-07 19:30 ` Bowers, AndrewX
2016-01-04 18:33 ` [Intel-wired-lan] [next PATCH S26 12/12] i40e/i40evf: bump version to 1.4.12/1.4.8 Deepthi Kavalur
2016-01-07 19:31 ` 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=20160104120205.00000b51@intel.com \
--to=jesse.brandeburg@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.