]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
3.13-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 10 Feb 2014 19:45:12 +0000 (11:45 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 10 Feb 2014 19:45:12 +0000 (11:45 -0800)
added patches:
iscsi-target-fix-connection-reset-hang-with-percpu_ida_alloc.patch

queue-3.13/iscsi-target-fix-connection-reset-hang-with-percpu_ida_alloc.patch [new file with mode: 0644]
queue-3.13/series

diff --git a/queue-3.13/iscsi-target-fix-connection-reset-hang-with-percpu_ida_alloc.patch b/queue-3.13/iscsi-target-fix-connection-reset-hang-with-percpu_ida_alloc.patch
new file mode 100644 (file)
index 0000000..bb1afdc
--- /dev/null
@@ -0,0 +1,70 @@
+From 555b270e25b0279b98083518a85f4b1da144a181 Mon Sep 17 00:00:00 2001
+From: Nicholas Bellinger <nab@linux-iscsi.org>
+Date: Mon, 20 Jan 2014 03:36:24 +0000
+Subject: iscsi-target: Fix connection reset hang with percpu_ida_alloc
+
+From: Nicholas Bellinger <nab@linux-iscsi.org>
+
+commit 555b270e25b0279b98083518a85f4b1da144a181 upstream.
+
+This patch addresses a bug where connection reset would hang
+indefinately once percpu_ida_alloc() was starved for tags, due
+to the fact that it always assumed uninterruptible sleep mode.
+
+So now make percpu_ida_alloc() check for signal_pending_state() for
+making interruptible sleep optional, and convert iscsit_allocate_cmd()
+to set TASK_INTERRUPTIBLE for GFP_KERNEL, or TASK_RUNNING for
+GFP_ATOMIC.
+
+Reported-by: Linus Torvalds <torvalds@linux-foundation.org>
+Cc: Kent Overstreet <kmo@daterainc.com>
+Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+
+---
+ drivers/target/iscsi/iscsi_target_util.c |    2 +-
+ lib/percpu_ida.c                         |    9 +++++++--
+ 2 files changed, 8 insertions(+), 3 deletions(-)
+
+--- a/drivers/target/iscsi/iscsi_target_util.c
++++ b/drivers/target/iscsi/iscsi_target_util.c
+@@ -156,7 +156,7 @@ struct iscsi_cmd *iscsit_allocate_cmd(st
+ {
+       struct iscsi_cmd *cmd;
+       struct se_session *se_sess = conn->sess->se_sess;
+-      int size, tag, state = (gfp_mask & __GFP_WAIT) ? TASK_UNINTERRUPTIBLE :
++      int size, tag, state = (gfp_mask & __GFP_WAIT) ? TASK_INTERRUPTIBLE :
+                               TASK_RUNNING;
+       tag = percpu_ida_alloc(&se_sess->sess_tag_pool, state);
+--- a/lib/percpu_ida.c
++++ b/lib/percpu_ida.c
+@@ -138,14 +138,14 @@ static inline unsigned alloc_local_tag(s
+  * tag_pool_init()), or otherwise -ENOSPC on allocation failure.
+  *
+  * Safe to be called from interrupt context (assuming it isn't passed
+- * TASK_UNINTERRUPTIBLE, of course).
++ * TASK_UNINTERRUPTIBLE | TASK_INTERRUPTIBLE, of course).
+  *
+  * @gfp indicates whether or not to wait until a free id is available (it's not
+  * used for internal memory allocations); thus if passed __GFP_WAIT we may sleep
+  * however long it takes until another thread frees an id (same semantics as a
+  * mempool).
+  *
+- * Will not fail if passed TASK_UNINTERRUPTIBLE.
++ * Will not fail if passed TASK_UNINTERRUPTIBLE | TASK_INTERRUPTIBLE.
+  */
+ int percpu_ida_alloc(struct percpu_ida *pool, int state)
+ {
+@@ -195,6 +195,11 @@ int percpu_ida_alloc(struct percpu_ida *
+               if (tag >= 0 || state == TASK_RUNNING)
+                       break;
++              if (signal_pending_state(state, current)) {
++                      tag = -ERESTARTSYS;
++                      break;
++              }
++
+               schedule();
+               local_irq_save(flags);
index 9262e680deffa6b3bd69b9d33f43b31a6a0e89bd..58ee0dd231139e5e0a6833189f25fd6113e48e4f 100644 (file)
@@ -60,5 +60,6 @@ drm-radeon-warn-users-when-hw_i2c-is-enabled-v2.patch
 drm-radeon-dpm-disable-mclk-switching-on-desktop-rv770.patch
 radeon-pm-guard-access-to-rdev-pm.power_state-array.patch
 percpu_ida-make-percpu_ida_alloc-callers-accept-task-state-bitmask.patch
+iscsi-target-fix-connection-reset-hang-with-percpu_ida_alloc.patch
 numa-add-a-sysctl-for-numa_balancing.patch
 mm-numa-initialise-numa-balancing-after-jump-label-initialisation.patch