From: Chris Zankel
The initiatorname field in se_acl_lun is only a copy of the same field
in se_node_acl, so remove it and use the version in se_node_acl where
needed (it's actually only used for pr_debug)
Signed-off-by: Chris Zankel
Signed-off-by: Spencer Baugh
---
drivers/t
From: Chris Zankel
The lun_flags field is not used, so drop it.
Signed-off-by: Chris Zankel
Signed-off-by: Spencer Baugh
---
include/target/target_core_base.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/target/target_core_base.h
b/include/target/target_core_base.h
index
rather than return an error.
Signed-off-by: Roland Dreier
Signed-off-by: Spencer Baugh
---
drivers/target/target_core_spc.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/drivers/target/target_core_spc.c b/drivers/target/target_core_spc.c
bind_addr, and set bound_dev_if to the destination address scope id
to correct it.
Signed-off-by: Spencer Baugh
---
drivers/infiniband/core/cma.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/infiniband/core/cma.c b/drivers/infiniband/core/cma.c
index 6a6b60a..
rather than return an error.
Signed-off-by: Roland Dreier
Signed-off-by: Spencer Baugh
---
drivers/target/target_core_spc.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/drivers/target/target_core_spc.c b/drivers/target/target_core_spc.c
From: Brian Bunker
AIX servers using VIOS servers that virtualize FC cards will have a
problem booting without support for START_STOP_UNIT.
v2: Cite sb3r36 exactly, clean up if conditions
Signed-off-by: Brian Bunker
Signed-off-by: Spencer Baugh
---
drivers/target/target_core_sbc.c | 36
b in there as soon as possible.
At a later point t_task_cdb is reset to the correct buffer,
but until then traces and printks don't cause a crash.
Signed-off-by: Alexei Potashnik
Signed-off-by: Spencer Baugh
---
drivers/target/iscsi/iscsi_target.c| 4 ++--
drivers/target/target_cor
From: Joern Engel
Code was responsible for ~150ms scheduler latencies.
Signed-off-by: Joern Engel
Signed-off-by: Spencer Baugh
---
drivers/net/ethernet/intel/e1000/e1000_hw.c | 13 -
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/drivers/net/ethernet/intel/e1000
better precision.
Signed-off-by: Joern Engel
Signed-off-by: Spencer Baugh
---
block/blk-timeout.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/block/blk-timeout.c b/block/blk-timeout.c
index 246dfb1..0d06162 100644
--- a/block/blk-timeout.c
+++ b/block/blk
+0xc0/0x150
[] ? finish_task_switch+0xc4/0xe0
[] do_notify_resume+0x65/0x80
[] retint_signal+0x4d/0x9f
---[ end trace 3506c05e4a0af3e5 ]---
Signed-off-by: Joern Engel
Signed-off-by: Spencer Baugh
---
lib/ioremap.c | 1 +
mm/memory.c | 1 +
mm/vmalloc.c | 1 +
3 files changed, 3 insertions
From: Joern Engel
Multiple nested loops. I have observed 590ms scheduler latency caused
by this loop and interrupts. Interrupts were responsible for 190ms, the
rest could have been avoided with a cond_resched.
Signed-off-by: Joern Engel
Signed-off-by: Spencer Baugh
---
drivers/pci/pcie/aer
From: Joern Engel
~150ms scheduler latency for both observed in the wild.
Signed-off-by: Joern Engel
Signed-off-by: Spencer Baugh
---
mm/hugetlb.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/mm/hugetlb.c b/mm/hugetlb.c
index a8c3087..2eb6919 100644
--- a/mm/hugetlb.c
+++ b/mm
there don't appear to be any other places that
use max_cmd_sn within the mutex more than once, so this lock can't be
providing any useful serialization.
v2: Get correct values for logging
v3: Fix whitespace damage
Signed-off-by: Roland Dreier
Signed-off-by: Spencer Baugh
---
drivers
there don't appear to be any other places that
use max_cmd_sn within the mutex more than once, so this lock can't be
providing any useful serialization.
v2: Get correct values for logging
Signed-off-by: Roland Dreier
Signed-off-by: Spencer Baugh
---
drivers/target/iscsi/
From: Roland Dreier
Solaris seems to go beserk if we respond to REPORT LUNS with a
LUN LIST LENGTH of 0.
Signed-off-by: Roland Dreier
Signed-off-by: Spencer Baugh
---
drivers/target/target_core_spc.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/target/target_core_spc.c
ands. The
Windows compliance test sends them.
Signed-off-by: Roland Dreier
Signed-off-by: Spencer Baugh
---
drivers/target/target_core_transport.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/target/target_core_transport.c
b/drivers/target/target_core_transpo
there don't appear to be any other places that
use max_cmd_sn within the mutex more than once, so this lock can't be
providing any useful serialization.
Signed-off-by: Roland Dreier
Signed-off-by: Spencer Baugh
---
drivers/target/iscsi/iscsi_target.c | 18 +---
From: Joern Engel
We don't touch the return value of qla2x00_loop_resync in
qla2x00_do_dpc, so don't bother collecting it. Fixes a compiler warning.
Signed-off-by: Joern Engel
Signed-off-by: Spencer Baugh
---
drivers/scsi/qla2xxx/qla_os.c | 3 +--
1 file changed, 1 insertion(+), 2
From: Joern Engel
Make the warning about unsupported SCSI opcode more useful:
- Add in the initiator name so we know who's sending it.
- Print the warning even for opcodes that spc_parse_cdb() knows about
but that we don't handle.
Signed-off-by: Joern Engel
Signed-off-by: Spe
Please disregard, depends on another patch. Will send again later. Sorry
for the noise.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read th
From: Dilip Kumar Uppugandla
Invoking get_cmd_state for qla2xxx always returns 0. Instead change it
to return the actual fabric state from qla_tgt_cmd. This will help with
debugging.
Signed-off-by: Dilip Kumar Uppugandla
Signed-off-by: Spencer Baugh
---
drivers/scsi/qla2xxx/tcm_qla2xxx.c | 6
Potashnik
Signed-off-by: Spencer Baugh
---
drivers/target/iscsi/iscsi_target.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/target/iscsi/iscsi_target.c
b/drivers/target/iscsi/iscsi_target.c
index a4cf58c..ebb1ece 100644
--- a/drivers/target/iscsi/iscsi_target.c
+++ b
From: Joern Engel
We have observed cases where the soft lockup detector triggered, but no
kernel bug existed. Instead we had a buggy realtime thread that
monopolized a cpu. So let's kill the responsible party and not panic
the entire system.
Signed-off-by: Joern Engel
Signed-off-by: Sp
b in there as soon as possible.
At a later point t_task_cdb is reset to the correct buffer,
but until then traces and printks don't cause a crash.
Signed-off-by: Alexei Potashnik
Signed-off-by: Spencer Baugh
---
drivers/target/target_core_device.c| 7 +++
drivers/target/target_co
From: Brian Bunker
AIX servers using VIOS servers that virtualize FC cards will have a
problem booting without support for START_STOP_UNIT.
Signed-off-by: Brian Bunker
Signed-off-by: Spencer Baugh
---
drivers/target/target_core_sbc.c | 33 +
1 file changed, 33
Christoph Hellwig writes:
> Looks good,
>
> but what's up with your From lines:
>
> On Mon, Jul 20, 2015 at 04:29:49PM -0700, Spencer Baugh wrote:
>> From: Spencer Baugh
>>
>> From: Chris Zankel
>
> plus another address for you in the actual ema
From: Alexei Potashnik
Current implementation assumes that all the buffers of an IO are linked
with a single SG list. Which makes it fail if SG chaining is used.
Signed-off-by: Alexei Potashnik
---
drivers/target/iscsi/iscsi_target.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(
From: Spencer Baugh
From: Chris Zankel
The lun_flags field is not used, so drop it.
Signed-off-by: Chris Zankel
---
include/target/target_core_base.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/target/target_core_base.h
b/include/target/target_core_base.h
index 17ae2d6
From: Spencer Baugh
From: Chris Zankel
The initiatorname field in se_acl_lun is only a copy of the same field
in se_node_acl, so remove it and use the version in se_node_acl where
needed (it's actually only used for pr_debug)
Signed-off-by: Chris Zankel
---
drivers/t
c?
drivers/staging/lustre/lustre/ptlrpc/sec_gc.c:241:6: warning: symbol
'sptlrpc_gc_fini' was not declared. Should it be static?
Signed-off-by: Spencer Baugh
---
drivers/staging/lustre/lustre/ptlrpc/sec_gc.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/
Fix errors reported by checkpatch of this kind:
ERROR: "foo * bar" should be "foo *bar"
Signed-off-by: Spencer Baugh
---
drivers/staging/lustre/lustre/include/lustre_import.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/l
Fix errors reported by checkpatch of this kind:
ERROR: open brace '{' following function declarations go on the next line
Signed-off-by: Spencer Baugh
---
drivers/staging/lustre/lustre/include/lustre_import.h | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git
Fix errors reported by checkpatch of this kind:
ERROR: "foo * bar" should be "foo *bar"
Signed-off-by: Spencer Baugh
---
drivers/staging/lustre/lustre/include/lustre_import.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/l
Fix errors reported by checkpatch of this kind:
ERROR: trailing whitespace
Signed-off-by: Spencer Baugh
---
drivers/staging/lustre/lustre/include/lustre_import.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_import.h
b
Fix errors reported by checkpatch of this kind:
ERROR: open brace '{' following function declarations go on the next line
Signed-off-by: Spencer Baugh
---
drivers/staging/lustre/lustre/include/lustre_import.h | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git
This patch fixes style errors and warnings reported by
scripts/checkpatch.pl
Signed-off-by: Spencer Baugh
---
.../staging/lustre/lustre/include/lustre_import.h | 21 -
1 file changed, 12 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include
36 matches
Mail list logo
